[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: add PEN for Boten

[Marco Chen (Code Review)]

Marco Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45807 )

Change subject: mb/google/dedede: add PEN for Boten
......................................................................

Patch Set 7:

(3 comments)

https://review.coreboot.org/c/coreboot/+/45807/7/src/mainboard/google/dedede... 
File src/mainboard/google/dedede/variants/boten/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/45807/7/src/mainboard/google/dedede... 
PS7, Line 12: |
nit: please align the "|"

https://review.coreboot.org/c/coreboot/+/45807/7/src/mainboard/google/dedede... 
PS7, Line 109: 		device pci 15.1 on
As I know, USI stylus and touchscreen controller are supported by I2C 2 and the pen detection here is not on I2C bus so why do we need to enable i2c 1 here?

https://review.coreboot.org/c/coreboot/+/45807/7/src/mainboard/google/dedede... 
PS7, Line 112: ACPI_GPIO_INPUT_ACTIVE_LOW
Wondering to know whether we should monitor both edge for pen inserted and ejected?