Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

ARM: dts: rockchip: Remove 0 point from brightness-levels on rk3288-veyron

The extra 0 only adds one point in the userspace visible range,
so this change is almost a noop with the current driver behavior.

We don't need the 0% point, userspace seems to handle this just fine
because it uses the bl_power property to turn off the display.

Furthermore after adding "backlight: pwm_bl: Fix interpolation" patch,
the backlight interpolation will work a little differently. So we need
to preemptively remove the 0-3 segment since otherwise we would have a
252 long interpolation that would slowly go between 0 and 3, looking
really bad in userspace. So it's almost a noop/cleanup now, but it will
be required in the future.

Signed-off-by: Alexandru Stan <amstan@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
Link: https://lore.kernel.org/r/20201021220404.v3.1.I96b8d872ec51171f19274e43e96cadc092881271@changeid
Signed-off-by: Heiko Stuebner <heiko@sntech.de>

authored by

Alexandru Stan and committed by
Heiko Stuebner
225c59b9 3650b228

+3 -3
+1 -1
arch/arm/boot/dts/rk3288-veyron-jaq.dts
··· 20 20 21 21 &backlight { 22 22 /* Jaq panel PWM must be >= 3%, so start non-zero brightness at 8 */ 23 - brightness-levels = <0 8 255>; 23 + brightness-levels = <8 255>; 24 24 num-interpolated-steps = <247>; 25 25 }; 26 26
+1 -1
arch/arm/boot/dts/rk3288-veyron-minnie.dts
··· 39 39 40 40 &backlight { 41 41 /* Minnie panel PWM must be >= 1%, so start non-zero brightness at 3 */ 42 - brightness-levels = <0 3 255>; 42 + brightness-levels = <3 255>; 43 43 num-interpolated-steps = <252>; 44 44 }; 45 45
+1 -1
arch/arm/boot/dts/rk3288-veyron-tiger.dts
··· 23 23 24 24 &backlight { 25 25 /* Tiger panel PWM must be >= 1%, so start non-zero brightness at 3 */ 26 - brightness-levels = <0 3 255>; 26 + brightness-levels = <3 255>; 27 27 num-interpolated-steps = <252>; 28 28 }; 29 29