[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