On 21/04/2020 14:26, Tomi Valkeinen wrote:
The led-backlight.txt is
a bit lacking (another thing to improve...) but led-backlight mimics
pwm-backlight, and pwm-backlight.txt says
default-brightness-level: The default brightness level (index into the array
defined by the "brightness-levels" property)
I think this implies we should improve the binding documentation!
The parenthetic text's main purpose is to make clear which scale should
be used when interpreting the default brightness. Just because the scale
is 1:1 doesn't render it meaningless.
If I read pwm_bl.c right, that's not how the code works. If pwm_bl has no 'brightness-levels', then
'default-brightness-level' is ignored. Which matches the binding documentation.
What you suggest makes sense, though, so I can adjust this patch to make led_bl behave like that.
On the other hand... If we want to use LED's (or PWM's) brightness levels directly, should the
default brightness be defined in the LED's (or PWM's) DT node?
And only if we create a backlight brightness-levels mapping, we need to add a new property to define
a default for those levels. Which, I think, the name implies with the "-level".
Well, in any case, there should be no harm in using 'default-brightness-level' also for the 1:1
mapping without the 'brightness-levels'. So maybe that's the best route.
Tomi
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
