Patch Set 1:
I do understand the requirement to report the mapped
pin number. What I don't understand is, when would we
ever need to have the raw number? i.e. why can't we
make the mapped numbers the default?
That was my original plan, use this if CONFIG_GENERIC_GPIO_LIB was defined but that ended up breaking a bunch of random boards that don't have a full GPIO implementation.
2 comments:
File src/vendorcode/google/chromeos/acpi.c:
Patch Set #1, Line 18: #if IS_ENABLED(CONFIG_CHROMEOS_ACPI_GPIO_GENERATE_PIN)
Would it hurt to always include the header file?
gpio.h is poor in this regard, if the SOC doesn't have an implementation it will fail to compile.
Patch Set #1, Line 34: #if IS_ENABLED(CONFIG_CHROMEOS_ACPI_GPIO_GENERATE_PIN)
Any reason to use pre-processor code instead of C?
Same reason as above, if the header is now guarded the code has to be as well.
To view, visit change 31080. To unsubscribe, or for help writing mail filters, visit settings.