Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41639 )
Change subject: ec/google/chromeec/acpi/superio: Add PS/2 Mouse ACPI entry ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41639/1/src/ec/google/chromeec/acpi... File src/ec/google/chromeec/acpi/superio.asl:
https://review.coreboot.org/c/coreboot/+/41639/1/src/ec/google/chromeec/acpi... PS1, Line 176: IO (Decode16, 0x60, 0x60, 0x01, 0x01) : IO (Decode16, 0x64, 0x64, 0x01, 0x01)
not sure if those should be added here again, since PS2K already uses those; ACPI might get unhappy […]
The driver actually only uses either the PS2K or PS2M IO resources: https://source.chromium.org/chromiumos/chromiumos/codesearch/+/master:src/th...
So they have to be identical.
https://review.coreboot.org/c/coreboot/+/41639/1/src/ec/google/chromeec/acpi... PS1, Line 184: }
is it intended to not have a _PRS method here? the io resources are identical to the ones from PS2K, […]
It was intentional. The linux kernel driver doesn't look at _PRS for resources, it only evaluates _CRS.