On Tue, Oct 22, 2013 at 07:07:22PM +0200, Rostislav Lisovy wrote: > Enable UART1 and FEC You have more than these two enabled. > > Signed-off-by: Rostislav Lisovy <lisovy@xxxxxxxxx> > > create mode 100644 arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi > > diff --git a/arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi b/arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi > new file mode 100644 > index 0000000..04695c9 > --- /dev/null > +++ b/arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi > @@ -0,0 +1,195 @@ > +/* > + * Copyright 2013 Rostislav Lisovy <lisovy@xxxxxxxxx>, PiKRON s.r.o. > + * > + * The code contained herein is licensed under the GNU General Public > + * License. You may obtain a copy of the GNU General Public License > + * Version 2 or later at the following locations: > + * > + * http://www.opensource.org/licenses/gpl-license.html > + * http://www.gnu.org/copyleft/gpl.html > + */ > + > +#include "imx53.dtsi" > + > +/ { > + model = "Voipac i.MX53 X53-DMM-668"; > + compatible = "vp,imx53-dmm-668", "fsl,imx53"; > + > + memory@0 { > + device_type = "memory"; > + reg = <0x70000000 0x20000000>; > + }; > + > + memory@1 { > + device_type = "memory"; > + reg = <0xb0000000 0x20000000>; > + }; The 'num' in memory@num should match the first number in 'reg' property, so they should looks like: memory@70000000 { device_type = "memory"; reg = <0x70000000 0x20000000>; } memory@b0000000 { device_type = "memory"; reg = <0xb0000000 0x20000000>; } > + > + regulators { > + compatible = "simple-bus"; > + > + reg_3p3v: 3p3v { > + compatible = "regulator-fixed"; > + regulator-name = "3P3V"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-always-on; > + }; > + }; > +}; > + > +&iomuxc { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_hog>; > + > + hog { > + pinctrl_hog: hoggrp { > + fsl,pins = < > + /* Make DA9053 regulator functional */ > + MX53_PAD_GPIO_16__GPIO7_11 0x80000000 > + Drop these blank lines. We generally only have blank lines between nodes. > + /* FEC Power enable */ > + MX53_PAD_GPIO_11__GPIO4_1 0x80000000 > + > + /* FEC RST */ > + MX53_PAD_GPIO_12__GPIO4_2 0x80000000 > + >; > + }; > + }; > +}; > + > +&uart1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_uart1_2>; > + status = "okay"; > +}; > + > +&i2c1 { Please sort these nodes alphabetically in label name, so that new node can find they slots easily. Shawn > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_i2c1_2>; > + status = "okay"; > + > + pmic: dialog@48 { > + compatible = "dlg,da9053-aa", "dlg,da9052"; > + reg = <0x48>; > + interrupt-parent = <&gpio7>; > + interrupts = <11 0x8>; /* low-level active IRQ at GPIO7_11 */ > + > + regulators { > + buck1_reg: buck1 { > + regulator-name = "BUCKCORE"; > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1400000>; > + regulator-always-on; > + }; > + > + buck2_reg: buck2 { > + regulator-name = "BUCKPRO"; > + regulator-min-microvolt = <900000>; > + regulator-max-microvolt = <1350000>; > + regulator-always-on; > + }; > + > + buck3_reg: buck3 { > + regulator-name = "BUCKMEM"; > + regulator-min-microvolt = <1420000>; > + regulator-max-microvolt = <1580000>; > + regulator-always-on; > + }; > + > + buck4_reg: buck4 { > + regulator-name = "BUCKPERI"; > + regulator-min-microvolt = <2370000>; > + regulator-max-microvolt = <2630000>; > + regulator-always-on; > + }; > + > + ldo1_reg: ldo1 { > + regulator-name = "ldo1_1v3"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1350000>; > + regulator-boot-on; > + regulator-always-on; > + }; > + > + ldo2_reg: ldo2 { > + regulator-name = "ldo2_1v3"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1350000>; > + regulator-always-on; > + }; > + > + ldo3_reg: ldo3 { > + regulator-name = "ldo3_3v3"; > + regulator-min-microvolt = <3250000>; > + regulator-max-microvolt = <3350000>; > + regulator-always-on; > + }; > + > + ldo4_reg: ldo4 { > + regulator-name = "ldo4_2v775"; > + regulator-min-microvolt = <2770000>; > + regulator-max-microvolt = <2780000>; > + regulator-always-on; > + }; > + > + ldo5_reg: ldo5 { > + regulator-name = "ldo5_3v3"; > + regulator-min-microvolt = <3250000>; > + regulator-max-microvolt = <3350000>; > + regulator-always-on; > + }; > + > + ldo6_reg: ldo6 { > + regulator-name = "ldo6_1v3"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1350000>; > + regulator-always-on; > + }; > + > + ldo7_reg: ldo7 { > + regulator-name = "ldo7_2v75"; > + regulator-min-microvolt = <2700000>; > + regulator-max-microvolt = <2800000>; > + regulator-always-on; > + }; > + > + ldo8_reg: ldo8 { > + regulator-name = "ldo8_1v8"; > + regulator-min-microvolt = <1750000>; > + regulator-max-microvolt = <1850000>; > + regulator-always-on; > + }; > + > + ldo9_reg: ldo9 { > + regulator-name = "ldo9_1v5"; > + regulator-min-microvolt = <1450000>; > + regulator-max-microvolt = <1550000>; > + regulator-always-on; > + }; > + > + ldo10_reg: ldo10 { > + regulator-name = "ldo10_1v3"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1350000>; > + regulator-always-on; > + }; > + }; > + }; > +}; > + > +&ecspi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_ecspi1_1>; > + fsl,spi-num-chipselects = <4>; > + cs-gpios = <&gpio2 30 0>, <&gpio3 19 0>, <&gpio2 16 0>, <&gpio2 17 0>; > + status = "okay"; > +}; > + > +&fec { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_fec_1>; > + phy-mode = "rmii"; > + phy-reset-gpios = <&gpio4 2 0>; > + status = "okay"; > +}; > -- > 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