On Thu, Nov 18, 2021 at 3:22 PM Rafał Miłecki <zajec5@xxxxxxxxx> 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
