[coreboot-gerrit] Change in coreboot[master]: soc/intel/{apl,cnl,dnv,icl,skl}: Move lpc.asl into common/block/acpi

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36455 )

Change subject: soc/intel/{apl,cnl,dnv,icl,skl}: Move lpc.asl into common/block/acpi
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36455/6/src/soc/intel/common/block/... 
File src/soc/intel/common/block/acpi/acpi/lpc.asl:

https://review.coreboot.org/c/coreboot/+/36455/6/src/soc/intel/common/block/... 
PS6, Line 20: 
Curious about some of the legacy devices here:

1. There is a forbidden list that Linux kernel maintains https://github.com/torvalds/linux/blob/master/drivers/acpi/acpi_platform.c#L.... Does it mean that these should not be exposed by the BIOS?

2. There are some HIDs that are not enumerated at all by the kernel. Are these added because they are used by other OSes?