Hi Geert,
On Monday 21 Nov 2016 09:36:22 Geert Uytterhoeven wrote:
> On Sat, Nov 19, 2016 at 4:28 AM, Laurent Pinchart wrote:
> > The panel backlight is controlled through a GPIO and a PWM channel.
> >
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
>
> > --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> > +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> > @@ -178,6 +178,16 @@
> > };
> > };
> > };
> > +
> > + backlight: backlight {
> > + compatible = "pwm-backlight";
> > + pwms = <&pwm1 0 50000>;
> > +
> > + brightness-levels = <256 128 64 16 8 4 0>;
>
> Would it make sense to define more and/or linear levels?
Possibly, this is pretty arbitrary. Linear levels might not be the best option
given that the human eye doesn't have a linear response to light power, but we
could certainly have more levels. In that case I'd prefer modifying the pwm-
backlight DT bindings though, and specifying the PWM resolution instead of
discrete levels.
Note that the LVDS panel backlight PWM control signal is multiplexed with the
external memory A21 signal on the Salvator-X board, with SW5 selecting which
how to route the signal. When using backlight control we can't access the
whole NOR flash anymore, so I'm not sure this patch should be merged.
> > + default-brightness-level = <6>;
--
Regards,
Laurent Pinchart
