On Wed, Apr 05, 2017 at 03:46:18PM +0800, Heyi Guo wrote:
Hi folks,
I found some _CID values of devices on ARM platforms may not be correct:
OpenPlatformPkg/Platforms/ARM/VExpress/AcpiTables/Dsdt.asl: Name(_CID, "PL011") OpenPlatformPkg/Platforms/ARM/Juno/AcpiTables/Dsdt.asl: Name(_CID, "PL050_KBD") OpenPlatformPkg/Platforms/ARM/Juno/AcpiTables/Dsdt.asl: Name(_CID, "PL011")
According to ACPI spec, _CID should be something as _HID or PCI ID string.
There are also lots of similar mistakes in Hisilicon platform ASL files. We can start to fix them when the above issue is confirmed.
Thanks and regards,
I did notice those a while ago and when I double checked the spec I did convince myself they are not actually banned by the spec.
I shall have to recheck again as I can't remember the exact details of why I convinced myself they were OK.
Graeme