Re: [PATCH v1 1/1] pinctrl: st: Avoid using of_node member of struct gpio_chip

From: Andy Shevchenko
Date: Mon Oct 10 2022 - 02:49:18 EST


On Sun, Oct 09, 2022 at 11:13:44PM -0700, Dmitry Torokhov wrote:
> On Tue, Oct 04, 2022 at 03:54:49PM +0300, Andy Shevchenko wrote:
> > The of_node member of the struct gpio_chip is obsoleted and
> > shouldn't be used. It will be removed in the future.
> >
> > Replace its use in st_pctl_dt_calculate_pin() by comparing
> > the fwnode pointers.
> >
> > Fixes: e75729b2f63f ("pinctrl: st: stop abusing of_get_named_gpio()")

...

> Are you sure chip's fwnode is properly set here (without your other
> patch)? I don't believe the driver sets it up, and I don't think gpiolib
> maps it from the gpio_chip->of_node.
>
> Actually, there is another reference to of_node in gpio chip in
> st_gpiolib_register_bank()...

You are right, this patch has been made on top of another not yet published
one. Hence, nothing to fix it right now.

--
With Best Regards,
Andy Shevchenko