On Mon, May 18, 2020 at 12:16 PM Sam Ravnborg <[email protected]> wrote: > Hi Linus. > > On Mon, May 18, 2020 at 10:10:12AM +0200, Linus Walleij wrote: > > On Sun, May 17, 2020 at 9:01 PM Sam Ravnborg <[email protected]> wrote: > > > > > Look up backlight device using devm_of_find_backlight(). > > > This simplifies the code and prevents us from hardcoding > > > the node name in the driver. > > > > > > v2: > > > - Added Cc: Peter Ujfalusi > > > > > > Signed-off-by: Sam Ravnborg <[email protected]> > > > Cc: Russell King <[email protected]> > > > Cc: Sam Ravnborg <[email protected]> > > > Cc: Bartlomiej Zolnierkiewicz <[email protected]> > > > Cc: Peter Ujfalusi <[email protected]> > > > Cc: Daniel Vetter <[email protected]> > > > Cc: Linus Walleij <[email protected]> > > > Cc: Jani Nikula <[email protected]> > > > Cc: Douglas Anderson <[email protected]> > > > > Acked-by: Linus Walleij <[email protected]> > Thanks. I went ahead and applied this now, so we could kill > the last user of of_find_backlight_by_node(). > > I hope we can make of_find_backlight_by_node() static after the merge window > so no new users appears.
For this driver (drivers/video/fbdev/amba-clcd.c) there are zero users after the merge window (all users moved over to DRM) so I plan to retire it completely. Yours, Linus Walleij _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
