On Wed, Oct 14, 2020 at 7:19 AM Daniel Thompson
[off-list ref] wrote:
On Tue, Oct 13, 2020 at 01:01:02AM -0700, Alexandru Stan wrote:
quoted
After the "PWM backlight interpolation adjustments" patches, the
backlight interpolation works a little differently. The way these
dts files were working before was relying on a bug (IMHO).
Remove the 0-3 range since otherwise we would have a 252 long
interpolation that would slowly go between 0 and 3, looking really bad
in userspace.
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.
Signed-off-by: Alexandru Stan <redacted>
Acked-by: Daniel Thompson <redacted>
Thank you!
Note also shouldn't this be patch 1 of the set. AFAICT it makes sense
whether or not the interpolation algorithm is changed.
Yeah, I guess it could be. Sorry I didn't think of it that way before,
I'm used to landing things in a group.
In particular on veyron I assume it will almost be a noop without
having my driver patch (especially with the findings of 0% not being
that important).
Feel free to land this independently.
Daniel.
Alexandru Stan (amstan)