Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/74527 )
(
1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: soc/amd/phoenix: Mark PCIe GPP bridges as hidden instead of off ......................................................................
soc/amd/phoenix: Mark PCIe GPP bridges as hidden instead of off
When one of the General-Purpose PCIe bridges is not used, it doesn't show up on the PCI bus at all, so coreboot notes it as an issue in the devicetree. This happens even if the device is marked as off. To solve this, we're marking the GPP bridge devices in devicetree as hidden, so they'll only show up in devicetree if they're actually used on a mainboard.
BUG=b:277997811 TEST=Build
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: I7b7577baa2dbb0ea7ebbcdb1a8ae81770e61d76f Reviewed-on: https://review.coreboot.org/c/coreboot/+/74527 Reviewed-by: Matt DeVillier matt.devillier@amd.corp-partner.google.com Reviewed-by: Eric Lai eric_lai@quanta.corp-partner.google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/amd/phoenix/chipset.cb 1 file changed, 34 insertions(+), 10 deletions(-)
Approvals: build bot (Jenkins): Verified Eric Lai: Looks good to me, approved Matt DeVillier: Looks good to me, but someone else must approve
diff --git a/src/soc/amd/phoenix/chipset.cb b/src/soc/amd/phoenix/chipset.cb index 055e125..97b7ec1 100644 --- a/src/soc/amd/phoenix/chipset.cb +++ b/src/soc/amd/phoenix/chipset.cb @@ -11,19 +11,19 @@
device pci 01.0 on end # Dummy Host Bridge, do not disable # The PCIe GPP aliases in this SoC match the device and function numbers - device pci 01.1 alias gpp_bridge_1_1 off ops amd_external_pcie_gpp_ops end - device pci 01.2 alias gpp_bridge_1_2 off ops amd_external_pcie_gpp_ops end - device pci 01.3 alias gpp_bridge_1_3 off ops amd_external_pcie_gpp_ops end - device pci 01.4 alias gpp_bridge_1_4 off ops amd_external_pcie_gpp_ops end + device pci 01.1 alias gpp_bridge_1_1 hidden ops amd_external_pcie_gpp_ops end + device pci 01.2 alias gpp_bridge_1_2 hidden ops amd_external_pcie_gpp_ops end + device pci 01.3 alias gpp_bridge_1_3 hidden ops amd_external_pcie_gpp_ops end + device pci 01.4 alias gpp_bridge_1_4 hidden ops amd_external_pcie_gpp_ops end
device pci 02.0 on end # Dummy Host Bridge, do not disable # The PCIe GPP aliases in this SoC match the device and function numbers - device pci 02.1 alias gpp_bridge_2_1 off ops amd_external_pcie_gpp_ops end - device pci 02.2 alias gpp_bridge_2_2 off ops amd_external_pcie_gpp_ops end - device pci 02.3 alias gpp_bridge_2_3 off ops amd_external_pcie_gpp_ops end - device pci 02.4 alias gpp_bridge_2_4 off ops amd_external_pcie_gpp_ops end - device pci 02.5 alias gpp_bridge_2_5 off ops amd_external_pcie_gpp_ops end - device pci 02.6 alias gpp_bridge_2_6 off ops amd_external_pcie_gpp_ops end + device pci 02.1 alias gpp_bridge_2_1 hidden ops amd_external_pcie_gpp_ops end + device pci 02.2 alias gpp_bridge_2_2 hidden ops amd_external_pcie_gpp_ops end + device pci 02.3 alias gpp_bridge_2_3 hidden ops amd_external_pcie_gpp_ops end + device pci 02.4 alias gpp_bridge_2_4 hidden ops amd_external_pcie_gpp_ops end + device pci 02.5 alias gpp_bridge_2_5 hidden ops amd_external_pcie_gpp_ops end + device pci 02.6 alias gpp_bridge_2_6 hidden ops amd_external_pcie_gpp_ops end
device pci 03.0 on end # Dummy Host Bridge, do not disable device pci 03.1 alias usb4_pcie_bridge_0 off end