Re: [PATCH] Platform/ARM/Juno: Remove bogus PL011 _CID from DSDT ACPI table

["Sami Mujawar" <sami.mujawar@arm.com>]

Hi Sunny,

Thank you for this patch.

I will merge this change shortly with the typo fixed.

Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>

Regards,

Sami Mujawar

On 22/03/2021, 08:07, "Ard Biesheuvel" <ardb@kernel.org> wrote:

    On Mon, 22 Mar 2021 at 07:13, Sunny Wang <Sunny.Wang@arm.com> wrote:
    >
    > PL011 is not a valid ACPI identifier so don't expose it as a _CID.
    > Since _CID (Comptable ID) is optional, let's just drop it.
    >

    Compatible

    > Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>
    > Cc: Sami Mujawar <sami.mujawar@arm.com>
    > Cc: Pierre Gondois <Pierre.Gondois@arm.com>
    > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
    > Signed-off-by: Sunny Wang <sunny.wang@arm.com>

    Acked-by: Ard Biesheuvel <ardb@kernel.org>

    > ---
    >  Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl | 1 -
    >  1 file changed, 1 deletion(-)
    >
    > diff --git a/Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl b/Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl
    > index c17c74e174..61353213fe 100644
    > --- a/Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl
    > +++ b/Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl
    > @@ -256,7 +256,6 @@ DefinitionBlock("DsdtTable.aml", "DSDT", 1, "ARMLTD", "ARM-JUNO", EFI_ACPI_ARM_O
    >      // UART PL011
    >      Device(COM0) {
    >        Name(_HID, "ARMH0011")
    > -      Name(_CID, "PL011")
    >        Name(_UID, Zero)
    >        Name(_CRS, ResourceTemplate() {
    >          Memory32Fixed(ReadWrite, 0x7FF80000, 0x1000)
    > --
    > 2.31.0.windows.1
    >

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.