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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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





[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux