On Mon, Feb 5, 2024 at 1:36 PM Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > > On Mon, Feb 05, 2024 at 10:34:03AM +0100, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> > > > > With the list of GPIO devices now protected with SRCU we can use > > gpio_device_find() to traverse it from sysfs. > > ... > > > +static int gpiofind_sysfs_register(struct gpio_chip *gc, void *data) > > +{ > > + struct gpio_device *gdev = gc->gpiodev; > > + int ret; > > + > > + if (gdev->mockdev) > > + return 0; > > + > > + ret = gpiochip_sysfs_register(gdev); > > + if (ret) > > + chip_err(gc, "failed to register the sysfs entry: %d\n", ret); > > > + return 0; > > ??? > Not sure what the ... and ??? mean? The commit message should have read "... traverse it from gpiofind_sysfs_register()" I agree but the latter? Bart > > +} > > -- > With Best Regards, > Andy Shevchenko > >