Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/61089 )
Change subject: soc/amd/sabrina/include/data_fabric: update IOMS0_FABRIC_ID ......................................................................
soc/amd/sabrina/include/data_fabric: update IOMS0_FABRIC_ID
The data fabric ID table in PPR #57243 Rev 1.50 has a different IOMS0 fabric ID than Cezanne.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I32890b5c03219f6ebf8180929d71ef726d382483 Reviewed-on: https://review.coreboot.org/c/coreboot/+/61089 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Marshall Dawson marshalldawson3rd@gmail.com Reviewed-by: Fred Reitberger reitbergerfred@gmail.com --- M src/soc/amd/sabrina/include/soc/data_fabric.h 1 file changed, 1 insertion(+), 3 deletions(-)
Approvals: build bot (Jenkins): Verified Marshall Dawson: Looks good to me, approved Fred Reitberger: Looks good to me, but someone else must approve
diff --git a/src/soc/amd/sabrina/include/soc/data_fabric.h b/src/soc/amd/sabrina/include/soc/data_fabric.h index 87db4d3e..fbae1c6 100644 --- a/src/soc/amd/sabrina/include/soc/data_fabric.h +++ b/src/soc/amd/sabrina/include/soc/data_fabric.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0-only */
-/* TODO: Check if this is still correct */ - #ifndef AMD_SABRINA_DATA_FABRIC_H #define AMD_SABRINA_DATA_FABRIC_H
@@ -10,7 +8,7 @@ /* SoC-specific bits in D18F0_MMIO_CTRL0 */ #define DF_MMIO_NP BIT(16)
-#define IOMS0_FABRIC_ID 10 +#define IOMS0_FABRIC_ID 9
#define NUM_NB_MMIO_REGS 8