Re: [PATCH v3 1/3] ARM: dts: rockchip: veyron: Remove 0 point from brightness-levels

2020-10-28 Thread Daniel Thompson
On Wed, Oct 21, 2020 at 10:04:43PM -0700, Alexandru Stan wrote: > 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

Re: [PATCH v3 1/3] ARM: dts: rockchip: veyron: Remove 0 point from brightness-levels

2020-10-22 Thread Doug Anderson
Hi, On Wed, Oct 21, 2020 at 10:05 PM Alexandru Stan wrote: > > 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

[PATCH v3 1/3] ARM: dts: rockchip: veyron: Remove 0 point from brightness-levels

2020-10-21 Thread Alexandru Stan
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