From 792ebb6374f2b92e2c4b84f5b8d151a129ed81cc Mon Sep 17 00:00:00 2001 From: Pierre Gondois Date: Wed, 7 Apr 2021 15:43:39 +0100 Subject: [PATCH] DynamicTablesPkg: Fix generated _HID value for SBSA SSDT tables describing an SBSA compatible serial port receive an '_HID' value of 'ARMH0011'. This value represents a PL011 serial port. This patch: - Generates an 'ARMHB000' instead - References the 'ACPI for Arm Components 1.0 - 2020' document specifying the '_HID' values to use. Signed-off-by: Pierre Gondois Reviewed-by: Sami Mujawar --- .../Common/SsdtSerialPortFixupLib/SsdtSerialPortFixupLib.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/DynamicTablesPkg/Library/Common/SsdtSerialPortFixupLib/SsdtSerialPortFixupLib.c b/DynamicTablesPkg/Library/Common/SsdtSerialPortFixupLib/SsdtSerialPortFixupLib.c index 389c9eb914..a65c1fe7e3 100644 --- a/DynamicTablesPkg/Library/Common/SsdtSerialPortFixupLib/SsdtSerialPortFixupLib.c +++ b/DynamicTablesPkg/Library/Common/SsdtSerialPortFixupLib/SsdtSerialPortFixupLib.c @@ -8,6 +8,7 @@ @par Reference(s): - Arm Server Base Boot Requirements (SBBR), s4.2.1.8 "SPCR". - Microsoft Debug Port Table 2 (DBG2) Specification - December 10, 2015. + - ACPI for Arm Components 1.0 - 2020 **/ #include @@ -176,7 +177,7 @@ FixupIds ( case EFI_ACPI_DBG2_PORT_SUBTYPE_SERIAL_ARM_SBSA_GENERIC_UART: case EFI_ACPI_DBG2_PORT_SUBTYPE_SERIAL_ARM_SBSA_GENERIC_UART_2X: { - HidString = "ARMH0011"; + HidString = "ARMHB000"; CidString = ""; break; }