On Tue, Nov 05, 2013 at 01:49:20PM +0100, Rostislav Lisovy wrote: > Signed-off-by: Rostislav Lisovy <lisovy@xxxxxxxxx> > --- > arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi b/arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi > index a7d03ad..4f1d8fa 100644 > --- a/arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi > +++ b/arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi > @@ -27,9 +27,12 @@ > > regulators { > compatible = "simple-bus"; > + #address-cells = <1>; > + #size-cells = <0>; > > - reg_3p3v: 3p3v { > + reg_3p3v: regulator@0 { Yeah, instead of using arbitrary node name, the generic regulator@num is more preferable. I will make this change to the rest of IMX dts files. Applied, thanks. Shawn > compatible = "regulator-fixed"; > + reg = <0>; > regulator-name = "3P3V"; > regulator-min-microvolt = <3300000>; > regulator-max-microvolt = <3300000>; > -- > 1.7.10.4 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html