Re: [PATCH 15/16] backlight: remove pandora_bl

[Andreas Kemnade <andreas@kemnade.info>]

On Mon, 10 Aug 2026 20:30:13 +0200
"H. Nikolaus Schaller" <hns@goldelico.com> wrote:

> > 
> > Yes that is what I mean. It needs to be ported somewhere. If it can be ported to
> > some compatible = "pwm_bl_for_tps61161" or what fits best it will be fine. Or it
> > could be in some compatible = "pandora_bl" driver.
> > 
> > Has to be worked out when the development machines are not blocked by 3 or 4 other
> > projects already running in parallel :)  
> 
> Nevertheless I did a quick look into the pandora_bl driver structure.
> 
> Here are my findings:
> - the code fragment is part of pandora_backlight_update_status()
> - priv->old_state remembers if brighness was set to 0 or not
> - so this code triggers on any change from bright = 0 to bright > 0
> - the pandora_bl driver is the only one to use TWL_MODULE_PWM besides twl-pwm
> 
> So the most natural way of transplanting this appears to me:
> - add this logic to the twl-pwm driver

That chip could be connected to any pwm driver, so
it does not make sense to have that in a pwm driver.

So it belongs into a backlight driver. And since it shared so much
code with the pwm_bl, it could be extended.
The compatible should have pwm in the name to be able to also add
a w1 driver later.

Regards,
Andreas