Hi Dan and Pavel, On 11/15/2017 11:23 PM, Pavel Machek wrote: > Hi! > >>> Shouldn't the driver be targeted for backlight subsystem then? >>> >>> Adding backlight maintainers. >>> >> >> Great point! I was not aware of the backlight subsystem. >> >> Looks like I need to create a back light version as well. >> >> Like the lp8788 did since this can be used as a LED driver beyond >> display back lighting. > > No, definitely not two drivers for lp8788 hardware. I agree. > If that does not yet exist... you want to create glue layer to be able > to use LED as a display backlight. (It may already exist, no idea). > > ... > > Actually or maybe a LED trigger. Just set LED's trigger to "this is > display backlight". There is one already: drivers/leds/trigger/ledtrig-backlight.c It adds a LED class device to the fb_notifier_list (drivers/video/fbdev/core/fb_notify.c) using fb_register_client(). The same is used in backlight_device_register (drivers/video/backlight/backlight.c). Actually why do you want to have this driver in the LED subsystem, if it is advertised as "designed for LCD display backlighting"? As a side note I can say that I've been always wondering why the two subsystems for similar type of hardware. -- Best regards, Jacek Anaszewski -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
