[PATCH 3/5] pinctrl: add helpers reading pins, groups & functions from DT

Linus Walleij linus.walleij at linaro.org
Sun Nov 21 15:53:15 PST 2021


On Thu, Nov 18, 2021 at 2:58 PM Andy Shevchenko
<andy.shevchenko at gmail.com> wrote:
> On Thu, Nov 18, 2021 at 3:22 PM Rafał Miłecki <zajec5 at gmail.com> wrote:

> > +#include <linux/of.h>
>
> I don't like this. This shows not thought through the design of the series.
>
> What I rather expect is a proper interfacing layer that you fill with
> options that can be provided by corresponding underlying
> implementation, e.g. DT.

I agree, the DT parts should land in pinctrl/devicetree.c in the
end with an opt-in for ACPI & board files. I don't know if we
can use software nodes fully instead of board file-specific
helpers at this point, it seems to have
grown pretty competent.

Yours,
Linus Walleij



More information about the linux-arm-kernel mailing list