Hi Mark,

-----Original Message-----
From: Mark Brown [mailto:broonie@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@lists.infradead.org; linux-kernel@vger.kernel.org;
devicetree@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 
N�Р骒r��y����b�X�肚�v�^�)藓{.n�+���z��z谵z)��骅w*jg�报
�����茛j.镗�\浇M��gj替a����' �┺⒏
⒎�j:+v���w�j�m������赙zZ+�����茛j"��!�i