Attention is currently required from: Julius Werner.
Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54737 )
Change subject: cbmem: Introduce "early" init hooks for console
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
File src/include/cbmem.h:
https://review.coreboot.org/c/coreboot/+/54737/comment/85fcc495_a80c6889
PS1, Line 150: _unused_
In file included from src/lib/cbmem_console.c:5:
src/lib/cbmem_console.c:153:26: error: redefinition of 'cbmemc_reinit_unused_'
RAMSTAGE_CBMEM_INIT_HOOK(cbmemc_reinit)
^~~~~~~~~~~~~
src/include/cbmem.h:129:27: note: in definition of macro 'RAMSTAGE_CBMEM_INIT_HOOK'
static cbmem_init_hook_t init_fn_ ## _unused_ = init_fn_;
^~~~~~~~
src/lib/cbmem_console.c:152:32: note: previous definition of 'cbmemc_reinit_unused_' was here
ROMSTAGE_CBMEM_INIT_HOOK_EARLY(cbmemc_reinit)
^~~~~~~~~~~~~
src/include/cbmem.h:150:27: note: in definition of macro 'ROMSTAGE_CBMEM_INIT_HOOK_EARLY'
static cbmem_init_hook_t init_fn_ ## _unused_ = init_fn_;
Add 'early' to unused so it doesn't collide? Should fix the compilation error right up.
--
To view, visit https://review.coreboot.org/c/coreboot/+/54737
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If2c849a89f07a87d448ec1edbad4ce404afb0746
Gerrit-Change-Number: 54737
Gerrit-PatchSet: 1
Gerrit-Owner: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Comment-Date: Thu, 27 May 2021 19:18:29 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Tim Wawrzynczak, Patrick Rudolph.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55021 )
Change subject: soc/intel/alderlake: Add placeholder SPD file
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/55021
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I38eb4bb684c511fff5ae148091c066682e9c35cb
Gerrit-Change-Number: 55021
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Thu, 27 May 2021 19:02:55 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Selma Bensaid, Paul Menzel, Bernardo Perez Priego.
Bora Guvendik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54490 )
Change subject: mb/intel/adlrvp_m: add ec device entry to devicetree
......................................................................
Patch Set 4:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/54490/comment/73e05703_6fb0adf3
PS3, Line 9: TEST=Boot to OS.
> and check what log or command output?
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/54490
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3c78ac44afa3515acef9ea2d59f22f95e6b45e90
Gerrit-Change-Number: 54490
Gerrit-PatchSet: 4
Gerrit-Owner: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Bernardo Perez Priego <bernardo.perez.priego(a)intel.com>
Gerrit-Reviewer: John Zhao <john.zhao(a)intel.com>
Gerrit-Reviewer: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Selma Bensaid <selma.bensaid(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Selma Bensaid <selma.bensaid(a)intel.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Bernardo Perez Priego <bernardo.perez.priego(a)intel.com>
Gerrit-Comment-Date: Thu, 27 May 2021 18:50:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: comment
Attention is currently required from: Bora Guvendik, Selma Bensaid, Bernardo Perez Priego.
Hello build bot (Jenkins), Selma Bensaid, John Zhao, John Zhao, Tim Wawrzynczak, Bernardo Perez Priego,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/54490
to look at the new patch set (#4).
Change subject: mb/intel/adlrvp_m: add ec device entry to devicetree
......................................................................
mb/intel/adlrvp_m: add ec device entry to devicetree
TEST=Boot to OS and verify acpi tables.
Signed-off-by: Bora Guvendik <bora.guvendik(a)intel.com>
Change-Id: I3c78ac44afa3515acef9ea2d59f22f95e6b45e90
---
M src/mainboard/intel/adlrvp/variants/adlrvp_m_ext_ec/overridetree.cb
1 file changed, 7 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/90/54490/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/54490
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3c78ac44afa3515acef9ea2d59f22f95e6b45e90
Gerrit-Change-Number: 54490
Gerrit-PatchSet: 4
Gerrit-Owner: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Bernardo Perez Priego <bernardo.perez.priego(a)intel.com>
Gerrit-Reviewer: John Zhao <john.zhao(a)intel.com>
Gerrit-Reviewer: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Selma Bensaid <selma.bensaid(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Attention: Selma Bensaid <selma.bensaid(a)intel.com>
Gerrit-Attention: Bernardo Perez Priego <bernardo.perez.priego(a)intel.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Furquan Shaikh, Paul Menzel, Scott Chao.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54941 )
Change subject: mb/google/brya: Create primus variant
......................................................................
Patch Set 4: Code-Review+1
(1 comment)
Patchset:
PS4:
Do you mind rebasing on top of CB:55021
--
To view, visit https://review.coreboot.org/c/coreboot/+/54941
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I441e4f0ebea3b9db813e321d32ca990e9c51c668
Gerrit-Change-Number: 54941
Gerrit-PatchSet: 4
Gerrit-Owner: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Casper Chang <casper_chang(a)wistron.corp-partner.google.com>
Gerrit-CC: Malik Hsu <malik_hsu(a)wistron.corp-partner.google.com>
Gerrit-CC: Mark Hsieh <mark_hsieh(a)wistron.corp-partner.google.com>
Gerrit-CC: Nick Chen <nick_xr_chen(a)wistron.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Furquan Shaikh <furquan(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-Comment-Date: Thu, 27 May 2021 18:46:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Furquan Shaikh, Paul Menzel, Scott Chao.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54941 )
Change subject: mb/google/brya: Create primus variant
......................................................................
Patch Set 4:
(1 comment)
File src/mainboard/google/brya/variants/primus/memory/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/54941/comment/0a93fdad_02ecc7bf
PS2, Line 5: placeholder.spd.hex
> You would need one in adl/spd too, right?
Maybe? Why does it still build... ?
--
To view, visit https://review.coreboot.org/c/coreboot/+/54941
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I441e4f0ebea3b9db813e321d32ca990e9c51c668
Gerrit-Change-Number: 54941
Gerrit-PatchSet: 4
Gerrit-Owner: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Casper Chang <casper_chang(a)wistron.corp-partner.google.com>
Gerrit-CC: Malik Hsu <malik_hsu(a)wistron.corp-partner.google.com>
Gerrit-CC: Mark Hsieh <mark_hsieh(a)wistron.corp-partner.google.com>
Gerrit-CC: Nick Chen <nick_xr_chen(a)wistron.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Furquan Shaikh <furquan(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-Comment-Date: Thu, 27 May 2021 18:42:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh <furquan(a)google.com>
Comment-In-Reply-To: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Comment-In-Reply-To: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-MessageType: comment
Francois Toguo Fotso has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49799 )
Change subject: acpi: Add support for reporting CrashLog in BERT table
......................................................................
Patch Set 10:
(1 comment)
Patchset:
PS10:
> since there's bert_errors_present() that returns the state of the bert memory region, i don't really […]
The Intel SOC flow to gather and check the data is quite different from the AMD's one it seems.
We are now moving into different directions, which confuses me a bit.
Once again could you please provide the link to the specfic common code which was being duplicated by https://review.coreboot.org/c/coreboot/+/49799/10/src/acpi/acpi.c#1585 ?
--
To view, visit https://review.coreboot.org/c/coreboot/+/49799
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I00e390d735d61beac2e89a726e39119d9b06b3df
Gerrit-Change-Number: 49799
Gerrit-PatchSet: 10
Gerrit-Owner: Francois Toguo Fotso <francois.toguo.fotso(a)intel.com>
Gerrit-Reviewer: Lance Zhao
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Nikunj Dadhania <nikunj.dadhania(a)intel.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: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: 9elements QA <hardwaretestrobot(a)gmail.com>
Gerrit-CC: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-CC: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Raj Astekar <raj.astekar(a)intel.com>
Gerrit-Comment-Date: Thu, 27 May 2021 18:39:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Francois Toguo Fotso <francois.toguo.fotso(a)intel.com>
Comment-In-Reply-To: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-MessageType: comment
Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Aaron Durbin, Karthik Ramasubramanian.
Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54098 )
Change subject: ec/google/chromeec: OEM variables support for DPTF
......................................................................
Patch Set 3:
(2 comments)
File src/ec/google/chromeec/ec_dptf_helpers.c:
https://review.coreboot.org/c/coreboot/+/54098/comment/f1822c64_5145979e
PS2, Line 314: ODVP
> I am a little confused. […]
write_ietm() is to populate those ODVP0-5 from override as mentioned in CL 54279
https://review.coreboot.org/c/coreboot/+/54098/comment/546f1d4d_f22832c9
PS2, Line 315: ODV0
> Where are these ODV0-5 defined? Where do these objects exist?
These object there in ec.asl under CL 51937
--
To view, visit https://review.coreboot.org/c/coreboot/+/54098
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I79fbf00521d54e66b6cd7d95b026a76fd3a4394b
Gerrit-Change-Number: 54098
Gerrit-PatchSet: 3
Gerrit-Owner: Sumeet R Pawnikar <sumeet.r.pawnikar(a)intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Furquan Shaikh <furquan(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Attention: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Comment-Date: Thu, 27 May 2021 18:26:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh <furquan(a)google.com>
Gerrit-MessageType: comment
Attention is currently required from: Paul Menzel, Scott Chao.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54941 )
Change subject: mb/google/brya: Create primus variant
......................................................................
Patch Set 4:
(1 comment)
File src/mainboard/google/brya/variants/primus/memory/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/54941/comment/4415648b_db77e8c0
PS2, Line 5: placeholder.spd.hex
> `HAVE_SPD_IN_CBFS is set but SPD_SOURCES is empty` […]
You would need one in adl/spd too, right?
--
To view, visit https://review.coreboot.org/c/coreboot/+/54941
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I441e4f0ebea3b9db813e321d32ca990e9c51c668
Gerrit-Change-Number: 54941
Gerrit-PatchSet: 4
Gerrit-Owner: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Casper Chang <casper_chang(a)wistron.corp-partner.google.com>
Gerrit-CC: Malik Hsu <malik_hsu(a)wistron.corp-partner.google.com>
Gerrit-CC: Mark Hsieh <mark_hsieh(a)wistron.corp-partner.google.com>
Gerrit-CC: Nick Chen <nick_xr_chen(a)wistron.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-Comment-Date: Thu, 27 May 2021 18:20:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh <furquan(a)google.com>
Comment-In-Reply-To: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Comment-In-Reply-To: Scott Chao <scott_chao(a)wistron.corp-partner.google.com>
Gerrit-MessageType: comment