On 08/09/15 16:41, Andrew Lunn wrote: > On Tue, Sep 08, 2015 at 02:19:36PM +0300, Tomi Valkeinen wrote: >> Add DT binding for led-backlight. >> >> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxx> >> --- >> .../bindings/video/backlight/led-backlight.txt | 30 ++++++++++++++++++++++ >> 1 file changed, 30 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/video/backlight/led-backlight.txt >> >> diff --git a/Documentation/devicetree/bindings/video/backlight/led-backlight.txt b/Documentation/devicetree/bindings/video/backlight/led-backlight.txt >> new file mode 100644 >> index 000000000000..fb77051ac230 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/video/backlight/led-backlight.txt >> @@ -0,0 +1,30 @@ >> +led-backlight bindings >> + >> +Required properties: >> + - compatible: "led-backlight" >> + - leds: phandle to a led OF node [0] >> + - brightness-levels: Array of distinct LED brightness levels. These >> + are in the range from 0 to 255, passed to the LED class driver. >> + - default-brightness-level: the default brightness level (index into the >> + array defined by the "brightness-levels" property) >> + - power-supply: regulator for supply voltage > > Hi Tomi > > Maybe this regulator should be optional? I could imagine blacklights > without one, in none power sensitive cases, e.g. industrial PC with a > touch screen display, train ticket machine, etc. Yes, I think so. Especially in this case as this led-backlight device is not exactly a specific HW device, but more of a virtual device. Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature
