Re: [PATCH 01/16] PM / OPP: Add 'supply-names' binding

From	Viresh Kumar <viresh.kumar@linaro.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 01/16] PM / OPP: Add 'supply-names' binding
Date	2015-10-27 09:20 +0100
Message-ID	<qo7E7-2QQ-15@gated-at.bofh.it> (permalink)
References	(3 earlier) <qk14e-6og-13@gated-at.bofh.it> <qk6ng-5PQ-7@gated-at.bofh.it> <qkiHM-7dx-13@gated-at.bofh.it> <qkr8l-2Jf-3@gated-at.bofh.it> <qmxCH-Il-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 23-10-15, 01:39, Mark Brown wrote:
> When we start doing this we also start having to worry about things like
> the sequencing of the updates between the various supplies and end up in
> full on power sequencing (or at least baking some sequencing into the DT
> which will doubtless need extending at some point).

Absolutely.

> I'm not sure that's
> a place we want to end up just yet, I think it's safer to just have a
> little bit of code in the kernel that glues things together in the cases
> where this is needed.  

So you are effectively saying that we shouldn't go ahead with multi
regulator support in OPP library, right?

I went ahead with it as it came as a requirement (specially from
Qcom).

To the problem of sequencing, maybe we can just support that for the
simple case, where supplies will be programmed in the order in which
they are present in the property I added in this patch. And not try to
solve problem for the complex cases, if we feel it is getting ugly.

@Stephen ?

-- 
viresh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

Re: [PATCH 01/16] PM / OPP: Add 'supply-names' binding Mark Brown <broonie@kernel.org> - 2015-10-23 01:40 +0200
  Re: [PATCH 01/16] PM / OPP: Add 'supply-names' binding Viresh Kumar <viresh.kumar@linaro.org> - 2015-10-27 09:20 +0100
    Re: [PATCH 01/16] PM / OPP: Add 'supply-names' binding Mark Brown <broonie@kernel.org> - 2015-10-28 09:20 +0100
      Re: [PATCH 01/16] PM / OPP: Add 'supply-names' binding Stephen Boyd <sboyd@codeaurora.org> - 2015-10-30 00:40 +0100

csiph-web