[XS] Change in coreboot[main]: soc/intel/cannonlake: Drop entries from soc_acpi_name()

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79013?usp=email )

Change subject: soc/intel/cannonlake: Drop entries from soc_acpi_name()
......................................................................

soc/intel/cannonlake: Drop entries from soc_acpi_name()

The THRM and SATA PCI devices do not currently have any ACPI devices
defined, so drop them from soc_acpi_name() so they do not end up in
the LPI constraint list. This eliminates the following errors
under Linux:

AE_NOT_FOUND: _SB_.PCI0.THRM
AE_NOT_FOUND: _SB_.PCI0.SATA

TEST= build/boot google/hatch (jinlon) and verify no ACPI errors.

Change-Id: I3827b152644e2eaecc1ad288d441d2dad4d76ccb
Signed-off-by: Matt DeVillier 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79013
Reviewed-by: Felix Singer 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/cannonlake/chip.c
1 file changed, 0 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Felix Singer: Looks good to me, approved

diff --git a/src/soc/intel/cannonlake/chip.c b/src/soc/intel/cannonlake/chip.c
index 2dc39b6..a1ba17e 100644
--- a/src/soc/intel/cannonlake/chip.c
+++ b/src/soc/intel/cannonlake/chip.c
@@ -85,7 +85,6 @@
    case SA_DEVFN_GNA:	return "GNA";
    case PCH_DEVFN_XHCI:	return "XHCI";
    case PCH_DEVFN_USBOTG:	return "XDCI";
-	case PCH_DEVFN_THERMAL:	return "THRM";
    case PCH_DEVFN_I2C0:	return "I2C0";
    case PCH_DEVFN_I2C1:	return "I2C1";
    case PCH_DEVFN_I2C2:	return "I2C2";
@@ -95,7 +94,6 @@
    case PCH_DEVFN_CSE_IDER:	return "CSED";
    case PCH_DEVFN_CSE_KT:	return "CSKT";
    case PCH_DEVFN_CSE_3:	return "CSE3";
-	case PCH_DEVFN_SATA:	return "SATA";
    case PCH_DEVFN_UART2:	return "UAR2";
    case PCH_DEVFN_I2C4:	return "I2C4";
    case PCH_DEVFN_I2C5:	return "I2C5";