[PATCH v3 08/11] backlight: lcd: Replace fb events with a dedicated function call

Thomas Zimmermann tzimmermann at suse.de
Fri Mar 21 08:16:18 UTC 2025


Hi

Am 18.03.25 um 10:40 schrieb Daniel Thompson:
> On Thu, Mar 06, 2025 at 03:05:50PM +0100, Thomas Zimmermann wrote:
>> Remove support for fb events from the lcd subsystem. Provide the
>> helper lcd_notify_blank_all() instead. In fbdev, call
>> lcd_notify_blank_all() to inform the lcd subsystem of changes
>> to a display's blank state.
>>
>> Fbdev maintains a list of all installed notifiers. Instead of fbdev
>> notifiers, maintain an internal list of lcd devices.
> I don't love the LCD devices list, however I can see the list of notifiers
> had the same semantic effect (only less explicit) so I can live with
> it ;-).
>
>> v3:
>> - export lcd_notify_mode_change_all() (kernel test robot)
>> v2:
>> - maintain global list of lcd devices
>> - avoid IS_REACHABLE() in source file
>> - use lock guards
>> - initialize lcd list and list mutex
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
> Happy with these changes, but have the same EXPORT_SYMBOL_GPL()
> questions I did with the backlight code.

As in backlight.c, all existing symbols are exported with 
EXPORT_SYMBOL(). I simply used what was already there.

Best regards
Thomas

>
>
> Daniel.
>

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)



More information about the dri-devel mailing list