On Tue, Nov 12, 2013 at 12:24:18PM +0100, Denis Carikli wrote:
> The following devices/functionalities were added:
>  * Main and secondary UARTs.
>  * i2c and the pcf8563 device.
>  * Ethernet.
>  * NAND.
>  * The BP1 button.
>  * The LED.
>  * Watchdog
>  * SD.
> 
> Cc: Rob Herring <[email protected]>
> Cc: Pawel Moll <[email protected]>
> Cc: Mark Rutland <[email protected]>
> Cc: Stephen Warren <[email protected]>
> Cc: Ian Campbell <[email protected]>
> Cc: Grant Likely <[email protected]>
> Cc: [email protected]
> Cc: Sascha Hauer <[email protected]>
> Cc: Shawn Guo <[email protected]>
> Cc: [email protected]
> Cc: Eric Bénard <[email protected]>
> Signed-off-by: Denis Carikli <[email protected]>
> Acked-by: Sascha Hauer <[email protected]>
> ---
> ChangeLog v7->v8:
> - The commit message was improved.
> - The board specific iomuxc pins group configuration that were in imx35.dtsi 
> were moved here.
> - The rest of the patch was updated accordingly.
> 
> ChangeLog v6->v7:
> - Added Grant Likely in the Cc list.
> - Shortened the license.
> - The tsc2007 pads were moved in another patch.
> 
> ChangeLog v5->v6:
> - Shawn Guo was added in the Cc.
> ---
>  arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi       |   58 +++++++++
>  .../boot/dts/imx35-eukrea-mbimxsd35-baseboard.dts  |  124 
> ++++++++++++++++++++
>  2 files changed, 182 insertions(+)
>  create mode 100644 arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi
>  create mode 100644 arch/arm/boot/dts/imx35-eukrea-mbimxsd35-baseboard.dts

Please add the dtb target into arch/arm/boot/dts/Makefile.

> 
> diff --git a/arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi 
> b/arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi
> new file mode 100644
> index 0000000..3260678
> --- /dev/null
> +++ b/arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi
> @@ -0,0 +1,58 @@
> +/*
> + * Copyright 2013 Eukréa Electromatique <[email protected]>
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License
> + * as published by the Free Software Foundation; either version 2
> + * of the License, or (at your option) any later version.
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + */
> +
> +#include "imx35.dtsi"
> +
> +/ {
> +     model = "Eukrea CPUIMX35";
> +     compatible = "eukrea,cpuimx35", "fsl,imx35";
> +
> +     memory {
> +             reg = <0x80000000 0x8000000>; /* 128M */
> +     };
> +};
> +
> +&fec {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_fec>;
> +     status = "okay";
> +};
> +
> +&i2c1 {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_i2c1>;
> +     status = "okay";

Nit: have a blank line here.

> +     pcf8563@51 {
> +             compatible = "nxp,pcf8563";
> +             reg = <0x51>;
> +     };
> +};
> +
> +&iomuxc {
> +     imx35-eukrea {
> +             pinctrl_fec: fecgrp {
> +                     fsl,pins = <MX35_FEC_PINGRP1>;
> +             };
> +
> +             pinctrl_i2c1: i2c1grp {
> +                     fsl,pins = <MX35_I2C1_PINGRP1>;
> +             };
> +     };
> +};
> +
> +&nfc {
> +     nand-bus-width = <8>;
> +     nand-ecc-mode = "hw";
> +     nand-on-flash-bbt;
> +     status = "okay";
> +};
> diff --git a/arch/arm/boot/dts/imx35-eukrea-mbimxsd35-baseboard.dts 
> b/arch/arm/boot/dts/imx35-eukrea-mbimxsd35-baseboard.dts
> new file mode 100644
> index 0000000..fe0f3fa
> --- /dev/null
> +++ b/arch/arm/boot/dts/imx35-eukrea-mbimxsd35-baseboard.dts
> @@ -0,0 +1,124 @@
> +/*
> + * Copyright 2013 Eukréa Electromatique <[email protected]>
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License
> + * as published by the Free Software Foundation; either version 2
> + * of the License, or (at your option) any later version.
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + */
> +
> +/dts-v1/;
> +#include "imx35-eukrea-cpuimx35.dtsi"
> +
> +/ {
> +     model = "Eukrea CPUIMX35";
> +     compatible = "eukrea,mbimxsd35-baseboard", "eukrea,cpuimx35", 
> "fsl,imx35";
> +
> +     gpio_keys {
> +             compatible = "gpio-keys";
> +
> +             bp1 {
> +                     label = "BP1";
> +                     gpios = <&gpio3 25 1>;
> +                     linux,code = <256>;
> +                     gpio-key,wakeup;
> +                     linux,input-type = <1>;
> +             };
> +     };
> +
> +     leds {
> +             compatible = "gpio-leds";
> +
> +             led1 {
> +                     label = "led1";
> +                     gpios = <&gpio3 29 1>;
> +                     linux,default-trigger = "heartbeat";
> +             };
> +     };
> +};
> +
> +&audmux {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_audmux>;
> +     status = "okay";
> +};
> +
> +&esdhc1 {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_esdhc1>;
> +     cd-gpios = <&gpio3 24 0>;
> +     status = "okay";
> +};
> +
> +&i2c1 {
> +     tlv320aic23: codec@1a {
> +             compatible = "ti,tlv320aic23";
> +             reg = <0x1a>;
> +     };
> +};
> +
> +&iomuxc {
> +        pinctrl-names = "default";
> +        pinctrl-0 = <&pinctrl_hog>;

Use tab instead of spaces.

> +
> +     imx35-eukrea {
> +             pinctrl_hog: hoggrp {
> +                     fsl,pins = <
> +                             MX35_PAD_LD18__GPIO3_24  0x80000000
> +                             MX35_PAD_LD19__GPIO3_25  0x80000000
> +                             MX35_PAD_LD23__GPIO3_29  0x80000000
> +                     >;
> +             };
> +
> +             pinctrl_audmux: audmuxgrp {
> +                     fsl,pins = <MX35_AUDMUX_PINGRP1>;
> +             };
> +
> +             pinctrl_esdhc1: esdhc1grp {
> +                     fsl,pins = <MX35_ESDHC1_PINGRP1>;
> +             };
> +
> +             pinctrl_fec: fecgrp {
> +                     fsl,pins = <MX35_FEC_PINGRP1>;
> +             };
> +
> +             pinctrl_i2c1: i2c1grp {
> +                     fsl,pins = <MX35_I2C1_PINGRP1>;
> +             };
> +
> +             pinctrl_reg_lcd_3v3: reg_lcd_3v3 {

'_' is used in label while '-' should be used in node name.

Shawn

> +                     fsl,pins = <MX35_PAD_D3_CLS__GPIO1_4 0x80000000>;
> +             };
> +
> +             pinctrl_uart1: uart1grp {
> +                     fsl,pins = <MX35_UART1_PINGRP1>;
> +             };
> +
> +             pinctrl_uart2: uart2grp {
> +                     fsl,pins = <MX35_UART2_PINGRP1>;
> +             };
> +     };
> +};
> +
> +&ssi1 {
> +     fsl,mode = "i2s-slave";
> +     status = "okay";
> +};
> +
> +&uart1 {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_uart1>;
> +     fsl,uart-has-rtscts;
> +     status = "okay";
> +};
> +
> +&uart2 {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_uart2>;
> +     fsl,uart-has-rtscts;
> +     status = "okay";
> +};
> -- 
> 1.7.9.5
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to