On Fri, Sep 15, 2023 at 4:15 PM Rob Herring <robh@xxxxxxxxxx> wrote: > > + /* > > + * The generic property "trigger-sources" is followed, > > + * and we hope that this is a GPIO. > > + */ > > + gpio_data->gpiod = fwnode_gpiod_get_index(dev->fwnode, > > + "trigger-sources", > > + 0, GPIOD_IN, > > + "led-trigger"); > > Isn't this going to look for "trigger-sources-gpio" and > ""trigger-sources-gpios"? Indeed. I'll simply code an exception for this into gpiolib-of.c, it's an OF pecularity after all. Yours, Linus Walleij