Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38978 )
Change subject: [WIP] mb/acer: Add Acer Aspire ES1-572 ......................................................................
Patch Set 9:
(2 comments)
https://review.coreboot.org/c/coreboot/+/38978/7//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38978/7//COMMIT_MSG@37 PS7, Line 37: - Touchpad. Hangs off either EC or some I2C controller and needs ACPI.
I doubt it's wired using PS/2 to the EC
Well, it's connected to the EC's PS/2 port 3 (according to the kb9012 ds). It could be I2C as well, depending on the actual ec. Anyway, direct I2C/SMBus is better than <whatever> via PS/2 through ec.
https://review.coreboot.org/c/coreboot/+/38978/5/src/mainboard/acer/es1-572/... File src/mainboard/acer/es1-572/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/38978/5/src/mainboard/acer/es1-572/... PS5, Line 223: register "SendVrMbxCmd" = "2"
I haven't seen any information on it either. […]
I don't think, there is any reason to show it as advanced setting in vendor fw, if that is for fixing an erratum. wdym with "goes gold"?