Re: [PATCH 1/2] regulator: core: Allow use of "status = disabled" in regulator dts nodes

[Hans de Goede]

Hi,

On 27-04-16 16:37, Mark Brown wrote:
> On Wed, Apr 27, 2016 at 04:31:02PM +0200, Hans de Goede wrote:
> > On 27-04-16 16:24, Mark Brown wrote:
>
> > > The regulator API should not touch any regulators that it doesn't have
> > > permission to change the state for.  All other regulators are strictly
> > > read only.
>
> > How do we give permission to change state ? Is omitting the dts node,
> > and thus not returning a node / constrains from regulator_of_get_init_data
> > enough for the regulator API to not have permission ?
>
> Yes, omit the DT node or mark it always on at the minute.

Or, since regulator_of_get_init_data uses for_each_available_child_of_node
which checks the "status" value we can actually already use
"status=disabled" cool. So this simply already works :)

Regards,

Hans