Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45590 )
Change subject: soc/intel/common/block/lpc: add acpi name ......................................................................
Patch Set 4:
(3 comments)
https://review.coreboot.org/c/coreboot/+/45590/4/src/soc/intel/icelake/chip.... File src/soc/intel/icelake/chip.c:
https://review.coreboot.org/c/coreboot/+/45590/4/src/soc/intel/icelake/chip.... PS4, Line 78: /* Keeping ACPI device name coherent with ec.asl */
This needs to be dropped as well.
Done
https://review.coreboot.org/c/coreboot/+/45590/4/src/soc/intel/jasperlake/ch... File src/soc/intel/jasperlake/chip.c:
https://review.coreboot.org/c/coreboot/+/45590/4/src/soc/intel/jasperlake/ch... PS4, Line 98: /* Keeping ACPI device name coherent with ec.asl */
This needs to be dropped as well.
Done
https://review.coreboot.org/c/coreboot/+/45590/4/src/soc/intel/tigerlake/chi... File src/soc/intel/tigerlake/chip.c:
https://review.coreboot.org/c/coreboot/+/45590/4/src/soc/intel/tigerlake/chi... PS4, Line 107: /* Keeping ACPI device name coherent with ec.asl */
Same here.
Done