1 comment:
File src/mainboard/google/hatch/variants/jinlon/overridetree.cb:
Patch Set #2, Line 86: ACPI_GPIO_INPUT_ACTIVE_LOW
My (limited) understanding is that this is an IRQ, would a GpioInterrupt be more appropriate?
I though that too. but then I had a nice conversation with Andy Shevchenko. If you want to read an irq pin, you have to declare it as GpioIo(), not as GpioInt().
https://lore.kernel.org/linux-gpio/CAHp75Vd7D7gArrgAWJ-e=u-KeGiQ5Ouj94YGfRmLPb62JOF-2w@mail.gmail.com/T/#t
https://lore.kernel.org/linux-gpio/CAHp75VdZiOnQdUirEM1BG27kV=htNX95Ar6eJ8LAK91MsOyYSg@mail.gmail.com/T/#u
Also there is no way (and kernel upstream do not want) to define a GpioInt() as EDGE and BOTH and ACTIVE_LOW().
To view, visit change 46962. To unsubscribe, or for help writing mail filters, visit settings.