3 comments:
File src/mainboard/google/hatch/variants/jinlon/overridetree.cb:
Patch Set #2, Line 86: ACPI_GPIO_INPUT_ACTIVE_LOW
I tried setting PAD_RX_POL(INVERT) on an EDGE_BOTH and it had no effect (check the attached patch). […]
Thanks for the pointers Ricardo. It is definitely an interesting case. As for why the above diff did not work, I think the kernel is reconfiguring the pad based on the information passed in ACPI. That is probably why it doesn't work. If interested, you can dump the configuration registers in coreboot and kernel to see if they change later on. Anyways, we can go ahead with the change you have in the latest patchset. Thanks for trying this out! :)
File src/mainboard/google/hatch/variants/jinlon/overridetree.cb:
Can you please use tabs instead of spaces here and for the lines below like the rest of the entries in this file?
Patch Set #4, Line 87: ACPI_GPIO_INPUT_ACTIVE_LOW
Can you please add a comment here explaining why this is being configured as GPIO instead of GPIO_IRQ? It would be good to have the context here so that it isn't accidentally changed later on and also helps if anyone decides to copy-paste.
To view, visit change 46962. To unsubscribe, or for help writing mail filters, visit settings.