On Wed, May 14, 2025 at 6:00 PM Mika Westerberg
<mika.westerberg@xxxxxxxxxxxxxxx> wrote:
> > drivers/gpio/Makefile | 1 +
> > .../{gpiolib-acpi.c => gpiolib-acpi-core.c} | 344 +----------------
> > drivers/gpio/gpiolib-acpi-quirks.c | 363 ++++++++++++++++++
> > drivers/gpio/gpiolib-acpi.h | 15 +
>
> All this -foo-core things look redundant to me. Why not just split it out
> and call it gpiolib-quirks.c and put there all the quirks not just ACPI? I
> Don't think we want to have gpiolib-of-quirks.c and gpiolog-swnode-quirks.c
> and so on.
For OF/device tree the quirks are in gpiolib-of.c and we probably do
not want to put these into a shared file with ACPI (and swnode?)
quirks as systems with OF compile objects (Makefile entries)
and ACPI compile objects are not always included in the same build,
so having them per-hw-config-principle cuts down compiletime
overhead. Also it's pretty clear separation of concerns I think.
Yours,
Linus Walleij
