On Sat, Sep 8, 2018 at 10:38 PM Jacek Anaszewski <jacek.anaszewski@xxxxxxxxx> wrote: > > + /* skip leds that aren't available */ > > + if (!gpio_is_valid(template->gpio)) { > > + return ERR_PTR(-ENOENT); > > + } > > Curly braces are not needed for single statement block. > > I can fix it up by myself if you agree. Sure thing, fix it as you like it and apply, thanks! :) Yours, Linus Walleij