Attention is currently required from: Anilkumar N, Maulik V Vaghela, Kangheui Won, Tim Wawrzynczak, Reka Norman, Krishna P Bhat D.
Usha P has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60193 )
Change subject: mb/intel/adlrvp_n: Add support for ADL-N LP5 RVP
......................................................................
Patch Set 2:
(1 comment)
File src/mainboard/intel/adlrvp/spd/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/60193/comment/ab33a59b_93f92652
PS1, Line 10: SPD_SOURCES += adlrvp_n_lp5 # 0b007
> Can you please capture in commit msg? […]
Ack
--
To view, visit https://review.coreboot.org/c/coreboot/+/60193
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib2f53e65f75e23793d8c85ee924827446fd9fea7
Gerrit-Change-Number: 60193
Gerrit-PatchSet: 2
Gerrit-Owner: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com>
Gerrit-Reviewer: Anilkumar N <anilkumar.n(a)intel.com>
Gerrit-Reviewer: Kangheui Won <khwon(a)chromium.org>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Anilkumar N <anilkumar.n(a)intel.com>
Gerrit-Attention: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Attention: Kangheui Won <khwon(a)chromium.org>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Attention: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com>
Gerrit-Comment-Date: Mon, 20 Dec 2021 07:40:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-MessageType: comment
Attention is currently required from: Anilkumar N, Kangheui Won, Tim Wawrzynczak, Reka Norman, Krishna P Bhat D.
Usha P has uploaded a new patch set (#2) to the change originally created by Krishna P Bhat D. ( https://review.coreboot.org/c/coreboot/+/60193 )
Change subject: mb/intel/adlrvp_n: Add support for ADL-N LP5 RVP
......................................................................
mb/intel/adlrvp_n: Add support for ADL-N LP5 RVP
Add support for Alder lake N LP5 RVP with board ID 0x7.
Since SPD index 7 is unused earlier, ADL-N will use it.
Change-Id: Ib2f53e65f75e23793d8c85ee924827446fd9fea7
Signed-off-by: Krishna Prasad Bhat <krishna.p.bhat.d(a)intel.com>
---
M src/mainboard/intel/adlrvp/include/baseboard/variants.h
M src/mainboard/intel/adlrvp/memory.c
M src/mainboard/intel/adlrvp/romstage_fsp_params.c
M src/mainboard/intel/adlrvp/spd/Makefile.inc
A src/mainboard/intel/adlrvp/spd/adlrvp_n_lp5.spd.hex
5 files changed, 100 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/60193/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/60193
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib2f53e65f75e23793d8c85ee924827446fd9fea7
Gerrit-Change-Number: 60193
Gerrit-PatchSet: 2
Gerrit-Owner: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com>
Gerrit-Reviewer: Anilkumar N <anilkumar.n(a)intel.com>
Gerrit-Reviewer: Kangheui Won <khwon(a)chromium.org>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Anilkumar N <anilkumar.n(a)intel.com>
Gerrit-Attention: Kangheui Won <khwon(a)chromium.org>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Attention: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Anilkumar N, Kangheui Won, Tim Wawrzynczak, Reka Norman, Krishna P Bhat D.
Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60193 )
Change subject: mb/intel/adlrvp_n: Add support for ADL-N LP5 RVP
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File src/mainboard/intel/adlrvp/spd/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/60193/comment/b2132dde_484f03e3
PS1, Line 10: SPD_SOURCES += adlrvp_n_lp5 # 0b007
Can you please capture in commit msg?
We're utilizing SPD index 7 since it's been unused earlier and ADL-N will use it.
--
To view, visit https://review.coreboot.org/c/coreboot/+/60193
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib2f53e65f75e23793d8c85ee924827446fd9fea7
Gerrit-Change-Number: 60193
Gerrit-PatchSet: 1
Gerrit-Owner: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com>
Gerrit-Reviewer: Anilkumar N <anilkumar.n(a)intel.com>
Gerrit-Reviewer: Kangheui Won <khwon(a)chromium.org>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Anilkumar N <anilkumar.n(a)intel.com>
Gerrit-Attention: Kangheui Won <khwon(a)chromium.org>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Attention: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com>
Gerrit-Comment-Date: Mon, 20 Dec 2021 07:29:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58412
to look at the new patch set (#3).
Change subject: mb/google/dedede/var/cret: Generate new SPD ID for new memory parts
......................................................................
mb/google/dedede/var/cret: Generate new SPD ID for new memory parts
Add new memory parts in the mem_parts_used.txt and generate the
SPD ID for the parts. The memory parts being added are:
1. Samsung K4U6E3S4AB-MGCL
2. Hynix H54G46CYRBX267
BUG=b:203837656
BRANCH=dedede
TEST=FW_NAME=cret emerge-dedede coreboot chromeos-bootimage
Signed-off-by: Dtrain Hsu <dtrain_hsu(a)compal.corp-partner.google.com>
Change-Id: I03980858f89e56320ddff3a808110a5f1dd57784
---
M src/mainboard/google/dedede/variants/cret/memory/Makefile.inc
M src/mainboard/google/dedede/variants/cret/memory/dram_id.generated.txt
M src/mainboard/google/dedede/variants/cret/memory/mem_parts_used.txt
3 files changed, 5 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/58412/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/58412
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I03980858f89e56320ddff3a808110a5f1dd57784
Gerrit-Change-Number: 58412
Gerrit-PatchSet: 3
Gerrit-Owner: Dtrain Hsu <dtrain_hsu(a)compal.corp-partner.google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: newpatchset
Dtrain Hsu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58412 )
Change subject: mb/google/dedede/var/cret: Generate new SPD ID for new memory parts
......................................................................
Set Ready For Review
--
To view, visit https://review.coreboot.org/c/coreboot/+/58412
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I03980858f89e56320ddff3a808110a5f1dd57784
Gerrit-Change-Number: 58412
Gerrit-PatchSet: 2
Gerrit-Owner: Dtrain Hsu <dtrain_hsu(a)compal.corp-partner.google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 20 Dec 2021 07:22:12 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Sean Rhodes.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60229 )
Change subject: starlabs: Convert EC_GPE_SCI to Kconfig
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://review.coreboot.org/c/coreboot/+/60229
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8d47ebe76394fe1bcb217e0c6211db1566f82189
Gerrit-Change-Number: 60229
Gerrit-PatchSet: 3
Gerrit-Owner: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Comment-Date: Mon, 20 Dec 2021 06:58:13 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Alex James.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/vboot/+/60235 )
Change subject: cgpt: Add endian definitions for macOS
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://review.coreboot.org/c/vboot/+/60235
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: vboot
Gerrit-Branch: main
Gerrit-Change-Id: I87e98ea157870417f5eba63807eef559e180d1f8
Gerrit-Change-Number: 60235
Gerrit-PatchSet: 1
Gerrit-Owner: Alex James <theracermaster(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Alex James <theracermaster(a)gmail.com>
Gerrit-Comment-Date: Mon, 20 Dec 2021 06:57:00 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment