Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74526 )
Change subject: soc/amd/mendocino: Mark PCIe GPP bridges as hidden instead of off ......................................................................
soc/amd/mendocino: 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=None TEST=Don't see the "PCI: Leftover static devices:" warning for these in the boot console. BRANCH=skyrim
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: I517776e4dedc70e957a0c836ab3c2e5d49e156d2 --- M src/soc/amd/mendocino/chipset_mendocino.cb M src/soc/amd/mendocino/chipset_rembrandt.cb 2 files changed, 32 insertions(+), 10 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/74526/1
diff --git a/src/soc/amd/mendocino/chipset_mendocino.cb b/src/soc/amd/mendocino/chipset_mendocino.cb index b4efdf5..c1da13c 100644 --- a/src/soc/amd/mendocino/chipset_mendocino.cb +++ b/src/soc/amd/mendocino/chipset_mendocino.cb @@ -10,10 +10,10 @@ device pci 01.0 on end # Dummy Host Bridge
device pci 02.0 on end # Dummy Host Bridge, do not disable - device pci 02.1 alias gpp_bridge_0 off ops amd_external_pcie_gpp_ops end - device pci 02.2 alias gpp_bridge_1 off ops amd_external_pcie_gpp_ops end - device pci 02.3 alias gpp_bridge_2 off ops amd_external_pcie_gpp_ops end - device pci 02.4 alias gpp_bridge_3 off ops amd_external_pcie_gpp_ops end + device pci 02.1 alias gpp_bridge_0 hidden ops amd_external_pcie_gpp_ops end + device pci 02.2 alias gpp_bridge_1 hidden ops amd_external_pcie_gpp_ops end + device pci 02.3 alias gpp_bridge_2 hidden ops amd_external_pcie_gpp_ops end + device pci 02.4 alias gpp_bridge_3 hidden ops amd_external_pcie_gpp_ops end
device pci 08.0 on end # Dummy Host Bridge, do not disable device pci 08.1 alias gpp_bridge_a off # Internal GPP Bridge 0 to Bus A diff --git a/src/soc/amd/mendocino/chipset_rembrandt.cb b/src/soc/amd/mendocino/chipset_rembrandt.cb index 2ecb2405..fe97cf8 100644 --- a/src/soc/amd/mendocino/chipset_rembrandt.cb +++ b/src/soc/amd/mendocino/chipset_rembrandt.cb @@ -10,12 +10,12 @@ device pci 01.0 on end # Dummy Host Bridge
device pci 02.0 on end # Dummy Host Bridge, do not disable - device pci 02.1 alias gpp_bridge_0 off ops amd_external_pcie_gpp_ops end - device pci 02.2 alias gpp_bridge_1 off ops amd_external_pcie_gpp_ops end - device pci 02.3 alias gpp_bridge_2 off ops amd_external_pcie_gpp_ops end - device pci 02.4 alias gpp_bridge_3 off ops amd_external_pcie_gpp_ops end - device pci 02.5 alias gpp_bridge_4 off ops amd_external_pcie_gpp_ops end - device pci 02.6 alias gpp_bridge_5 off ops amd_external_pcie_gpp_ops end + device pci 02.1 alias gpp_bridge_0 hidden ops amd_external_pcie_gpp_ops end + device pci 02.2 alias gpp_bridge_1 hidden ops amd_external_pcie_gpp_ops end + device pci 02.3 alias gpp_bridge_2 hidden ops amd_external_pcie_gpp_ops end + device pci 02.4 alias gpp_bridge_3 hidden ops amd_external_pcie_gpp_ops end + device pci 02.5 alias gpp_bridge_4 hidden ops amd_external_pcie_gpp_ops end + device pci 02.6 alias gpp_bridge_5 hidden ops amd_external_pcie_gpp_ops end
device pci 08.0 on end # Dummy Host Bridge, do not disable device pci 08.1 alias gpp_bridge_a off # Internal GPP Bridge 0 to Bus A