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 ?韬{.n?????%??檩??w?{.n????z谵{???塄}?财??j:+v??????2??璀??摺?囤??z夸z罐?+?????w棹f