Re: [PATCH v4 2/2] ARM: tegra: paz00: enable backlight

[Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>]

On Saturday 17 November 2012 08:35:34 Thierry Reding wrote:
> On Sat, Nov 17, 2012 at 12:20:32AM +0100, Marc Dietrich wrote:
> [...]
> 
> > +				step1 {
> > +					type = "gpio";
> > +					gpio = <&gpio 4 0>;   /* gpio PA4, en_vdd_pnl  */
> > +					value = <1>;
> > +				};
> > +				step2 {
> > +					type = "gpio";
> > +					gpio = <&gpio 176 0>; /* gpio PW0, bl_vdd */
> > +					value = <1>;
> > +				};
> 
> These look like regulators, so maybe they should be abstracted as such?

Honstly, I never understood why we need to redefine a gpio as a regulator. 
IMHO, gpios don't regulate, they switch 0/1. But yes, these gpios just go to 
some FET which switches on some supply voltage.

Marc