Re: [PATCH] backlight: led_bl: Add support for an "enable" GPIO

Le Tue, Nov 02, 2021 at 11:25:14AM +0000, Daniel Thompson a écrit :

On Tue, Nov 02, 2021 at 11:19:42AM +0000, Daniel Thompson wrote:

quoted

On Tue, Nov 02, 2021 at 10:04:55AM +0000, Corentin LABBE wrote:

quoted

From: Jean-Jacques Hiblot <redacted>

This patch adds support for an "enable GPIO".

Before taking this kind of change is there a good reason why backlight
should manage the GPIO? I thought that the LED subsystem was a sub
system for LEDs (not LED controllers). In other words if you direct
that the LED be lit up then isn't it the LED driver's job to manage
the GPIO and ensure that it lights up?

Sorry... I should have paid more attention to my sense of déjà vu with
this patch.

This approach was discussed and rejected when we first introduced the
led_bl driver:
https://lore.kernel.org/linux-leds/20190705100851.zn2jkipj4fxq5we6@devuan/ (local)

Hello

I am sorry, I didnt checked if the patch was already submitted or not.

Regards