RE: [PATCH 2/2] regulator: add documentation for ACT8945A's regulator DT bindings

[Yang, Wenyou]

Hi Mark,

> -----Original Message-----
> From: Mark Brown [mailto:broonie@xxxxxxxxxx]
> Sent: 2016年1月16日 1:59
> To: Yang, Wenyou <Wenyou.Yang@xxxxxxxxx>
> Cc: Liam Girdwood <lgirdwood@xxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>;
> Pawel Moll <pawel.moll@xxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>; Ian
> Campbell <ijc+devicetree@xxxxxxxxxxxxxx>; Kumar Gala <galak@xxxxxxxxxxxxxx>;
> Ferre, Nicolas <Nicolas.FERRE@xxxxxxxxx>; linux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 2/2] regulator: add documentation for ACT8945A's regulator
> DT bindings
> 
> On Fri, Jan 08, 2016 at 10:08:58AM +0800, Wenyou Yang wrote:
> 
> > +Optional properties:
> > +- active-semi,vsel-high: Indicates if the VSEL pin is set to logic-high.
> > +  If this property is missing, assume the VSEL pin is set to logic-low.
> 
> Wouldn't such a pin normally be connected to a GPIO for runtime control
> - why is it expected to be tied statically all the time?

On the reference board, it is connected to the VSYS_5V statically.

Yes, it should be connected to a GPIO, it should be controllable at the necessary time.


Best Regards,
Wenyou Yang