Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43623 )
Change subject: mb/system76/lemp9: enable I2C HID touchpad ......................................................................
Patch Set 1: Code-Review+1
(3 comments)
https://review.coreboot.org/c/coreboot/+/43623/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/43623/1//COMMIT_MSG@9 PS1, Line 9: Tested on lemp9, touchpad confirmed to use I2C HID "confirmed to use I2C HID"? What do you mean?
Also add a trailing period.
https://review.coreboot.org/c/coreboot/+/43623/1/src/mainboard/system76/lemp... File src/mainboard/system76/lemp9/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/43623/1/src/mainboard/system76/lemp... PS1, Line 47: PchSerialIoPci What's here?
https://review.coreboot.org/c/coreboot/+/43623/1/src/mainboard/system76/lemp... PS1, Line 56: PchSerialIoPci And here? (looks like it was enabled on the devicetree before)