EXTERNAL: [PATCH 0/2] backlight: mp3309c: Drop pwm_apply_args()

FLAVIO SULIGOI f.suligoi at asem.it
Mon Jul 7 15:44:25 UTC 2025


Hi Uwe,

> -----Original Message-----
> From: Uwe Kleine-König <u.kleine-koenig at baylibre.com>
> Sent: martedì 1 luglio 2025 11:23
> To: FLAVIO SULIGOI <f.suligoi at asem.it>; Lee Jones <lee at kernel.org>;
> Daniel Thompson <danielt at kernel.org>; Jingoo Han <jingoohan1 at gmail.com>;
> Helge Deller <deller at gmx.de>
> Cc: Daniel Thompson <daniel.thompson at linaro.org>; dri-
> devel at lists.freedesktop.org; linux-fbdev at vger.kernel.org; linux-
> pwm at vger.kernel.org
> Subject: EXTERNAL: [PATCH 0/2] backlight: mp3309c: Drop pwm_apply_args()
> 
> [Use caution with links & attachments]
> 
> 
> 
> Hello,
> 
> the first patch of this series is what I really care about: There are
> hardly any drivers left that use pwm_apply_args(). When all of them are
> converted to not use it any more, I intend to drop that function.
> 
> The 2nd patch is just a change that I noticed while editing the driver
> that is IMHO nice. If you don't agree and only apply the first patch, I
> won't argue. It's an alternative approach to what Daniel Thompson did in
> commit 7ee6478d5aa9 ("backlight: mp3309c: Fully initialize
> backlight_properties during probe").

I've tested your patches on my board and all is ok.
Thanks!

> 
> Best regards
> Uwe
> 
> Uwe Kleine-König (2):
>   backlight: mp3309c: Drop pwm_apply_args()
>   backlight: mp3309c: Initialize backlight properties without memset
> 
>  drivers/video/backlight/mp3309c.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> 
> base-commit: 1343433ed38923a21425c602e92120a1f1db5f7a
> --
> 2.49.0


Flavio


More information about the dri-devel mailing list