[S] Change in coreboot[main]: mb/google/brya/var/xol: Change touchpad I2C interrupt type to GPIO_INT

[Nick Vaccaro (Code Review)]

Attention is currently required from: Dinesh Gehlot, Edward Doan, Eric Lai, Jamie Chen, Kapil Porwal, Nick Vaccaro, SH Kim, Subrata Banik, YH Lin.

Nick Vaccaro has posted comments on this change by SH Kim. ( https://review.coreboot.org/c/coreboot/+/83346?usp=email )

Change subject: mb/google/brya/var/xol: Change touchpad I2C interrupt type to GPIO_INT
......................................................................

Patch Set 5:

(2 comments)

File src/mainboard/google/brya/variants/xol/gpio.c:

https://review.coreboot.org/c/coreboot/+/83346/comment/ed7d3c37_7529ac5b?usp... :
PS1, Line 123: 	PAD_CFG_GPI_INT_LOCK(GPP_F14, NONE, LEVEL, LOCK_CONFIG),
> Acknowledged

I'm not sure where that clip came from, but the GPP_F14 gpio is not locked in mainboard/google/brya/variants/baseboard/brya/gpio.c

File src/mainboard/google/brya/variants/xol/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/83346/comment/a0f56820_1ddcdefc?usp... :
PS1, Line 347: 				register "generic.detect" = "1"
> Acknowledged

Won't this enable the interrupt? (IIRC, the touchpad is powered directly off of a power rail, so no independent way to disable interrupt for it.

If it enables the interrupt but there's no handler in coreboot for it / interrupt handler is in the kernel, couldn't that cause a problem if the touchpad interrupts before kernel is loaded?