Attention is currently required from: Patrick Rudolph, Arthur Heymans.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68814 )
Change subject: mb/prodrive/hermes: Remove FSP workaround
......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/68814/comment/a469934d_a2511756
PS1, Line 7: mb/prodrive/hermes: Remove FSP workaround
Maybe:
> Increase CBFS size to 16 MB after FSP fix
https://review.coreboot.org/c/coreboot/+/68814/comment/792e43bc_78f10620
PS1, Line 9: FSP releases
Would be nice if you documented the first fixed release.
--
To view, visit https://review.coreboot.org/c/coreboot/+/68814
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If024a3de808c7d9f62171d75350545bc9fdeb9e0
Gerrit-Change-Number: 68814
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Comment-Date: Tue, 25 Oct 2022 14:51:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Christian Walter, Maximilian Brune, Angel Pons, Lean Sheng Tan.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68783 )
Change subject: mb/prodrive/atlas: Add IBECC Kconfig option
......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/68783/comment/748ead3f_5ba55543
PS1, Line 10: indurance testing
Is the test currently failing, or is the feature tested?
https://review.coreboot.org/c/coreboot/+/68783/comment/45101696_26239992
PS1, Line 10: Code), which is currently needed for indurance testing
Please add a dot/period at the end of sentences.
--
To view, visit https://review.coreboot.org/c/coreboot/+/68783
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I71ee2401136e2dc70b3164db6c99af03a3e1f346
Gerrit-Change-Number: 68783
Gerrit-PatchSet: 1
Gerrit-Owner: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Reviewer: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Attention: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Comment-Date: Tue, 25 Oct 2022 14:47:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Christian Walter, Tim Wawrzynczak, Maximilian Brune, Angel Pons, Lean Sheng Tan, David Milosevic.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68756 )
Change subject: src/soc/intel/alderlake: Add IBECC to Alderlake
......................................................................
Patch Set 5:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/68756/comment/67b21940_f466ce52
PS5, Line 7: Alderlake
Alder Lake, but it’s redundant as it’s in the prefix.
https://review.coreboot.org/c/coreboot/+/68756/comment/1e483e4a_f8224da9
PS5, Line 10: Board
board?
https://review.coreboot.org/c/coreboot/+/68756/comment/9355f93e_ea9181a7
PS5, Line 11:
Please elaborate. Is the FSP doing that?
Please add how and with what memory modules you tested this.
Patchset:
PS5:
Should something be logged, when this is enabled?
--
To view, visit https://review.coreboot.org/c/coreboot/+/68756
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9cc2ed6defa1223aa422b9b0d8145f8f8b3dd12e
Gerrit-Change-Number: 68756
Gerrit-PatchSet: 5
Gerrit-Owner: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Reviewer: David Milosevic <David.Milosevic(a)9elements.com>
Gerrit-Reviewer: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Attention: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Attention: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Attention: David Milosevic <David.Milosevic(a)9elements.com>
Gerrit-Comment-Date: Tue, 25 Oct 2022 14:47:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Nikolai Vyssotski has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/49868 )
Change subject: soc/amd/picasso/Kconfig: turn on GOP
......................................................................
Abandoned
--
To view, visit https://review.coreboot.org/c/coreboot/+/49868
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6fb8c36174ef7615211a8efddbd4c9fb5d374fd3
Gerrit-Change-Number: 49868
Gerrit-PatchSet: 15
Gerrit-Owner: Nikolai Vyssotski <nikolai.vyssotski(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: abandon
Attention is currently required from: Frank Chu, Nick Vaccaro.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68575 )
Change subject: mb/google/brya: Create marasov variant
......................................................................
Patch Set 3:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/68575/comment/970fe1a2_ce2316ac
PS3, Line 2: FrankChu
Frank Chu
git commit --amend --author="Frank Chu <frank_chu(a)pegatron.corp-partner.google.com>"
https://review.coreboot.org/c/coreboot/+/68575/comment/ac2a4f11_a394b9ef
PS3, Line 12: (Auto-Generated by create_coreboot_variant.sh version 4.5.0).
Please move the . inside, and fix the script.
--
To view, visit https://review.coreboot.org/c/coreboot/+/68575
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib2e30c203ee0dfa6c887a707e9b5074ce89f57ac
Gerrit-Change-Number: 68575
Gerrit-PatchSet: 3
Gerrit-Owner: Frank Chu <frank_chu(a)pegatron.corp-partner.google.com>
Gerrit-Reviewer: Frank Chu <frank_chu(a)pegatron.corp-partner.google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Ken Lu <ken_lu(a)pegatron.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Frank Chu <frank_chu(a)pegatron.corp-partner.google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Comment-Date: Tue, 25 Oct 2022 14:45:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Jason Glenesk, Fred Reitberger, Felix Held.
Nikolai Vyssotski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68802 )
Change subject: mainboard/amd/chausie: Don't use APCB_FT6_Updatable
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
> Does the APCB in 3rdparty/blobs need to be renamed, or is this only for site-local work?
It should be renamed or removed. This name implies that we have "Updateable" (RW) APCB binary which s not the case. We only have one APCB binary which is used for both default (0x60) and recovery (0x68) APCBs.
--
To view, visit https://review.coreboot.org/c/coreboot/+/68802
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4d317ae31cf226b5481619f1539abb6237033f7c
Gerrit-Change-Number: 68802
Gerrit-PatchSet: 2
Gerrit-Owner: Nikolai Vyssotski <nikolai.vyssotski(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Attention: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Tue, 25 Oct 2022 14:44:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Jason Glenesk, Raul Rangel, Matt DeVillier, Fred Reitberger, Felix Held.
Nikolai Vyssotski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68801 )
Change subject: src/soc/amd/mendocino: Enable override of MAINBOARD_BLOBS_DIR
......................................................................
Patch Set 3:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/68801/comment/286b2933_f405c8bc
PS2, Line 7: site-local: Enable override of MAINBOARD_BLOBS_DIR
> nit: this is soc/amd/mendocino more than site-local
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/68801
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I028042b947887d1182642ad4482dd1bba7ad8e23
Gerrit-Change-Number: 68801
Gerrit-PatchSet: 3
Gerrit-Owner: Nikolai Vyssotski <nikolai.vyssotski(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Attention: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)amd.corp-partner.google.com>
Gerrit-Attention: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Tue, 25 Oct 2022 14:34:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Jason Glenesk, Raul Rangel, Matt DeVillier, Nikolai Vyssotski, Felix Held.
Hello build bot (Jenkins), Jason Glenesk, Raul Rangel, Matt DeVillier, Fred Reitberger, Felix Held,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/68801
to look at the new patch set (#3).
Change subject: src/soc/amd/mendocino: Enable override of MAINBOARD_BLOBS_DIR
......................................................................
src/soc/amd/mendocino: Enable override of MAINBOARD_BLOBS_DIR
When using site-local we need to have ability to override
MAINBOARD_BLOBS_DIR with a different location (presumably somewhere in
site-local). site-local Makefiles.inc should be pulled in first
(different CL) allowing MAINBOARD_BLOBS_DIR to be overwritten.
Change-Id: I028042b947887d1182642ad4482dd1bba7ad8e23
Signed-off-by: Nikolai Vyssotski <nikolai.vyssotski(a)amd.corp-partner.google.com>
---
M src/soc/amd/mendocino/Makefile.inc
1 file changed, 18 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/01/68801/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/68801
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I028042b947887d1182642ad4482dd1bba7ad8e23
Gerrit-Change-Number: 68801
Gerrit-PatchSet: 3
Gerrit-Owner: Nikolai Vyssotski <nikolai.vyssotski(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Attention: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)amd.corp-partner.google.com>
Gerrit-Attention: Nikolai Vyssotski <nikolai.vyssotski(a)amd.corp-partner.google.com>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: Shelley Chen, Taniya Das, Julius Werner, Sudheer Amrabadi.
Venkat Thogaru has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68384 )
Change subject: soc/qualcomm/sc7280: Add socinfo_pro_part() function in coreboot
......................................................................
Patch Set 6:
(3 comments)
File src/soc/qualcomm/sc7280/socinfo.c:
https://review.coreboot.org/c/coreboot/+/68384/comment/af723466_8f002a97
PS4, Line 57: return false;
> I would consider putting a die() or at least an assert(0) at the end of these functions because it s […]
Done. Update in latest patch 6
https://review.coreboot.org/c/coreboot/+/68384/comment/df7afee2_b1d79858
PS4, Line 67: printk(BIOS_INFO, "jtagid = %x\n", jtagid);
> We already print the SKU ID that is constructed from this, we don't need to spam the log with so man […]
Done
https://review.coreboot.org/c/coreboot/+/68384/comment/cd38b0aa_e9b4bbd4
PS4, Line 70: printk(BIOS_INFO, "Pro_support: %d\n", chipinfolut[i].pro);
> same
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/68384
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id9f23696384a6c1a89000292eafebd8a16c273ca
Gerrit-Change-Number: 68384
Gerrit-PatchSet: 6
Gerrit-Owner: Sudheer Amrabadi <samrabad(a)codeaurora.org>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Taniya Das <quic_tdas(a)quicinc.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Julius Werner <jwerner(a)chromium.org>
Gerrit-CC: Venkat Thogaru <thogaru(a)qualcomm.corp-partner.google.com>
Gerrit-Attention: Shelley Chen <shchen(a)google.com>
Gerrit-Attention: Taniya Das <quic_tdas(a)quicinc.com>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Attention: Sudheer Amrabadi <samrabad(a)codeaurora.org>
Gerrit-Comment-Date: Tue, 25 Oct 2022 14:12:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Julius Werner <jwerner(a)chromium.org>
Gerrit-MessageType: comment