| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <017401d3edb0$0e3d6d10$2ab84730$@codeaurora.org>
Date: Thu, 17 May 2018 10:24:05 +0300
From: <ilialin@...eaurora.org>
To: "'Amit Kucheria'" <amit.kucheria@...aro.org>
Cc: "'Michael Turquette'" <mturquette@...libre.com>,
<sboyd@...nel.org>, "'Rob Herring'" <robh@...nel.org>,
"'Mark Rutland'" <mark.rutland@....com>,
"'Viresh Kumar'" <viresh.kumar@...aro.org>, <nm@...com>,
<lgirdwood@...il.com>, <broonie@...nel.org>,
"'Andy Gross'" <andy.gross@...aro.org>,
"'David Brown'" <david.brown@...aro.org>,
<catalin.marinas@....com>, <will.deacon@....com>,
"'Rafael J. Wysocki'" <rjw@...ysocki.net>,
<linux-clk@...r.kernel.org>, <devicetree@...r.kernel.org>,
"'LKML'" <linux-kernel@...r.kernel.org>,
"'Linux PM list'" <linux-pm@...r.kernel.org>,
<linux-arm-msm@...r.kernel.org>, <linux-soc@...r.kernel.org>,
"'lakml'" <linux-arm-kernel@...ts.infradead.org>,
"'Rajendra Nayak'" <rnayak@...eaurora.org>,
<nicolas.dechesne@...aro.org>, <celster@...eaurora.org>,
<tfinkel@...eaurora.org>
Subject: RE: [PATCH v7 10/14] dt-bindings: qcom_spmi: Add support for SAW documentation
> -----Original Message-----
> From: amit.kucheria@...durent.com <amit.kucheria@...durent.com> On
> Behalf Of Amit Kucheria
> Sent: Wednesday, May 16, 2018 16:13
> To: Ilia Lin <ilialin@...eaurora.org>
> Cc: Michael Turquette <mturquette@...libre.com>; sboyd@...nel.org; Rob
> Herring <robh@...nel.org>; Mark Rutland <mark.rutland@....com>; Viresh
> Kumar <viresh.kumar@...aro.org>; nm@...com; lgirdwood@...il.com;
> broonie@...nel.org; Andy Gross <andy.gross@...aro.org>; David Brown
> <david.brown@...aro.org>; catalin.marinas@....com;
> will.deacon@....com; Rafael J. Wysocki <rjw@...ysocki.net>; linux-
> clk@...r.kernel.org; devicetree@...r.kernel.org; LKML <linux-
> kernel@...r.kernel.org>; Linux PM list <linux-pm@...r.kernel.org>; linux-
> arm-msm@...r.kernel.org; linux-soc@...r.kernel.org; lakml <linux-arm-
> kernel@...ts.infradead.org>; Rajendra Nayak <rnayak@...eaurora.org>;
> nicolas.dechesne@...aro.org; celster@...eaurora.org;
> tfinkel@...eaurora.org
> Subject: Re: [PATCH v7 10/14] dt-bindings: qcom_spmi: Add support for SAW
> documentation
>
> On Tue, May 15, 2018 at 12:13 PM, Ilia Lin <ilialin@...eaurora.org> wrote:
> > Add support for SAW controlled regulators.
> > The regulators defined as SAW controlled in the device tree will be
> > controlled through special CPU registers instead of direct SPMI
> > accesses.
> > This is required especially for CPU supply regulators to synchronize
> > with clock scaling and for Automatic Voltage Switching.
> > Document it.
>
> Replace this boiler plate with what this patch actual does. Besides changing
> the subject, it could be, for example,
>
> "Document the DT bindings for the SAW regulators.
>
> The saw-slave property allows ganging (grouping) of several regulators so
> that their outputs can be combined... blah blah.
>
> The saw-leader is the only one that then is configurable in DT"
Actually, I invested some fantasy to write this explanation. But I'll try to revise it.
>
>
> > Signed-off-by: Ilia Lin <ilialin@...eaurora.org>
> > Reviewed-by: Rob Herring <robh@...nel.org>
> > ---
> > .../bindings/regulator/qcom,spmi-regulator.txt | 45
> ++++++++++++++++++++++
> > 1 file changed, 45 insertions(+)
> >
> > diff --git
> > a/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.txt
> > b/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.txt
> > index 57d2c65..406f2e5 100644
> > ---
> > a/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.txt
> > +++ b/Documentation/devicetree/bindings/regulator/qcom,spmi-
> regulator.
> > +++ txt
> > @@ -110,6 +110,11 @@ Qualcomm SPMI Regulators
> > Definition: Reference to regulator supplying the input pin, as
> > described in the data sheet.
> >
> > +- qcom,saw-reg:
> > + Usage: optional
> > + Value type: <phandle>
> > + Description: Reference to syscon node defining the SAW registers.
> > +
> >
> > The regulator node houses sub-nodes for each regulator within the
> > device. Each sub-node is identified using the node's name, with valid
> > values listed for each @@ -201,6 +206,17 @@ see regulator.txt - with
> additional custom properties described below:
> > 2 = 0.55 uA
> > 3 = 0.75 uA
> >
> > +- qcom,saw-slave:
> > + Usage: optional
> > + Value type: <boo>
> > + Description: SAW controlled gang slave. Will not be configured.
> > +
> > +- qcom,saw-leader:
> > + Usage: optional
> > + Value type: <boo>
> > + Description: SAW controlled gang leader. Will be configured as
> > + SAW regulator.
> > +
> > Example:
> >
> > regulators {
> > @@ -221,3 +237,32 @@ Example:
> >
> > ....
> > };
> > +
> > +Example 2:
> > +
> > + saw3: syscon@...0000 {
> > + compatible = "syscon";
> > + reg = <0x9A10000 0x1000>;
> > + };
> > +
> > + ...
> > +
> > + spm-regulators {
> > + compatible = "qcom,pm8994-regulators";
> > + qcom,saw-reg = <&saw3>;
> > + s8 {
> > + qcom,saw-slave;
> > + };
> > + s9 {
> > + qcom,saw-slave;
> > + };
> > + s10 {
> > + qcom,saw-slave;
> > + };
> > + pm8994_s11_saw: s11 {
> > + qcom,saw-leader;
> > + regulator-always-on;
> > + regulator-min-microvolt = <900000>;
> > + regulator-max-microvolt = <1140000>;
> > + };
> > + };
> > --
> > 1.9.1
> >
Powered by blists - more mailing lists