[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/lpc: Include correct acpi.h

[Marc Jones (Code Review)]

Marc Jones has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48251 )

Change subject: soc/intel/common/block/lpc: Include correct acpi.h
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/48251/3/src/soc/intel/common/block/... 
File src/soc/intel/common/block/lpc/lpc.c:

https://review.coreboot.org/c/coreboot/+/48251/3/src/soc/intel/common/block/... 
PS3, Line 14: <soc/acpi.h>
> I know it's not the best, but would it make sense to drop the redundant declarations in skylake SoC  […]

I don't think that using common block defines if the common block isn't use is correct. There are other dependencies across the common block that are similar to this and this change seems fairly minor.