Hi Mark,

-----Original Message-----
From: Mark Brown [mailto:broonie at kernel.org]
Sent: 2016?1?16? 1:59
To: Yang, Wenyou <redacted>
Cc: Liam Girdwood <lgirdwood@gmail.com>; Rob Herring <robh+dt@kernel.org>;
Pawel Moll [off-list ref]; Mark Rutland [off-list ref]; Ian
Campbell [off-list ref]; Kumar Gala [off-list ref];
Ferre, Nicolas [off-list ref]; 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:

quoted

+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