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

Andy Shevchenko andy.shevchenko at gmail.com
Thu Nov 18 05:57:41 PST 2021


On Thu, Nov 18, 2021 at 3:22 PM Rafał Miłecki <zajec5 at gmail.com> wrote:

...

> --- a/drivers/pinctrl/pinmux.c
> +++ b/drivers/pinctrl/pinmux.c

> +#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.

Moreover, before doing this you probably would need to refactor the
pin control core to get rid of DT specifics, i.e. abstract them away
first.

-- 
With Best Regards,
Andy Shevchenko



More information about the linux-arm-kernel mailing list