On Wed, Sep 23, 2015 at 10:23:58AM -0700, Bjorn Andersson wrote: > On Wed 23 Sep 09:53 PDT 2015, Andy Gross wrote: > > > This patch adds support and documentation for the PM8916 regulators > > found on MSM8916 platforms. > > > > Signed-off-by: Andy Gross <agr...@codeaurora.org> > > --- > > .../bindings/soc/qcom/qcom,smd-rpm-regulator.txt | 18 ++++++ > > drivers/regulator/qcom_smd-regulator.c | 64 > > ++++++++++++++++++++ > > 2 files changed, 82 insertions(+) > > > > diff --git > > a/Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm-regulator.txt > > b/Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm-regulator.txt > > index 7084474..05dfc1a 100644 > > --- a/Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm-regulator.txt > > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm-regulator.txt > > @@ -18,6 +18,7 @@ Regulator nodes are identified by their compatible: > > Definition: must be one of: > > "qcom,rpm-pm8841-regulators" > > "qcom,rpm-pm8941-regulators" > > + "qcom,rpm-pm8916-regulators" > > Alphabetical please > > > > > - vdd_s1-supply: > > - vdd_s2-supply: > > @@ -50,6 +51,19 @@ Regulator nodes are identified by their compatible: > > 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_l1_l2_l3-supply > > +- vdd_l4_l5_l6-supply > > +- vdd_l7-supply > > End these with ':' as well > > > +- vdd_l8_l9_l10_l11_l12_l13_l14_l15_l16_l17_l18-supply: > > + Usage: optional (pm8916 only) > > + Value type: <phandle> > > + Definition: reference to regulator supplying the input pin, as > > + described in the data sheet > > + > [..] > > diff --git a/drivers/regulator/qcom_smd-regulator.c > > b/drivers/regulator/qcom_smd-regulator.c > [..] > > static const struct of_device_id rpm_of_match[] = { > > + { .compatible = "qcom,rpm-pm8916-regulators", .data = > > &rpm_pm8916_regulators }, > > pm8916 > pm8841 > > > { .compatible = "qcom,rpm-pm8841-regulators", .data = > > &rpm_pm8841_regulators }, > > { .compatible = "qcom,rpm-pm8941-regulators", .data = > > &rpm_pm8941_regulators }, > > {} > > With these nits: > > Acked-by: Bjorn Andersson <bjorn.anders...@sonymobile.com> > > Regards, > Bjorn
I'll fixup this when I apply it. Thanks! -- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/