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

[Yang, Wenyou]

Hi Mark,

> -----Original Message-----
> From: Mark Brown [mailto:broonie at kernel.org]
> Sent: 2016年1月16日 1:59
> To: Yang, Wenyou <Wenyou.Yang at atmel.com>
> Cc: Liam Girdwood <lgirdwood at gmail.com>; Rob Herring <robh+dt at kernel.org>;
> Pawel Moll <pawel.moll at arm.com>; Mark Rutland <mark.rutland at arm.com>; Ian
> Campbell <ijc+devicetree at hellion.org.uk>; Kumar Gala <galak at codeaurora.org>;
> Ferre, Nicolas <Nicolas.FERRE at atmel.com>; linux-arm-
> kernel at lists.infradead.org; linux-kernel at vger.kernel.org;
> devicetree at vger.kernel.org
> 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