mbox series

[0/2] backlight: pwm_bl: prevent backlight flicker when switching PWM on

Message ID 1509022186-19636-1-git-send-email-LW@KARO-electronics.de
Headers show
Series backlight: pwm_bl: prevent backlight flicker when switching PWM on | expand

Message

Lothar Waßmann Oct. 26, 2017, 12:49 p.m. UTC
These patches implement some measures to prevent backlight flicker
when the backlight is being switched on for a display with an active
low brightness control pin.
--
To unsubscribe from this list: send the line "unsubscribe linux-pwm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Daniel Thompson Nov. 8, 2017, 10:48 a.m. UTC | #1
On 26/10/17 13:49, Lothar Waßmann wrote:
> These patches implement some measures to prevent backlight flicker
> when the backlight is being switched on for a display with an active
> low brightness control pin.
> GIT: [PATCH 1/2] backlight: pwm_bl: Enable PWM before switching regulator
> GIT: [PATCH 2/2] backlight: pwm_bl: add configurable delay between

Other than hoisting the pwm_enable() even earlier in the setup sequence 
this patchset seems to have a significant overlap with Enric's recent 
posting:

   https://lkml.org/lkml/2017/7/21/211

Any chance of a shared view on this, especially on the DT bindings?



Daniel.
--
To unsubscribe from this list: send the line "unsubscribe linux-pwm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Enric Balletbo i Serra Nov. 10, 2017, 11:22 a.m. UTC | #2
Hi all,

On 08/11/17 11:48, Daniel Thompson wrote:
> On 26/10/17 13:49, Lothar Waßmann wrote:
>> These patches implement some measures to prevent backlight flicker
>> when the backlight is being switched on for a display with an active
>> low brightness control pin.
>> GIT: [PATCH 1/2] backlight: pwm_bl: Enable PWM before switching regulator
>> GIT: [PATCH 2/2] backlight: pwm_bl: add configurable delay between
> 
> Other than hoisting the pwm_enable() even earlier in the setup sequence this
> patchset seems to have a significant overlap with Enric's recent posting:
> 
>   https://lkml.org/lkml/2017/7/21/211
> 
> Any chance of a shared view on this, especially on the DT bindings?
> 

The DT binding were discussed some time ago for the patch series I sent, though
there isn't a final ack from DT maintainer.

Lothar, the series I sent have been reviewed and acked, can you test if the
series fixes the problem for you too?

Thanks,
 Enric

> 
> 
> Daniel.
--
To unsubscribe from this list: send the line "unsubscribe linux-pwm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Lothar Waßmann Nov. 14, 2017, 7:56 a.m. UTC | #3
Hi,

On Fri, 10 Nov 2017 12:22:15 +0100 Enric Balletbo i Serra wrote:
> Hi all,
> 
> On 08/11/17 11:48, Daniel Thompson wrote:
> > On 26/10/17 13:49, Lothar Waßmann wrote:
> >> These patches implement some measures to prevent backlight flicker
> >> when the backlight is being switched on for a display with an active
> >> low brightness control pin.
> >> GIT: [PATCH 1/2] backlight: pwm_bl: Enable PWM before switching regulator
> >> GIT: [PATCH 2/2] backlight: pwm_bl: add configurable delay between
> > 
> > Other than hoisting the pwm_enable() even earlier in the setup sequence this
> > patchset seems to have a significant overlap with Enric's recent posting:
> > 
> >   https://lkml.org/lkml/2017/7/21/211
> > 
> > Any chance of a shared view on this, especially on the DT bindings?
> > 
> 
> The DT binding were discussed some time ago for the patch series I sent, though
> there isn't a final ack from DT maintainer.
> 
> Lothar, the series I sent have been reviewed and acked, can you test if the
> series fixes the problem for you too?
> 
I'll try to test it within the next couple of days and will report back.


Lothar Waßmann
--
To unsubscribe from this list: send the line "unsubscribe linux-pwm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html