Lean Sheng Tan has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84795?usp=email )
(
2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: soc/intel/xeon_sp: Fix iiostack.asl ......................................................................
soc/intel/xeon_sp: Fix iiostack.asl
Align DSDT names with SSDT naming scheme, as provided by iio_domain_set_acpi_name() and hide unused devices by implementing the _STA method as done on newer platforms.
Change-Id: I8488907f28a78a6f71046dba54ba9cbd4b0652eb Signed-off-by: Patrick Rudolph patrick.rudolph@9elements.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/84795 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Shuo Liu shuo.liu@intel.com --- M src/soc/intel/xeon_sp/acpi/gen1/iiostack.asl 1 file changed, 16 insertions(+), 4 deletions(-)
Approvals: Shuo Liu: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/soc/intel/xeon_sp/acpi/gen1/iiostack.asl b/src/soc/intel/xeon_sp/acpi/gen1/iiostack.asl index 0dd39a0..53e144a 100644 --- a/src/soc/intel/xeon_sp/acpi/gen1/iiostack.asl +++ b/src/soc/intel/xeon_sp/acpi/gen1/iiostack.asl @@ -6,6 +6,17 @@ Name (_HID, EisaId ("PNP0A08") /* PCI Express Bus */) \ Name (_CID, EisaId ("PNP0A03") /* PCI Bus */) \ Name (_UID, 0x##id) \ + Method (_STA, 0, NotSerialized) \ + { \ + If (CondRefOf (_CRS)) \ + { \ + Return (0xf) \ + } \ + Else \ + { \ + Return (0) \ + } \ + } \ Method (_PRT, 0, NotSerialized) \ { \ If (PICM) \ @@ -60,14 +71,15 @@ } \ }
+// Keep in sync with iio_domain_set_acpi_name()! MAKE_IIO_DEV(00, 00) MAKE_IIO_DEV(01, 10) MAKE_IIO_DEV(02, 20) MAKE_IIO_DEV(03, 28)
#if (CONFIG_MAX_SOCKET > 1) -MAKE_IIO_DEV(06, 40) -MAKE_IIO_DEV(07, 50) -MAKE_IIO_DEV(08, 60) -MAKE_IIO_DEV(09, 68) +MAKE_IIO_DEV(20, 40) +MAKE_IIO_DEV(21, 50) +MAKE_IIO_DEV(22, 60) +MAKE_IIO_DEV(23, 68) #endif