Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44952 )
Change subject: soc/intel/cnl: Add new Kconfig option which matches its FSPs name
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/44952
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I44b717bb942fbcd359c7a06ef1a0ef4306697f64
Gerrit-Change-Number: 44952
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Jeremy Soller <jeremy(a)system76.com>
Gerrit-Reviewer: Michael Niewöhner
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Tue, 01 Sep 2020 13:47:17 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44857 )
Change subject: soc/intel/alderlake/bootblock: Do initial SoC commit till bootblock
......................................................................
Patch Set 2:
@Tim, if you can take a look
--
To view, visit https://review.coreboot.org/c/coreboot/+/44857
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I204e692fabb84fce297bebee465f4ca624c6fe56
Gerrit-Change-Number: 44857
Gerrit-PatchSet: 2
Gerrit-Owner: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 01 Sep 2020 12:18:04 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Hello Jenny Ro,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/gerrit-avatars/+/44977
to review the following change.
Change subject: Add my avatar
......................................................................
Add my avatar
Signed-off-by: jennyro <jenny_ro(a)wistron.corp-partner.google.com>
Change-Id: I5a4ec9887704e6cef82a25831b5f7eedb2cfaab8
---
A 1002712.jpg
1 file changed, 0 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/gerrit-avatars refs/changes/77/44977/1
diff --git a/1002712.jpg b/1002712.jpg
new file mode 100644
index 0000000..a05f7be
--- /dev/null
+++ b/1002712.jpg
Binary files differ
--
To view, visit https://review.coreboot.org/c/gerrit-avatars/+/44977
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: gerrit-avatars
Gerrit-Branch: master
Gerrit-Change-Id: I5a4ec9887704e6cef82a25831b5f7eedb2cfaab8
Gerrit-Change-Number: 44977
Gerrit-PatchSet: 1
Gerrit-Owner: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-Reviewer: Jenny Ro <jenny_ro(a)wistron.corp-partner.google.com>
Gerrit-MessageType: newchange
Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40300 )
Change subject: mainboard: add Dell Latitude E7240
......................................................................
Patch Set 2:
Cool stuff. I have a Latitude 6430u I wanted to port, which have MEC5055 or MEC5054(?) too. Will gladly review it in free time.
--
To view, visit https://review.coreboot.org/c/coreboot/+/40300
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6933bdbcc8d0bbb85d62657624740266284ac71c
Gerrit-Change-Number: 40300
Gerrit-PatchSet: 2
Gerrit-Owner: Iru Cai (vimacs) <mytbk920423(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Tue, 01 Sep 2020 07:36:39 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Angel Pons, Iru Cai,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/40300
to look at the new patch set (#2).
Change subject: mainboard: add Dell Latitude E7240
......................................................................
mainboard: add Dell Latitude E7240
Based on autoport output.
It boots from USB and mSATA with SeaBIOS.
Change-Id: I6933bdbcc8d0bbb85d62657624740266284ac71c
Signed-off-by: Iru Cai <mytbk920423(a)gmail.com>
---
A src/mainboard/dell/latitude_e7240/Kconfig
A src/mainboard/dell/latitude_e7240/Kconfig.name
A src/mainboard/dell/latitude_e7240/Makefile.inc
A src/mainboard/dell/latitude_e7240/acpi/ec.asl
A src/mainboard/dell/latitude_e7240/acpi/platform.asl
A src/mainboard/dell/latitude_e7240/acpi/superio.asl
A src/mainboard/dell/latitude_e7240/acpi_tables.c
A src/mainboard/dell/latitude_e7240/board_info.txt
A src/mainboard/dell/latitude_e7240/bootblock.c
A src/mainboard/dell/latitude_e7240/devicetree.cb
A src/mainboard/dell/latitude_e7240/dsdt.asl
A src/mainboard/dell/latitude_e7240/gma-mainboard.ads
A src/mainboard/dell/latitude_e7240/gpio.c
A src/mainboard/dell/latitude_e7240/hda_verb.c
A src/mainboard/dell/latitude_e7240/mainboard.c
A src/mainboard/dell/latitude_e7240/romstage.c
M src/superio/smsc/Makefile.inc
17 files changed, 423 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/40300/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/40300
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6933bdbcc8d0bbb85d62657624740266284ac71c
Gerrit-Change-Number: 40300
Gerrit-PatchSet: 2
Gerrit-Owner: Iru Cai (vimacs) <mytbk920423(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset
Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43494 )
Change subject: soc/intel/common/cpu: Update COS mask calculation for NEM enhanced mode
......................................................................
Patch Set 4:
> Patch Set 4:
>
> Aamir, have you had a chance to test on other chipsets besides TGL ?
I was able to boot up the JSL SoC with this change. I am yet to verify the mask contents in MSR.
--
To view, visit https://review.coreboot.org/c/coreboot/+/43494
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I54e047161853bfc70516c1d607aa479e68836d04
Gerrit-Change-Number: 43494
Gerrit-PatchSet: 4
Gerrit-Owner: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: David Guckian <david.guckian(a)intel.com>
Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-CC: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Shreesh Chhabbi <shreesh.chhabbi(a)intel.com>
Gerrit-Comment-Date: Tue, 01 Sep 2020 06:55:11 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment