| Message ID | 1443188657-13501-1-git-send-email-javier@osg.samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Javier, On 25 September 2015 at 19:14, Javier Martinez Canillas <javier@osg.samsung.com> wrote: > The regulator-compatible property from the regulator DT binding was > deprecated and the correct approach is to use the node's name. > > This patch has no functional changes but by not using a deprecated > property, new DTS based on this one will not carry the same issue. > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> > > --- > > arch/arm/boot/dts/exynos4412-trats2.dts | 105 +++++++++++--------------------- > 1 file changed, 35 insertions(+), 70 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts b/arch/arm/boot/dts/exynos4412-trats2.dts > index 2a1ebb76ebe0..9ee55119e7d2 100644 > --- a/arch/arm/boot/dts/exynos4412-trats2.dts > +++ b/arch/arm/boot/dts/exynos4412-trats2.dts > @@ -564,16 +564,14 @@ > #clock-cells = <1>; > > voltage-regulators { > - ldo1_reg: ldo1 { > - regulator-compatible = "LDO1"; > + ldo1_reg: LDO1 { > regulator-name = "VALIVE_1.0V_AP"; > regulator-min-microvolt = <1000000>; > regulator-max-microvolt = <1000000>; > regulator-always-on; > }; > > - ldo2_reg: ldo2 { > - regulator-compatible = "LDO2"; > + ldo2_reg: LDO2 { > regulator-name = "VM1M2_1.2V_AP"; > regulator-min-microvolt = <1200000>; > regulator-max-microvolt = <1200000>; > @@ -583,32 +581,28 @@ > }; > }; > > - ldo3_reg: ldo3 { > - regulator-compatible = "LDO3"; > + ldo3_reg: LDO3 { > regulator-name = "VCC_1.8V_AP"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > regulator-always-on; > }; > > - ldo4_reg: ldo4 { > - regulator-compatible = "LDO4"; > + ldo4_reg: LDO4 { > regulator-name = "VCC_2.8V_AP"; > regulator-min-microvolt = <2800000>; > regulator-max-microvolt = <2800000>; > regulator-always-on; > }; > > - ldo5_reg: ldo5 { > - regulator-compatible = "LDO5"; > + ldo5_reg: LDO5 { > regulator-name = "VCC_1.8V_IO"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > regulator-always-on; > }; > > - ldo6_reg: ldo6 { > - regulator-compatible = "LDO6"; > + ldo6_reg: LDO6 { > regulator-name = "VMPLL_1.0V_AP"; > regulator-min-microvolt = <1000000>; > regulator-max-microvolt = <1000000>; > @@ -618,8 +612,7 @@ > }; > }; > > - ldo7_reg: ldo7 { > - regulator-compatible = "LDO7"; > + ldo7_reg: LDO7 { > regulator-name = "VPLL_1.0V_AP"; > regulator-min-microvolt = <1000000>; > regulator-max-microvolt = <1000000>; > @@ -629,8 +622,7 @@ > }; > }; > > - ldo8_reg: ldo8 { > - regulator-compatible = "LDO8"; > + ldo8_reg: LDO8 { > regulator-name = "VMIPI_1.0V"; > regulator-min-microvolt = <1000000>; > regulator-max-microvolt = <1000000>; > @@ -639,15 +631,13 @@ > }; > }; > > - ldo9_reg: ldo9 { > - regulator-compatible = "LDO9"; > + ldo9_reg: LDO9 { > regulator-name = "CAM_ISP_MIPI_1.2V"; > regulator-min-microvolt = <1200000>; > regulator-max-microvolt = <1200000>; > }; > > - ldo10_reg: ldo10 { > - regulator-compatible = "LDO10"; Missing the node name 'LDO10' > + ldo10_reg: DO10 { > regulator-name = "VMIPI_1.8V"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > @@ -656,8 +646,7 @@ > }; > }; > > - ldo11_reg: ldo11 { > - regulator-compatible = "LDO11"; > + ldo11_reg: LDO11 { > regulator-name = "VABB1_1.95V"; > regulator-min-microvolt = <1950000>; > regulator-max-microvolt = <1950000>; > @@ -667,8 +656,7 @@ > }; > }; > > - ldo12_reg: ldo12 { > - regulator-compatible = "LDO12"; > + ldo12_reg: LDO12 { > regulator-name = "VUOTG_3.0V"; > regulator-min-microvolt = <3000000>; > regulator-max-microvolt = <3000000>; > @@ -677,15 +665,13 @@ > }; > }; > > - ldo13_reg: ldo13 { > - regulator-compatible = "LDO13"; > + ldo13_reg: LDO13 { > regulator-name = "NFC_AVDD_1.8V"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > }; > > - ldo14_reg: ldo14 { > - regulator-compatible = "LDO14"; > + ldo14_reg: LDO14 { > regulator-name = "VABB2_1.95V"; > regulator-min-microvolt = <1950000>; > regulator-max-microvolt = <1950000>; > @@ -695,8 +681,7 @@ > }; > }; > > - ldo15_reg: ldo15 { > - regulator-compatible = "LDO15"; > + ldo15_reg: LDO15 { > regulator-name = "VHSIC_1.0V"; > regulator-min-microvolt = <1000000>; > regulator-max-microvolt = <1000000>; > @@ -705,8 +690,7 @@ > }; > }; > > - ldo16_reg: ldo16 { > - regulator-compatible = "LDO16"; > + ldo16_reg: LDO16 { > regulator-name = "VHSIC_1.8V"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > @@ -715,80 +699,69 @@ > }; > }; > > - ldo17_reg: ldo17 { > - regulator-compatible = "LDO17"; > + ldo17_reg: LDO17 { > regulator-name = "CAM_SENSOR_CORE_1.2V"; > regulator-min-microvolt = <1200000>; > regulator-max-microvolt = <1200000>; > }; > > - ldo18_reg: ldo18 { > - regulator-compatible = "LDO18"; > + ldo18_reg: LDO18 { > regulator-name = "CAM_ISP_SEN_IO_1.8V"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > }; > > - ldo19_reg: ldo19 { > - regulator-compatible = "LDO19"; > + ldo19_reg: LDO19 { > regulator-name = "VT_CAM_1.8V"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > }; > > - ldo20_reg: ldo20 { > - regulator-compatible = "LDO20"; > + ldo20_reg: LDO20 { > regulator-name = "VDDQ_PRE_1.8V"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > }; > > - ldo21_reg: ldo21 { > - regulator-compatible = "LDO21"; > + ldo21_reg: LDO21 { > regulator-name = "VTF_2.8V"; > regulator-min-microvolt = <2800000>; > regulator-max-microvolt = <2800000>; > maxim,ena-gpios = <&gpy2 0 GPIO_ACTIVE_HIGH>; > }; > > - ldo22_reg: ldo22 { > - regulator-compatible = "LDO22"; > + ldo22_reg: LDO22 { > regulator-name = "VMEM_VDD_2.8V"; > regulator-min-microvolt = <2800000>; > regulator-max-microvolt = <2800000>; > maxim,ena-gpios = <&gpk0 2 GPIO_ACTIVE_HIGH>; > }; > > - ldo23_reg: ldo23 { > - regulator-compatible = "LDO23"; > + ldo23_reg: LDO23 { > regulator-name = "TSP_AVDD_3.3V"; > regulator-min-microvolt = <3300000>; > regulator-max-microvolt = <3300000>; > }; > > - ldo24_reg: ldo24 { > - regulator-compatible = "LDO24"; > + ldo24_reg: LDO24 { > regulator-name = "TSP_VDD_1.8V"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > }; > > - ldo25_reg: ldo25 { > - regulator-compatible = "LDO25"; > + ldo25_reg: LDO25 { > regulator-name = "LCD_VCC_3.3V"; > regulator-min-microvolt = <2800000>; > regulator-max-microvolt = <2800000>; > }; > > - ldo26_reg: ldo26 { > - regulator-compatible = "LDO26"; > + ldo26_reg: LDO26 { > regulator-name = "MOTOR_VCC_3.0V"; > regulator-min-microvolt = <3000000>; > regulator-max-microvolt = <3000000>; > }; > > - buck1_reg: buck1 { > - regulator-compatible = "BUCK1"; > + buck1_reg: BUCK1 { > regulator-name = "vdd_mif"; > regulator-min-microvolt = <850000>; > regulator-max-microvolt = <1100000>; > @@ -799,8 +772,7 @@ > }; > }; > > - buck2_reg: buck2 { > - regulator-compatible = "BUCK2"; > + buck2_reg: BUCK2 { > regulator-name = "vdd_arm"; > regulator-min-microvolt = <850000>; > regulator-max-microvolt = <1500000>; > @@ -811,8 +783,7 @@ > }; > }; > > - buck3_reg: buck3 { > - regulator-compatible = "BUCK3"; > + buck3_reg: BUCK3 { > regulator-name = "vdd_int"; > regulator-min-microvolt = <850000>; > regulator-max-microvolt = <1150000>; > @@ -823,8 +794,7 @@ > }; > }; > > - buck4_reg: buck4 { > - regulator-compatible = "BUCK4"; > + buck4_reg: BUCK4 { > regulator-name = "vdd_g3d"; > regulator-min-microvolt = <850000>; > regulator-max-microvolt = <1150000>; > @@ -834,40 +804,35 @@ > }; > }; > > - buck5_reg: buck5 { > - regulator-compatible = "BUCK5"; > + buck5_reg: BUCK5 { > regulator-name = "VMEM_1.2V_AP"; > regulator-min-microvolt = <1200000>; > regulator-max-microvolt = <1200000>; > regulator-always-on; > }; > > - buck6_reg: buck6 { > - regulator-compatible = "BUCK6"; > + buck6_reg: BUCK6 { > regulator-name = "VCC_SUB_1.35V"; > regulator-min-microvolt = <1350000>; > regulator-max-microvolt = <1350000>; > regulator-always-on; > }; > > - buck7_reg: buck7 { > - regulator-compatible = "BUCK7"; > + buck7_reg: BUCK7 { > regulator-name = "VCC_SUB_2.0V"; > regulator-min-microvolt = <2000000>; > regulator-max-microvolt = <2000000>; > regulator-always-on; > }; > > - buck8_reg: buck8 { > - regulator-compatible = "BUCK8"; > + buck8_reg: BUCK8 { > regulator-name = "VMEM_VDDF_3.0V"; > regulator-min-microvolt = <2850000>; > regulator-max-microvolt = <2850000>; > maxim,ena-gpios = <&gpk0 2 GPIO_ACTIVE_HIGH>; > }; > > - buck9_reg: buck9 { > - regulator-compatible = "BUCK9"; > + buck9_reg: BUCK9 { > regulator-name = "CAM_ISP_CORE_1.2V"; > regulator-min-microvolt = <1000000>; > regulator-max-microvolt = <1200000>; > -- > 2.4.3 > -Anand Moon > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hello Anand, On 09/28/2015 12:10 PM, Anand Moon wrote: > Hi Javier, > [snip] >> >> - ldo10_reg: ldo10 { >> - regulator-compatible = "LDO10"; > Missing the node name 'LDO10' >> + ldo10_reg: DO10 { Thanks for pointing out, I had to look twice to understand what you were saying. Isn't that the LDO10 is missing but that was mistyped as "DO10". I posted a v2 fixing it. Best regards,
diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts b/arch/arm/boot/dts/exynos4412-trats2.dts index 2a1ebb76ebe0..9ee55119e7d2 100644 --- a/arch/arm/boot/dts/exynos4412-trats2.dts +++ b/arch/arm/boot/dts/exynos4412-trats2.dts @@ -564,16 +564,14 @@ #clock-cells = <1>; voltage-regulators { - ldo1_reg: ldo1 { - regulator-compatible = "LDO1"; + ldo1_reg: LDO1 { regulator-name = "VALIVE_1.0V_AP"; regulator-min-microvolt = <1000000>; regulator-max-microvolt = <1000000>; regulator-always-on; }; - ldo2_reg: ldo2 { - regulator-compatible = "LDO2"; + ldo2_reg: LDO2 { regulator-name = "VM1M2_1.2V_AP"; regulator-min-microvolt = <1200000>; regulator-max-microvolt = <1200000>; @@ -583,32 +581,28 @@ }; }; - ldo3_reg: ldo3 { - regulator-compatible = "LDO3"; + ldo3_reg: LDO3 { regulator-name = "VCC_1.8V_AP"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; regulator-always-on; }; - ldo4_reg: ldo4 { - regulator-compatible = "LDO4"; + ldo4_reg: LDO4 { regulator-name = "VCC_2.8V_AP"; regulator-min-microvolt = <2800000>; regulator-max-microvolt = <2800000>; regulator-always-on; }; - ldo5_reg: ldo5 { - regulator-compatible = "LDO5"; + ldo5_reg: LDO5 { regulator-name = "VCC_1.8V_IO"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; regulator-always-on; }; - ldo6_reg: ldo6 { - regulator-compatible = "LDO6"; + ldo6_reg: LDO6 { regulator-name = "VMPLL_1.0V_AP"; regulator-min-microvolt = <1000000>; regulator-max-microvolt = <1000000>; @@ -618,8 +612,7 @@ }; }; - ldo7_reg: ldo7 { - regulator-compatible = "LDO7"; + ldo7_reg: LDO7 { regulator-name = "VPLL_1.0V_AP"; regulator-min-microvolt = <1000000>; regulator-max-microvolt = <1000000>; @@ -629,8 +622,7 @@ }; }; - ldo8_reg: ldo8 { - regulator-compatible = "LDO8"; + ldo8_reg: LDO8 { regulator-name = "VMIPI_1.0V"; regulator-min-microvolt = <1000000>; regulator-max-microvolt = <1000000>; @@ -639,15 +631,13 @@ }; }; - ldo9_reg: ldo9 { - regulator-compatible = "LDO9"; + ldo9_reg: LDO9 { regulator-name = "CAM_ISP_MIPI_1.2V"; regulator-min-microvolt = <1200000>; regulator-max-microvolt = <1200000>; }; - ldo10_reg: ldo10 { - regulator-compatible = "LDO10"; + ldo10_reg: DO10 { regulator-name = "VMIPI_1.8V"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; @@ -656,8 +646,7 @@ }; }; - ldo11_reg: ldo11 { - regulator-compatible = "LDO11"; + ldo11_reg: LDO11 { regulator-name = "VABB1_1.95V"; regulator-min-microvolt = <1950000>; regulator-max-microvolt = <1950000>; @@ -667,8 +656,7 @@ }; }; - ldo12_reg: ldo12 { - regulator-compatible = "LDO12"; + ldo12_reg: LDO12 { regulator-name = "VUOTG_3.0V"; regulator-min-microvolt = <3000000>; regulator-max-microvolt = <3000000>; @@ -677,15 +665,13 @@ }; }; - ldo13_reg: ldo13 { - regulator-compatible = "LDO13"; + ldo13_reg: LDO13 { regulator-name = "NFC_AVDD_1.8V"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; }; - ldo14_reg: ldo14 { - regulator-compatible = "LDO14"; + ldo14_reg: LDO14 { regulator-name = "VABB2_1.95V"; regulator-min-microvolt = <1950000>; regulator-max-microvolt = <1950000>; @@ -695,8 +681,7 @@ }; }; - ldo15_reg: ldo15 { - regulator-compatible = "LDO15"; + ldo15_reg: LDO15 { regulator-name = "VHSIC_1.0V"; regulator-min-microvolt = <1000000>; regulator-max-microvolt = <1000000>; @@ -705,8 +690,7 @@ }; }; - ldo16_reg: ldo16 { - regulator-compatible = "LDO16"; + ldo16_reg: LDO16 { regulator-name = "VHSIC_1.8V"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; @@ -715,80 +699,69 @@ }; }; - ldo17_reg: ldo17 { - regulator-compatible = "LDO17"; + ldo17_reg: LDO17 { regulator-name = "CAM_SENSOR_CORE_1.2V"; regulator-min-microvolt = <1200000>; regulator-max-microvolt = <1200000>; }; - ldo18_reg: ldo18 { - regulator-compatible = "LDO18"; + ldo18_reg: LDO18 { regulator-name = "CAM_ISP_SEN_IO_1.8V"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; }; - ldo19_reg: ldo19 { - regulator-compatible = "LDO19"; + ldo19_reg: LDO19 { regulator-name = "VT_CAM_1.8V"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; }; - ldo20_reg: ldo20 { - regulator-compatible = "LDO20"; + ldo20_reg: LDO20 { regulator-name = "VDDQ_PRE_1.8V"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; }; - ldo21_reg: ldo21 { - regulator-compatible = "LDO21"; + ldo21_reg: LDO21 { regulator-name = "VTF_2.8V"; regulator-min-microvolt = <2800000>; regulator-max-microvolt = <2800000>; maxim,ena-gpios = <&gpy2 0 GPIO_ACTIVE_HIGH>; }; - ldo22_reg: ldo22 { - regulator-compatible = "LDO22"; + ldo22_reg: LDO22 { regulator-name = "VMEM_VDD_2.8V"; regulator-min-microvolt = <2800000>; regulator-max-microvolt = <2800000>; maxim,ena-gpios = <&gpk0 2 GPIO_ACTIVE_HIGH>; }; - ldo23_reg: ldo23 { - regulator-compatible = "LDO23"; + ldo23_reg: LDO23 { regulator-name = "TSP_AVDD_3.3V"; regulator-min-microvolt = <3300000>; regulator-max-microvolt = <3300000>; }; - ldo24_reg: ldo24 { - regulator-compatible = "LDO24"; + ldo24_reg: LDO24 { regulator-name = "TSP_VDD_1.8V"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; }; - ldo25_reg: ldo25 { - regulator-compatible = "LDO25"; + ldo25_reg: LDO25 { regulator-name = "LCD_VCC_3.3V"; regulator-min-microvolt = <2800000>; regulator-max-microvolt = <2800000>; }; - ldo26_reg: ldo26 { - regulator-compatible = "LDO26"; + ldo26_reg: LDO26 { regulator-name = "MOTOR_VCC_3.0V"; regulator-min-microvolt = <3000000>; regulator-max-microvolt = <3000000>; }; - buck1_reg: buck1 { - regulator-compatible = "BUCK1"; + buck1_reg: BUCK1 { regulator-name = "vdd_mif"; regulator-min-microvolt = <850000>; regulator-max-microvolt = <1100000>; @@ -799,8 +772,7 @@ }; }; - buck2_reg: buck2 { - regulator-compatible = "BUCK2"; + buck2_reg: BUCK2 { regulator-name = "vdd_arm"; regulator-min-microvolt = <850000>; regulator-max-microvolt = <1500000>; @@ -811,8 +783,7 @@ }; }; - buck3_reg: buck3 { - regulator-compatible = "BUCK3"; + buck3_reg: BUCK3 { regulator-name = "vdd_int"; regulator-min-microvolt = <850000>; regulator-max-microvolt = <1150000>; @@ -823,8 +794,7 @@ }; }; - buck4_reg: buck4 { - regulator-compatible = "BUCK4"; + buck4_reg: BUCK4 { regulator-name = "vdd_g3d"; regulator-min-microvolt = <850000>; regulator-max-microvolt = <1150000>; @@ -834,40 +804,35 @@ }; }; - buck5_reg: buck5 { - regulator-compatible = "BUCK5"; + buck5_reg: BUCK5 { regulator-name = "VMEM_1.2V_AP"; regulator-min-microvolt = <1200000>; regulator-max-microvolt = <1200000>; regulator-always-on; }; - buck6_reg: buck6 { - regulator-compatible = "BUCK6"; + buck6_reg: BUCK6 { regulator-name = "VCC_SUB_1.35V"; regulator-min-microvolt = <1350000>; regulator-max-microvolt = <1350000>; regulator-always-on; }; - buck7_reg: buck7 { - regulator-compatible = "BUCK7"; + buck7_reg: BUCK7 { regulator-name = "VCC_SUB_2.0V"; regulator-min-microvolt = <2000000>; regulator-max-microvolt = <2000000>; regulator-always-on; }; - buck8_reg: buck8 { - regulator-compatible = "BUCK8"; + buck8_reg: BUCK8 { regulator-name = "VMEM_VDDF_3.0V"; regulator-min-microvolt = <2850000>; regulator-max-microvolt = <2850000>; maxim,ena-gpios = <&gpk0 2 GPIO_ACTIVE_HIGH>; }; - buck9_reg: buck9 { - regulator-compatible = "BUCK9"; + buck9_reg: BUCK9 { regulator-name = "CAM_ISP_CORE_1.2V"; regulator-min-microvolt = <1000000>; regulator-max-microvolt = <1200000>;
The regulator-compatible property from the regulator DT binding was deprecated and the correct approach is to use the node's name. This patch has no functional changes but by not using a deprecated property, new DTS based on this one will not carry the same issue. Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> --- arch/arm/boot/dts/exynos4412-trats2.dts | 105 +++++++++++--------------------- 1 file changed, 35 insertions(+), 70 deletions(-)