Attention is currently required from: Cliff Huang, Intel coreboot Reviewers, Jérémy Compostella, Nick Vaccaro, Sean Rhodes.
Matt DeVillier has posted comments on this change by Sean Rhodes. ( https://review.coreboot.org/c/coreboot/+/86403?usp=email )
Change subject: soc/intel/common: Add support for RTD3 on CNVi
......................................................................
Patch Set 21:
(1 comment)
Patchset:
PS20:
> should we check if Chipset has support for RTD3 or not ?
…
[View More]might not be necessary, since everything is conditional (CondRefOf) on the control method, and that being present depends on the GPIO being defined in devicetree, so it's a no-op for all boards ATM
--
To view, visit https://review.coreboot.org/c/coreboot/+/86403?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I22292ad97c439e50fe5d7a6b79f77847e71ca62c
Gerrit-Change-Number: 86403
Gerrit-PatchSet: 21
Gerrit-Owner: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Reviewer: Intel coreboot Reviewers <intel_coreboot_reviewers(a)intel.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Jérémy Compostella <jeremy.compostella(a)intel.com>
Gerrit-CC: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Attention: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Attention: Intel coreboot Reviewers <intel_coreboot_reviewers(a)intel.com>
Gerrit-Attention: Jérémy Compostella <jeremy.compostella(a)intel.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Comment-Date: Tue, 18 Feb 2025 15:31:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik <subratabanik(a)google.com>
[View Less]
Attention is currently required from: Patrick Rudolph.
Sean Rhodes has posted comments on this change by Sean Rhodes. ( https://review.coreboot.org/c/coreboot/+/86485?usp=email )
Change subject: drivers/usb/intel_bluetooth: Correct WASSERT2 usage
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
> Can't you simply set the 'invert' bit on the GPIO pad?
Of course - but it's technically wrong
--
To view, visit https://…
[View More]review.coreboot.org/c/coreboot/+/86485?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: If8a6e0c00e799cdcb1af3f9edd90a3e5db2fcd17
Gerrit-Change-Number: 86485
Gerrit-PatchSet: 2
Gerrit-Owner: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Comment-Date: Tue, 18 Feb 2025 15:27:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph(a)9elements.com>
[View Less]
Attention is currently required from: Arthur Heymans, Felix Held.
Benjamin Doron has posted comments on this change by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/69792?usp=email )
Change subject: payloads/Kconfig: Don't show FIT payload option on x86
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
Can you hold off on this until we merge https://review.coreboot.org/c/coreboot/+/84796? We're about to make FIT …
[View More]support arch-independent
--
To view, visit https://review.coreboot.org/c/coreboot/+/69792?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I7afc3b94c5ab8044af0920c87cc8d190467c2a88
Gerrit-Change-Number: 69792
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Tue, 18 Feb 2025 15:20:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
[View Less]
Attention is currently required from: Sean Rhodes.
Patrick Rudolph has posted comments on this change by Sean Rhodes. ( https://review.coreboot.org/c/coreboot/+/86485?usp=email )
Change subject: drivers/usb/intel_bluetooth: Correct WASSERT2 usage
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
Can't you simply set the 'invert' bit on the GPIO pad?
--
To view, visit https://review.coreboot.org/c/coreboot/+/86485?usp=email…
[View More]
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: If8a6e0c00e799cdcb1af3f9edd90a3e5db2fcd17
Gerrit-Change-Number: 86485
Gerrit-PatchSet: 2
Gerrit-Owner: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Attention: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Comment-Date: Tue, 18 Feb 2025 14:44:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
[View Less]
Attention is currently required from: Cliff Huang, Lance Zhao, Matt DeVillier, Sean Rhodes, Tim Wawrzynczak.
Patrick Rudolph has posted comments on this change by Matt DeVillier. ( https://review.coreboot.org/c/coreboot/+/86427?usp=email )
Change subject: src/acpi/acpigen: Increase LENSTACK_SIZE from 10 to 15
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/86427?usp=email
To …
[View More]unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I8362b3a63a23bea0ce47920e5d41cd2535dbc084
Gerrit-Change-Number: 86427
Gerrit-PatchSet: 3
Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Reviewer: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Attention: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Attention: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Attention: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Comment-Date: Tue, 18 Feb 2025 14:43:11 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
[View Less]
Attention is currently required from: Ana Carolina Cabral, Felix Held, Fred Reitberger, Jason Glenesk, Matt DeVillier, Nick Kochlowski, Paul Menzel.
Patrick Rudolph has posted comments on this change by Nick Kochlowski. ( https://review.coreboot.org/c/coreboot/+/85719?usp=email )
Change subject: soc/amd/phoenix: Remove outdated TODO comments
......................................................................
Patch Set 16:
(1 comment)
File src/soc/amd/phoenix/Kconfig:
https://review.…
[View More]coreboot.org/c/coreboot/+/85719/comment/642b862e_ca9fc543?us… :
PS16, Line 30: TODO
What about the other todos?
--
To view, visit https://review.coreboot.org/c/coreboot/+/85719?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I137dbba5094ae8cbf842b45d6137c5b0528e5413
Gerrit-Change-Number: 85719
Gerrit-PatchSet: 16
Gerrit-Owner: Nick Kochlowski <nickkochlowski(a)gmail.com>
Gerrit-Reviewer: Ana Carolina Cabral <ana.cpmelo95(a)gmail.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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)amd.corp-partner.google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Nick Kochlowski <nickkochlowski(a)gmail.com>
Gerrit-Attention: Fred Reitberger <reitbergerfred(a)gmail.com>
Gerrit-Attention: Ana Carolina Cabral <ana.cpmelo95(a)gmail.com>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Tue, 18 Feb 2025 14:35:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
[View Less]
Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/86498?usp=email )
Change subject: soc/amd/glinda: Assert on unsupported settings
......................................................................
soc/amd/glinda: Assert on unsupported settings
The FSP for glinda with version RXBCB0090C doesn't support ASPM L0s
and when a PCIe device is configured for L1 SS it never wakes and thus
is gone forever.
Assert when an unsupported configuration is …
[View More]used until the FSP has been
fixed.
Change-Id: If54542e17e6ac923b8d73bbb80712b9fdf07b44d
Signed-off-by: Patrick Rudolph <patrick.rudolph(a)9elements.com>
---
M src/soc/amd/glinda/fsp_m_params.c
1 file changed, 10 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/98/86498/1
diff --git a/src/soc/amd/glinda/fsp_m_params.c b/src/soc/amd/glinda/fsp_m_params.c
index 1d297d4..92614ce 100644
--- a/src/soc/amd/glinda/fsp_m_params.c
+++ b/src/soc/amd/glinda/fsp_m_params.c
@@ -58,6 +58,16 @@
mainboard_get_dxio_ddi_descriptors(&fsp_dxio, &num_dxio,
&fsp_ddi, &num_ddi);
+
+ for (size_t i = 0; i < num_dxio; i++) {
+ // ASPM L0s isn't supported by FSP
+ assert (fsp_dxio[i].link_aspm != ASPM_L0s && fsp_dxio[i].link_aspm != ASPM_L0sL1);
+
+ // Device in L1 SS never wake...
+ assert (!fsp_dxio[i].link_aspm_L1_1);
+ assert (!fsp_dxio[i].link_aspm_L1_2);
+ }
+
fill_dxio_descriptors(mcfg, fsp_dxio, num_dxio);
fill_ddi_descriptors(mcfg, fsp_ddi, num_ddi);
}
--
To view, visit https://review.coreboot.org/c/coreboot/+/86498?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: If54542e17e6ac923b8d73bbb80712b9fdf07b44d
Gerrit-Change-Number: 86498
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
[View Less]
Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/86496?usp=email )
Change subject: mb/amd/birman_plus/glinda: Add onboard devices
......................................................................
mb/amd/birman_plus/glinda: Add onboard devices
Add SD Express and GBE PCIe devices.
Change-Id: Ia589f115fc5c16540daa6210e2624572767ad12e
Signed-off-by: Patrick Rudolph <patrick.rudolph(a)9elements.com>
---
M src/mainboard/amd/birman_plus/…
[View More]devicetree_glinda.cb
1 file changed, 6 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/96/86496/1
diff --git a/src/mainboard/amd/birman_plus/devicetree_glinda.cb b/src/mainboard/amd/birman_plus/devicetree_glinda.cb
index 07642aa..5a7dd16 100644
--- a/src/mainboard/amd/birman_plus/devicetree_glinda.cb
+++ b/src/mainboard/amd/birman_plus/devicetree_glinda.cb
@@ -187,9 +187,13 @@
device domain 0 on
device ref iommu on end
device ref gpp_bridge_2_1 on end # NVME SSD0
- device ref gpp_bridge_2_2 on end # SD
+ device ref gpp_bridge_2_2 on # SD Express
+ device pci 0.0 on end # RTS5261
+ end
device ref gpp_bridge_2_3 on end # WLAN
- device ref gpp_bridge_2_4 on end # GBE
+ device ref gpp_bridge_2_4 on # GBE
+ device pci 0.0 on end # RTL8111
+ end
device ref gpp_bridge_2_5 on end # WWAN
device ref gpp_bridge_3_1 on end # PCIe x4/x8 (ENABLE_SSD1_BIRMANPLUS)
device ref gpp_bridge_3_2 on end # NVME SSD1 (ENABLE_SSD1_BIRMANPLUS)
--
To view, visit https://review.coreboot.org/c/coreboot/+/86496?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia589f115fc5c16540daa6210e2624572767ad12e
Gerrit-Change-Number: 86496
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
[View Less]