Re: [PATCH V4 2/2] regulator: mt6323: Add support for MT6323 regulator

From: Mark Brown
Date: Thu Jan 28 2016 - 06:38:58 EST


On Thu, Jan 28, 2016 at 03:16:41PM +0800, Henry Chen wrote:
> On Wed, 2016-01-27 at 14:41 +0000, Mark Brown wrote:

> > No, drivers should *never* enable things that weren't explictly enabled
> > by the machine constraints. This misses the whole point of having
> > constraints. They are there so that the system integrator can enable
> > the functionality that is safe on a given board.

> Okay..the constrains should be define on device tree.
> But which optional properties was suitable to fill on device tree if consumers want to call
> regulator_set_mode directly ?
> I have check the of_regulator.c and not found the suitable property name which can set valid_modes_mask & valid_ops_mask.

If you need to change the mode at runtime you will need to develop a
binding for that, there isn't one at present.

Attachment: signature.asc
Description: PGP signature