On Wed, Feb 04, 2026 at 04:33:50AM -0600, Bartosz Golaszewski wrote: > On Tue, 3 Feb 2026 07:10:50 +0100, Tzung-Bi Shih <[email protected]> said: > > diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c > > index a6dd07be126c..3137e6f1108a 100644 > > --- a/drivers/gpio/gpiolib.c > > +++ b/drivers/gpio/gpiolib.c > > @@ -1281,7 +1281,7 @@ void gpiochip_remove(struct gpio_chip *gc) > > struct gpio_device *gdev = gc->gpiodev; > > > > /* FIXME: should the legacy sysfs handling be moved to gpio_device? */ > > - gpiochip_sysfs_unregister(gdev); > > + gpiochip_sysfs_unregister(gdev, gc); > > I understand the intention here but I really don't like passing both gc and > gdev here. We can get the address of the gpio_device from gpio_chip so why not > do this and pass only variable?
Ack, will fix in the next version.
