On Wed, Jun 29, 2016 at 04:50:27PM +0530, Ritesh Harjani wrote: > QUALCOMM sdhci-msm driver follows a pwr_irq method > (deviation from regular standard) for any pwr related > operations (which also involves regulator settings changes). > > vmmc/vqmmc regulator names are used by core layer, > so change the names to vdd/vdd-io so that it can > be used by sdhci-msm driver alone. > > Signed-off-by: Ritesh Harjani <riteshh@xxxxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/mmc/sdhci-msm.txt | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > index 485483a..851e66d 100644 > --- a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > +++ b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > @@ -27,8 +27,8 @@ Example: > bus-width = <8>; > non-removable; > > - vmmc-supply = <&pm8941_l20>; > - vqmmc-supply = <&pm8941_s3>; > + vdd-supply = <&pm8941_l20>; > + vdd-io-supply = <&pm8941_s3>; If you are adding supplies, you need to specify them in the required properties. And if these properties are required by a specific compat you need to add the compat. Because this binding has consumers, you need to think about them as well. Is the pwr_irq stuff still part of the v4 controller? If not and this is a new revision, perhaps a new binding/compat is required. This would make dealing with it in the driver easier as well. > > pinctrl-names = "default"; > pinctrl-0 = <&sdc1_clk &sdc1_cmd &sdc1_data>; > @@ -44,8 +44,8 @@ Example: > bus-width = <4>; > cd-gpios = <&msmgpio 62 0x1>; > > - vmmc-supply = <&pm8941_l21>; > - vqmmc-supply = <&pm8941_l13>; > + vdd-supply = <&pm8941_l21>; > + vdd-io-supply = <&pm8941_l13>; ditto above > > pinctrl-names = "default"; > pinctrl-0 = <&sdc2_clk &sdc2_cmd &sdc2_data>; -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
