linux-kernel - RE: [PATCH 2/2] dt-bindings: Add support for SAW documentation

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <09f301d3b6d6$fa79cfe0$ef6d6fa0$@codeaurora.org>
Date:   Thu, 8 Mar 2018 14:14:12 +0200
From:   <ilialin@...eaurora.org>
To:     "'Rob Herring'" <robh@...nel.org>
Cc:     <linux-kernel@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <lgirdwood@...il.com>, <broonie@...nel.org>,
        <mark.rutland@....com>, <amit.kucheria@...aro.org>,
        <nicolas.dechesne@...aro.org>, <celster@...eaurora.org>,
        <tfinkel@...eaurora.org>
Subject: RE: [PATCH 2/2] dt-bindings: Add support for SAW documentation

Thanks for the review, Rob.

I used existing drivers as reference and defined in the same way. Could you
point me to a reference that is correct from your point of view?

> -----Original Message-----
> From: Rob Herring [mailto:robh@...nel.org]
> Sent: Wednesday, March 7, 2018 22:41
> To: Ilia Lin <ilialin@...eaurora.org>
> Cc: linux-kernel@...r.kernel.org; devicetree@...r.kernel.org;
> lgirdwood@...il.com; broonie@...nel.org; mark.rutland@....com;
> amit.kucheria@...aro.org; nicolas.dechesne@...aro.org;
> celster@...eaurora.org; tfinkel@...eaurora.org
> Subject: Re: [PATCH 2/2] dt-bindings: Add support for SAW documentation
> 
> On Sun, Mar 04, 2018 at 02:31:49PM +0200, Ilia Lin wrote:
> > Add support for SAW controlled regulators in 8x96.
> > Document it.
> >
> > Signed-off-by: Ilia Lin <ilialin@...eaurora.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";
> 
> syscon should not be used alone. It should have a specific compatible too.
> 
> > +		reg = <0x9A10000 0x1000>;
> > +	};
> > +
> > +	...
> > +
> > +	spm-regulators {
> > +		compatible = "qcom,pm8994-regulators";
> > +		qcom,saw-reg = <&saw3>;
> 
> Why not just a child of the syscon?
> 
> > +		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>;
> > +		};
> > +	};
> > --
> > Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> > a Linux Foundation Collaborative Project
> >