[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAOCk7NrGgDNEczP5mqirUzwoE+wKF-D5nYvJXfa-e3eFni62iw@mail.gmail.com>
Date: Thu, 13 Jun 2019 09:40:27 -0600
From: Jeffrey Hugo <jeffrey.l.hugo@...il.com>
To: Bjorn Andersson <bjorn.andersson@...aro.org>
Cc: lgirdwood@...il.com, broonie@...nel.org,
Andy Gross <agross@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
MSM <linux-arm-msm@...r.kernel.org>,
lkml <linux-kernel@...r.kernel.org>, devicetree@...r.kernel.org,
Jorge Ramirez <jorge.ramirez-ortiz@...aro.org>
Subject: Re: [PATCH v3 6/7] dt-bindings: qcom_spmi: Document pms405 support
On Thu, Jun 13, 2019 at 9:36 AM Bjorn Andersson
<bjorn.andersson@...aro.org> wrote:
>
> On Thu 13 Jun 07:24 PDT 2019, Jeffrey Hugo wrote:
>
> > From: Jorge Ramirez <jorge.ramirez-ortiz@...aro.org>
> >
> > The PMS405 supports 5 SMPS and 13 LDO regulators.
> >
> > Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@...aro.org>
> > Reviewed-by: Rob Herring <robh@...nel.org>
> > Signed-off-by: Jeffrey Hugo <jeffrey.l.hugo@...il.com>
> > ---
> > .../regulator/qcom,spmi-regulator.txt | 24 +++++++++++++++++++
> > 1 file changed, 24 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.txt b/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.txt
> > index ba94bc2d407a..19cffb239094 100644
> > --- a/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.txt
> > +++ b/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.txt
> > @@ -10,6 +10,7 @@ Qualcomm SPMI Regulators
> > "qcom,pm8941-regulators"
> > "qcom,pm8994-regulators"
> > "qcom,pmi8994-regulators"
> > + "qcom,pms405-regulators"
> >
> > - interrupts:
> > Usage: optional
> > @@ -111,6 +112,29 @@ Qualcomm SPMI Regulators
> > Definition: Reference to regulator supplying the input pin, as
> > described in the data sheet.
> >
> > +- vdd_s1-supply:
> > +- vdd_s2-supply:
> > +- vdd_s3-supply:
> > +- vdd_s4-supply:
> > +- vdd_s5-supply:
> > +- vdd_l1-supply:
> > +- vdd_l2-supply:
> > +- vdd_l3-supply:
> > +- vdd_l4-supply:
> > +- vdd_l5-supply:
> > +- vdd_l6-supply:
> > +- vdd_l7-supply:
> > +- vdd_l8-supply:
> > +- vdd_l9-supply:
> > +- vdd_l10-supply:
> > +- vdd_l11-supply:
> > +- vdd_l12-supply:
> > +- vdd_l13-supply:
>
> No, the supply pins are as follows:
>
> - vdd_l1_l2-supply:
> - vdd_l3_l8-supply:
> - vdd_l4-supply:
> - vdd_l5_l6-supply:
> - vdd_l10_l11_l12_l13-supply:
> - vdd_l7-supply:
> - vdd_l9-supply:
> - vdd_s1-supply:
> - vdd_s2-supply:
> - vdd_s3-supply:
> - vdd_s4-supply:
> - vdd_s5-supply:
Sure, will update.
>
>
> Regards,
> Bjorn
>
> > + Usage: optional (pms405 only)
> > + Value type: <phandle>
> > + Definition: Reference to regulator supplying the input pin, as
> > + described in the data sheet.
> > +
> > - qcom,saw-reg:
> > Usage: optional
> > Value type: <phandle>
> > --
> > 2.17.1
> >
Powered by blists - more mailing lists