Re: [PATCHv7][ 2/2] video: backlight: gpio-backlight: Add DT support.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> On Thu, Dec 05, 2013 at 06:55:09PM +0100, Denis Carikli wrote:
> [...]
> > +Optional properties:
> > +  - default-state: The initial state of the backlight.
> > +    Valid values are "on", "off", and "keep".
> > +    The "keep" setting will keep the backlight at whatever its current
> > +    state is, without producing a glitch. The default is keep if this
> > +    property is not present.
> 
> I'm not sure if "on", "off" and "keep" are a good choice for this
> binding. Having strings for these tristate values seems suboptimal.
> Other bindings have chosen a representation that, transposed to this
> use-case, would read something like this:
> 
> 	- default-state: The initial state of the backlight. Valid
> 	  values:
> 	  - 0: off
> 	  - 1: on
> 
> 	If the "default-state" property is not present, the default
> 	will be to keep the current backlight state.
> 
> Which is in fact the exact behaviour that your binding describes, but
> it's much more intuitive in my opinion.

Why we cannot use GPIO bindings for active level here?
What a reason for "keep" state? Can this be an additional property?

---
��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux