Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32142 )
Change subject: ec/google/chromeec: Remove redundant use of ACPI offset operator ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/32142/1/src/ec/google/chromeec/acpi/ec.asl File src/ec/google/chromeec/acpi/ec.asl:
https://review.coreboot.org/#/c/32142/1/src/ec/google/chromeec/acpi/ec.asl@a... PS1, Line 57: I don't think this is redundant.
https://review.coreboot.org/#/c/32142/1/src/ec/google/chromeec/acpi/emem.asl File src/ec/google/chromeec/acpi/emem.asl:
https://review.coreboot.org/#/c/32142/1/src/ec/google/chromeec/acpi/emem.asl... PS1, Line 61: Offset (0x80), This seems redundant?