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 <lis...@gmail.com>
> 
>  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 <lis...@gmail.com>, 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 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/

Reply via email to