summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTarun Tuli <taruntuli@google.com>2022-05-03 20:34:32 +0000
committerFelix Held <felix-coreboot@felixheld.de>2022-05-06 13:38:31 +0000
commitd8d522884bfbc2b5239a9ece77ab074c846167bd (patch)
tree4ea89fcf9a5b78b03bef843f4e0b1377b9fbf1f9
parent8e14df3b6f668e4b333d6eed6ac5c848b268db1a (diff)
soc/intel/alderlake: Add missing ACPI device path names
A few ACPI device path name handlers are missing. Add handling to ensure that these names are returned during acpi_device_path() calls. TEST=Built and tested on brya Signed-off-by: Tarun Tuli <taruntuli@google.com> Change-Id: I37d6dd5df921c931af72dd469c3f4067c61b0df3 Reviewed-on: https://review.coreboot.org/c/coreboot/+/63984 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Subrata Banik <subratabanik@google.com>
-rw-r--r--src/soc/intel/alderlake/chip.c4
-rw-r--r--src/soc/intel/common/block/uart/uart.c4
2 files changed, 8 insertions, 0 deletions
diff --git a/src/soc/intel/alderlake/chip.c b/src/soc/intel/alderlake/chip.c
index 699b39d419..59f006634a 100644
--- a/src/soc/intel/alderlake/chip.c
+++ b/src/soc/intel/alderlake/chip.c
@@ -75,6 +75,7 @@ const char *soc_acpi_name(const struct device *dev)
case SA_DEVFN_TBT2: return "TRP2";
case SA_DEVFN_TBT3: return "TRP3";
case SA_DEVFN_IPU: return "IPU0";
+ case SA_DEVFN_DPTF: return "DPTF";
case PCH_DEVFN_ISH: return "ISHB";
case PCH_DEVFN_XHCI: return "XHCI";
case PCH_DEVFN_I2C0: return "I2C0";
@@ -111,6 +112,9 @@ const char *soc_acpi_name(const struct device *dev)
case PCH_DEVFN_HDA: return "HDAS";
case PCH_DEVFN_SMBUS: return "SBUS";
case PCH_DEVFN_GBE: return "GLAN";
+ case PCH_DEVFN_SRAM: return "SRAM";
+ case PCH_DEVFN_SPI: return "FSPI";
+ case PCH_DEVFN_CSE: return "HEC1";
#if CONFIG(SOC_INTEL_ALDERLAKE_PCH_N)
case PCH_DEVFN_EMMC: return "EMMC";
#endif
diff --git a/src/soc/intel/common/block/uart/uart.c b/src/soc/intel/common/block/uart/uart.c
index 12f28829f6..3e2e6975c4 100644
--- a/src/soc/intel/common/block/uart/uart.c
+++ b/src/soc/intel/common/block/uart/uart.c
@@ -308,24 +308,28 @@ static const char *uart_acpi_hid(const struct device *dev)
static const char *uart_acpi_name(const struct device *dev)
{
switch (dev->device) {
+ case PCI_DID_INTEL_ADP_P_UART0:
case PCI_DID_INTEL_APL_UART0:
case PCI_DID_INTEL_GLK_UART0:
case PCI_DID_INTEL_SPT_UART0:
case PCI_DID_INTEL_SPT_H_UART0:
case PCI_DID_INTEL_CNP_H_UART0:
return "UAR0";
+ case PCI_DID_INTEL_ADP_P_UART1:
case PCI_DID_INTEL_APL_UART1:
case PCI_DID_INTEL_GLK_UART1:
case PCI_DID_INTEL_SPT_UART1:
case PCI_DID_INTEL_SPT_H_UART1:
case PCI_DID_INTEL_CNP_H_UART1:
return "UAR1";
+ case PCI_DID_INTEL_ADP_P_UART2:
case PCI_DID_INTEL_APL_UART2:
case PCI_DID_INTEL_GLK_UART2:
case PCI_DID_INTEL_SPT_UART2:
case PCI_DID_INTEL_SPT_H_UART2:
case PCI_DID_INTEL_CNP_H_UART2:
return "UAR2";
+ case PCI_DID_INTEL_ADP_P_UART3:
case PCI_DID_INTEL_GLK_UART3:
return "UAR3";
default: