Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46268 )
Change subject: soc/intel/common/block/acpi: Factor out common buttons.asl ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46268/1/src/mainboard/intel/icelake... File src/mainboard/intel/icelake_rvp/acpi/mainboard.asl:
https://review.coreboot.org/c/coreboot/+/46268/1/src/mainboard/intel/icelake... PS1, Line 6: Device (PWRB)
This is not required. Please see the commit message here: https://review.coreboot. […]
Then good to clear up and remove all instance of PWRB from intel/mainboard @Furquan?