soc/amd/phoenix/chipset.cb: add missing GPP bridges on device 1
commitbee5c6084cd7555beefd933e44f6c2b67365f563
authorFelix Held <felix-coreboot@felixheld.de>
Fri, 3 Feb 2023 00:41:16 +0000 (3 01:41 +0100)
committerFelix Held <felix-coreboot@felixheld.de>
Sat, 4 Feb 2023 19:11:53 +0000 (4 19:11 +0000)
tree0a45bb9d2fb0fdabac20bf9b49d7396f58ba9621
parent8e1bb93fb88bc9cc20aab33a1fe09fb4c0c652a0
soc/amd/phoenix/chipset.cb: add missing GPP bridges on device 1

Only the PCIe ports on the functions of device 2 were present in the
devicetree and had the amd_external_pcie_gpp_ops ops assigned. Add the
missing PCIe ports on the functions of device 1 and assign the
amd_external_pcie_gpp_ops ops to them.

This SoC uses a slightly different naming scheme for its PCIe GPP ports.
Previously the PCIe GPP bridge number from the PCI Device ID Assignments
table from the PPR was used. Those bridge numbers are one less than the
function numbers of the device. This is due to function 0 being a dummy
bridge to avoid having to shuffle around the function numbers when the
first bridge is unused, since the PCIe specification mandates the
function 0 to be implemented if any other function on the same device is
implemented. In order for the device aliases to be consistent with the
PCIe device and function numbers which is way more commonly used and
also what lspci shows and what goes into the DXIO descriptors, change
the naming scheme of the aliases.

This was checked with PPR #57019 Rev 1.65 and PPR #57396 Rev 1.54.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Ib5c62c1df585877d9b6986a462a3636d4f2eb4c7
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72736
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
src/soc/amd/phoenix/chipset.cb