Alex Levin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44948 )
Change subject: mb/google/volteer/variants/volteer: route GPP_F14 via APIC ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44948/1/src/mainboard/google/voltee... File src/mainboard/google/volteer/variants/volteer/gpio.c:
https://review.coreboot.org/c/coreboot/+/44948/1/src/mainboard/google/voltee... PS1, Line 128: PAD_CFG_GPI_APIC(GPP_F14, NONE, PLTRST, LEVEL, INVERT),
Why the inversion? What code is referencing this IRQ?
sx9310 is referencing this. I thought that inversion would we the way to mark this as being active low. from your question I realize I might be wrong 😊