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

[Angel Pons (Code Review)]

Angel Pons 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 3:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36455/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/36455/3//COMMIT_MSG@14 
PS3, Line 14: rather
rather than

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

https://review.coreboot.org/c/coreboot/+/36455/3/src/soc/intel/common/block/... 
PS3, Line 4:  * Copyright (C) 2007-2009 coresystems GmbH
> > IANAL but I don't think it is legit to remove these […]

I'd leave the previous copyright lines because AUTHORS is not complete yet. Also, given the rather high similarity index, I would say dropping copyright lines is not a good idea.

IANAL either, so I would play safe: how about keeping the previous copyright years, and updating the years for Intel Corporation?