[PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA

[PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA

[Enric Balletbo i Serra]

The IGEP COM AQUILA is industrial processors SODIMM module with
following highlights:

   o AM3352/AM3354/AM3358/AM3359 Texas Instruments processor
   o Cortex-A8 ARM CPU
   o 3.3 volts Inputs / Outputs use industrial
   o 256 MB DDR3 SDRAM / 128 Megabytes FLASH
   o MicroSD card reader on-board
   o Ethernet controller on-board
   o JTAG debug connector available
   o Designed for industrial range purposes

Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
---
 arch/arm/boot/dts/am335x-igep0033.dtsi | 269 +++++++++++++++++++++++++++++++++
 1 file changed, 269 insertions(+)
 create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi

diff --git a/arch/arm/boot/dts/am335x-igep0033.dtsi b/arch/arm/boot/dts/am335x-igep0033.dtsi
new file mode 100644
index 0000000..1d70141
--- /dev/null
+++ b/arch/arm/boot/dts/am335x-igep0033.dtsi
@@ -0,0 +1,269 @@
+/*
+ * Copyright (C) 2013 ISEE 2007 SL - http://www.isee.biz/
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+/dts-v1/;
+
+#include "am33xx.dtsi"
+
+/ {
+	cpus {
+		cpu@0 {
+			cpu0-supply = <&vdd1_reg>;
+		};
+	};
+
+	memory {
+		device_type = "memory";
+		reg = <0x80000000 0x10000000>; /* 256 MB */
+	};
+
+	am33xx_pinmux: pinmux@44e10800 {
+		pinctrl-names = "default";
+
+		i2c0_pins: pinmux_i2c0_pins {
+			pinctrl-single,pins = <
+				0x188 (PIN_INPUT_PULLUP | MUX_MODE0)	/* i2c0_sda.i2c0_sda */
+				0x18c (PIN_INPUT_PULLUP | MUX_MODE0)	/* i2c0_scl.i2c0_scl */
+			>;
+		};
+
+		nandflash_pins: pinmux_nandflash_pins {
+			pinctrl-single,pins = <
+				0x0 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad0.gpmc_ad0 */
+				0x4 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad1.gpmc_ad1 */
+				0x8 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad2.gpmc_ad2 */
+				0xc (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad3.gpmc_ad3 */
+				0x10 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad4.gpmc_ad4 */
+				0x14 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad5.gpmc_ad5 */
+				0x18 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad6.gpmc_ad6 */
+				0x1c (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad7.gpmc_ad7 */
+				0x70 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_wait0.gpmc_wait0 */
+				0x74 (PIN_INPUT_PULLUP | MUX_MODE7)	/* gpmc_wpn.gpio0_30 */
+				0x7c (PIN_OUTPUT | MUX_MODE0)		/* gpmc_csn0.gpmc_csn0  */
+				0x90 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_advn_ale.gpmc_advn_ale */
+				0x94 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_oen_ren.gpmc_oen_ren */
+				0x98 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_wen.gpmc_wen */
+				0x9c (PIN_OUTPUT | MUX_MODE0)		/* gpmc_be0n_cle.gpmc_be0n_cle */
+			>;
+		};
+
+		uart0_pins: pinmux_uart0_pins {
+			pinctrl-single,pins = <
+				0x170 (PIN_INPUT_PULLUP | MUX_MODE0)	/* uart0_rxd.uart0_rxd */
+				0x174 (PIN_OUTPUT_PULLDOWN | MUX_MODE0)	/* uart0_txd.uart0_txd */
+			>;
+		};
+
+		user_leds: pinmux_user_leds_pins {
+			pinctrl-single,pins = <
+				0x5c (PIN_OUTPUT_PULLDOWN | MUX_MODE7)	/* gpmc_a7.gpio1_23 */
+			>;
+		};
+ 
+	};
+
+	ocp {
+		uart0: serial@44e09000 {
+			pinctrl-names = "default";
+			pinctrl-0 = <&uart0_pins>;
+			status = "okay";
+		};
+
+		i2c0: i2c@44e0b000 {
+			pinctrl-names = "default";
+			pinctrl-0 = <&i2c0_pins>;
+
+			status = "okay";
+			clock-frequency = <400000>;
+
+			tps: tps@2d {
+				reg = <0x2d>;
+			};
+		};
+
+		elm: elm@48080000 {
+			status = "okay";
+		};
+
+		gpmc: gpmc@50000000 {
+			pinctrl-names = "default";
+			pinctrl-0 = <&nandflash_pins>;
+
+			status = "okay";
+			ranges = <0 0 0x08000000 0x10000000>;	/* CS0: NAND */
+
+			nand@0,0 {
+				reg = <0 0 0>; /* CS0, offset 0 */
+				nand-bus-width = <8>;
+				ti,nand-ecc-opt = "bch8";
+				gpmc,device-nand = "true";
+				gpmc,device-width = <1>;
+				gpmc,sync-clk-ps = <0>;
+				gpmc,cs-on-ns = <0>;
+				gpmc,cs-rd-off-ns = <44>;
+				gpmc,cs-wr-off-ns = <44>;
+				gpmc,adv-on-ns = <6>;
+				gpmc,adv-rd-off-ns = <34>;
+				gpmc,adv-wr-off-ns = <44>;
+				gpmc,we-on-ns = <0>;
+				gpmc,we-off-ns = <40>;
+				gpmc,oe-on-ns = <0>;
+				gpmc,oe-off-ns = <54>;
+				gpmc,access-ns = <64>;
+				gpmc,rd-cycle-ns = <82>;
+				gpmc,wr-cycle-ns = <82>;
+				gpmc,wait-on-read = "true";
+				gpmc,wait-on-write = "true";
+				gpmc,bus-turnaround-ns = <0>;
+				gpmc,cycle2cycle-delay-ns = <0>;
+				gpmc,clk-activation-ns = <0>;
+				gpmc,wait-monitoring-ns = <0>;
+				gpmc,wr-access-ns = <40>;
+				gpmc,wr-data-mux-bus-ns = <0>;
+
+				#address-cells = <1>;
+				#size-cells = <1>;
+				elm_id = <&elm>;
+
+				/* MTD partition table */
+				partition@0 {
+					label = "SPL";
+					reg = <0x00000000 0x000080000>;
+				};
+
+				partition@1 {
+					label = "U-boot";
+					reg = <0x00080000 0x001e0000>;
+				};
+
+				partition@2 {
+					label = "U-Boot Env";
+					reg = <0x00260000 0x00020000>;
+				};
+
+				partition@3 {
+					label = "Kernel";
+					reg = <0x00280000 0x00500000>;
+				};
+
+				partition@4 {
+					label = "File System";
+					reg = <0x00780000 0x0F880000>;
+				};
+			};
+		};
+	};
+
+	leds {
+		pinctrl-names = "default";
+		pinctrl-0 = <&user_leds>;
+
+		compatible = "gpio-leds";
+
+		led@0 {
+			label = "com:green:user";
+			gpios = <&gpio1 23 GPIO_ACTIVE_HIGH>;
+			default-state = "on";
+		};
+	};
+
+
+	vbat: fixedregulator@0 {
+		compatible = "regulator-fixed";
+		regulator-name = "vbat";
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+		regulator-boot-on;
+	};
+};
+
+#include "tps65910.dtsi"
+
+&tps {
+	vcc1-supply = <&vbat>;
+	vcc2-supply = <&vbat>;
+	vcc3-supply = <&vbat>;
+	vcc4-supply = <&vbat>;
+	vcc5-supply = <&vbat>;
+	vcc6-supply = <&vbat>;
+	vcc7-supply = <&vbat>;
+	vccio-supply = <&vbat>;
+
+	regulators {
+		vrtc_reg: regulator@0 {
+			regulator-always-on;
+		};
+
+		vio_reg: regulator@1 {
+			regulator-always-on;
+		};
+
+		vdd1_reg: regulator@2 {
+			/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
+			regulator-name = "vdd_mpu";
+			regulator-min-microvolt = <912500>;
+			regulator-max-microvolt = <1312500>;
+			regulator-boot-on;
+			regulator-always-on;
+		};
+
+		vdd2_reg: regulator@3 {
+			/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
+			regulator-name = "vdd_core";
+			regulator-min-microvolt = <912500>;
+			regulator-max-microvolt = <1150000>;
+			regulator-boot-on;
+			regulator-always-on;
+		};
+
+		vdd3_reg: regulator@4 {
+			regulator-always-on;
+		};
+
+		vdig1_reg: regulator@5 {
+			regulator-always-on;
+		};
+
+		vdig2_reg: regulator@6 {
+			regulator-always-on;
+		};
+
+		vpll_reg: regulator@7 {
+			regulator-always-on;
+		};
+
+		vdac_reg: regulator@8 {
+			regulator-always-on;
+		};
+
+		vaux1_reg: regulator@9 {
+			regulator-always-on;
+		};
+
+		vaux2_reg: regulator@10 {
+			regulator-always-on;
+		};
+
+		vaux33_reg: regulator@11 {
+			regulator-always-on;
+		};
+
+		vmmc_reg: regulator@12 {
+			regulator-always-on;
+		};
+	};
+};
+
+&cpsw_emac0 {
+	phy_id = <&davinci_mdio>, <0>;
+};
+
+&cpsw_emac1 {
+	phy_id = <&davinci_mdio>, <1>;
+};
+
-- 
1.8.1.2

Re: [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA

[Benoit Cousson]

Hi Enric,

On 06/19/2013 03:27 AM, Enric Balletbo i Serra wrote:
> The IGEP COM AQUILA is industrial processors SODIMM module with
> following highlights:
>
>     o AM3352/AM3354/AM3358/AM3359 Texas Instruments processor
>     o Cortex-A8 ARM CPU
>     o 3.3 volts Inputs / Outputs use industrial
>     o 256 MB DDR3 SDRAM / 128 Megabytes FLASH
>     o MicroSD card reader on-board
>     o Ethernet controller on-board
>     o JTAG debug connector available
>     o Designed for industrial range purposes
>
> Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
> ---
>   arch/arm/boot/dts/am335x-igep0033.dtsi | 269 +++++++++++++++++++++++++++++++++
>   1 file changed, 269 insertions(+)
>   create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi
>
> diff --git a/arch/arm/boot/dts/am335x-igep0033.dtsi b/arch/arm/boot/dts/am335x-igep0033.dtsi
> new file mode 100644
> index 0000000..1d70141
> --- /dev/null
> +++ b/arch/arm/boot/dts/am335x-igep0033.dtsi
> @@ -0,0 +1,269 @@
> +/*
> + * Copyright (C) 2013 ISEE 2007 SL - http://www.isee.biz/
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +/dts-v1/;
> +
> +#include "am33xx.dtsi"
> +
> +/ {
> +	cpus {
> +		cpu@0 {
> +			cpu0-supply = <&vdd1_reg>;
> +		};
> +	};
> +
> +	memory {
> +		device_type = "memory";
> +		reg = <0x80000000 0x10000000>; /* 256 MB */
> +	};
> +
> +	am33xx_pinmux: pinmux@44e10800 {

That node should be inside the ocp one since the control module is a 
regular IP connected to the OCP interconnect.


> +		pinctrl-names = "default";
> +
> +		i2c0_pins: pinmux_i2c0_pins {
> +			pinctrl-single,pins = <
> +				0x188 (PIN_INPUT_PULLUP | MUX_MODE0)	/* i2c0_sda.i2c0_sda */
> +				0x18c (PIN_INPUT_PULLUP | MUX_MODE0)	/* i2c0_scl.i2c0_scl */
> +			>;
> +		};
> +
> +		nandflash_pins: pinmux_nandflash_pins {
> +			pinctrl-single,pins = <
> +				0x0 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad0.gpmc_ad0 */
> +				0x4 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad1.gpmc_ad1 */
> +				0x8 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad2.gpmc_ad2 */
> +				0xc (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad3.gpmc_ad3 */
> +				0x10 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad4.gpmc_ad4 */
> +				0x14 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad5.gpmc_ad5 */
> +				0x18 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad6.gpmc_ad6 */
> +				0x1c (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad7.gpmc_ad7 */
> +				0x70 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_wait0.gpmc_wait0 */
> +				0x74 (PIN_INPUT_PULLUP | MUX_MODE7)	/* gpmc_wpn.gpio0_30 */
> +				0x7c (PIN_OUTPUT | MUX_MODE0)		/* gpmc_csn0.gpmc_csn0  */
> +				0x90 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_advn_ale.gpmc_advn_ale */
> +				0x94 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_oen_ren.gpmc_oen_ren */
> +				0x98 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_wen.gpmc_wen */
> +				0x9c (PIN_OUTPUT | MUX_MODE0)		/* gpmc_be0n_cle.gpmc_be0n_cle */
> +			>;
> +		};
> +
> +		uart0_pins: pinmux_uart0_pins {
> +			pinctrl-single,pins = <
> +				0x170 (PIN_INPUT_PULLUP | MUX_MODE0)	/* uart0_rxd.uart0_rxd */
> +				0x174 (PIN_OUTPUT_PULLDOWN | MUX_MODE0)	/* uart0_txd.uart0_txd */
> +			>;
> +		};
> +
> +		user_leds: pinmux_user_leds_pins {
> +			pinctrl-single,pins = <
> +				0x5c (PIN_OUTPUT_PULLDOWN | MUX_MODE7)	/* gpmc_a7.gpio1_23 */
> +			>;
> +		};
> +
> +	};
> +
> +	ocp {
> +		uart0: serial@44e09000 {
> +			pinctrl-names = "default";
> +			pinctrl-0 = <&uart0_pins>;
> +			status = "okay";
> +		};
> +
> +		i2c0: i2c@44e0b000 {
> +			pinctrl-names = "default";
> +			pinctrl-0 = <&i2c0_pins>;
> +
> +			status = "okay";
> +			clock-frequency = <400000>;
> +
> +			tps: tps@2d {
> +				reg = <0x2d>;
> +			};
> +		};
> +
> +		elm: elm@48080000 {
> +			status = "okay";
> +		};
> +
> +		gpmc: gpmc@50000000 {
> +			pinctrl-names = "default";
> +			pinctrl-0 = <&nandflash_pins>;
> +
> +			status = "okay";
> +			ranges = <0 0 0x08000000 0x10000000>;	/* CS0: NAND */
> +
> +			nand@0,0 {
> +				reg = <0 0 0>; /* CS0, offset 0 */
> +				nand-bus-width = <8>;
> +				ti,nand-ecc-opt = "bch8";
> +				gpmc,device-nand = "true";
> +				gpmc,device-width = <1>;
> +				gpmc,sync-clk-ps = <0>;
> +				gpmc,cs-on-ns = <0>;
> +				gpmc,cs-rd-off-ns = <44>;
> +				gpmc,cs-wr-off-ns = <44>;
> +				gpmc,adv-on-ns = <6>;
> +				gpmc,adv-rd-off-ns = <34>;
> +				gpmc,adv-wr-off-ns = <44>;
> +				gpmc,we-on-ns = <0>;
> +				gpmc,we-off-ns = <40>;
> +				gpmc,oe-on-ns = <0>;
> +				gpmc,oe-off-ns = <54>;
> +				gpmc,access-ns = <64>;
> +				gpmc,rd-cycle-ns = <82>;
> +				gpmc,wr-cycle-ns = <82>;
> +				gpmc,wait-on-read = "true";
> +				gpmc,wait-on-write = "true";
> +				gpmc,bus-turnaround-ns = <0>;
> +				gpmc,cycle2cycle-delay-ns = <0>;
> +				gpmc,clk-activation-ns = <0>;
> +				gpmc,wait-monitoring-ns = <0>;
> +				gpmc,wr-access-ns = <40>;
> +				gpmc,wr-data-mux-bus-ns = <0>;
> +
> +				#address-cells = <1>;
> +				#size-cells = <1>;
> +				elm_id = <&elm>;
> +
> +				/* MTD partition table */
> +				partition@0 {
> +					label = "SPL";
> +					reg = <0x00000000 0x000080000>;
> +				};
> +
> +				partition@1 {
> +					label = "U-boot";
> +					reg = <0x00080000 0x001e0000>;
> +				};
> +
> +				partition@2 {
> +					label = "U-Boot Env";
> +					reg = <0x00260000 0x00020000>;
> +				};
> +
> +				partition@3 {
> +					label = "Kernel";
> +					reg = <0x00280000 0x00500000>;
> +				};
> +
> +				partition@4 {
> +					label = "File System";
> +					reg = <0x00780000 0x0F880000>;
> +				};
> +			};
> +		};
> +	};
> +
> +	leds {
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&user_leds>;
> +
> +		compatible = "gpio-leds";
> +
> +		led@0 {
> +			label = "com:green:user";
> +			gpios = <&gpio1 23 GPIO_ACTIVE_HIGH>;
> +			default-state = "on";
> +		};
> +	};
> +
> +
> +	vbat: fixedregulator@0 {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vbat";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		regulator-boot-on;
> +	};
> +};
> +
> +#include "tps65910.dtsi"
> +
> +&tps {
> +	vcc1-supply = <&vbat>;
> +	vcc2-supply = <&vbat>;
> +	vcc3-supply = <&vbat>;
> +	vcc4-supply = <&vbat>;
> +	vcc5-supply = <&vbat>;
> +	vcc6-supply = <&vbat>;
> +	vcc7-supply = <&vbat>;
> +	vccio-supply = <&vbat>;
> +
> +	regulators {
> +		vrtc_reg: regulator@0 {
> +			regulator-always-on;
> +		};
> +
> +		vio_reg: regulator@1 {
> +			regulator-always-on;
> +		};
> +
> +		vdd1_reg: regulator@2 {
> +			/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
> +			regulator-name = "vdd_mpu";
> +			regulator-min-microvolt = <912500>;
> +			regulator-max-microvolt = <1312500>;
> +			regulator-boot-on;
> +			regulator-always-on;
> +		};
> +
> +		vdd2_reg: regulator@3 {
> +			/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
> +			regulator-name = "vdd_core";
> +			regulator-min-microvolt = <912500>;
> +			regulator-max-microvolt = <1150000>;
> +			regulator-boot-on;
> +			regulator-always-on;
> +		};
> +
> +		vdd3_reg: regulator@4 {
> +			regulator-always-on;
> +		};
> +
> +		vdig1_reg: regulator@5 {
> +			regulator-always-on;
> +		};
> +
> +		vdig2_reg: regulator@6 {
> +			regulator-always-on;
> +		};
> +
> +		vpll_reg: regulator@7 {
> +			regulator-always-on;
> +		};
> +
> +		vdac_reg: regulator@8 {
> +			regulator-always-on;
> +		};
> +
> +		vaux1_reg: regulator@9 {
> +			regulator-always-on;
> +		};
> +
> +		vaux2_reg: regulator@10 {
> +			regulator-always-on;
> +		};
> +
> +		vaux33_reg: regulator@11 {
> +			regulator-always-on;
> +		};
> +
> +		vmmc_reg: regulator@12 {
> +			regulator-always-on;
> +		};
> +	};
> +};
> +
> +&cpsw_emac0 {
> +	phy_id = <&davinci_mdio>, <0>;
> +};
> +
> +&cpsw_emac1 {
> +	phy_id = <&davinci_mdio>, <1>;
> +};
> +
>


Otherwise, it looks good.

Thanks,
Benoit

Re: [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA

[Javier Martinez Canillas]

Hi,

On Wed, Jun 19, 2013 at 12:46 PM, Benoit Cousson <b-cousson@ti.com> wrote:
> Hi Enric,
>
>
> On 06/19/2013 03:27 AM, Enric Balletbo i Serra wrote:
>>
>> The IGEP COM AQUILA is industrial processors SODIMM module with
>> following highlights:
>>
>>     o AM3352/AM3354/AM3358/AM3359 Texas Instruments processor
>>     o Cortex-A8 ARM CPU
>>     o 3.3 volts Inputs / Outputs use industrial
>>     o 256 MB DDR3 SDRAM / 128 Megabytes FLASH
>>     o MicroSD card reader on-board
>>     o Ethernet controller on-board
>>     o JTAG debug connector available
>>     o Designed for industrial range purposes
>>
>> Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
>> ---
>>   arch/arm/boot/dts/am335x-igep0033.dtsi | 269
>> +++++++++++++++++++++++++++++++++
>>   1 file changed, 269 insertions(+)
>>   create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi
>>
>> diff --git a/arch/arm/boot/dts/am335x-igep0033.dtsi
>> b/arch/arm/boot/dts/am335x-igep0033.dtsi
>> new file mode 100644
>> index 0000000..1d70141
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/am335x-igep0033.dtsi
>> @@ -0,0 +1,269 @@
>> +/*
>> + * Copyright (C) 2013 ISEE 2007 SL - http://www.isee.biz/
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License version 2 as
>> + * published by the Free Software Foundation.
>> + */
>> +
>> +/dts-v1/;
>> +
>> +#include "am33xx.dtsi"
>> +
>> +/ {
>> +       cpus {
>> +               cpu@0 {
>> +                       cpu0-supply = <&vdd1_reg>;
>> +               };
>> +       };
>> +
>> +       memory {
>> +               device_type = "memory";
>> +               reg = <0x80000000 0x10000000>; /* 256 MB */
>> +       };
>> +
>> +       am33xx_pinmux: pinmux@44e10800 {
>
>
> That node should be inside the ocp one since the control module is a regular
> IP connected to the OCP interconnect.
>
>
>

In fact, I don't think that there should be a definition of the On
Chip Peripherals interconnect or any child node of the ocp in a DTS
file.
These should be defined in the included dtsi file since it will be
common to all boards based on this SoC.

DTS files that describe a board can reference these nodes defined in
the dtsi and add their custom peripherals as childs of them.

So, instead defining in the DTS:

am33xx_pinmux: pinmux@44e10800 {
...
};

gpmc: gpmc@50000000 {
...
};

i2c0: i2c@44e0b000 {
...
};

uart0: serial@44e09000 {
..
};

It has to be defined as:

&am33xx_pinmux {
...
};

&gpmc {
...
};

&i2c0 {
...
}

I'm looking at other am33xx dts such as am335x-bone.dts and
am335x-evm.dts and I see that these define device nodes already
defined in the included .dtsi which is wrong in my opinion.

The OMAP{3,4,5} dtsi and dts do correctly and can be used as a
reference on how the device nodes have to be defined and referenced.

Best regards,
Javier

Re: [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA

[Cousson, Benoit]

Hi Javier,

On 6/19/2013 8:32 AM, Javier Martinez Canillas wrote:
> Hi,
>
> On Wed, Jun 19, 2013 at 12:46 PM, Benoit Cousson <b-cousson@ti.com> wrote:
>> Hi Enric,
>>
>>
>> On 06/19/2013 03:27 AM, Enric Balletbo i Serra wrote:
>>>
>>> The IGEP COM AQUILA is industrial processors SODIMM module with
>>> following highlights:
>>>
>>>      o AM3352/AM3354/AM3358/AM3359 Texas Instruments processor
>>>      o Cortex-A8 ARM CPU
>>>      o 3.3 volts Inputs / Outputs use industrial
>>>      o 256 MB DDR3 SDRAM / 128 Megabytes FLASH
>>>      o MicroSD card reader on-board
>>>      o Ethernet controller on-board
>>>      o JTAG debug connector available
>>>      o Designed for industrial range purposes
>>>
>>> Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
>>> ---
>>>    arch/arm/boot/dts/am335x-igep0033.dtsi | 269
>>> +++++++++++++++++++++++++++++++++
>>>    1 file changed, 269 insertions(+)
>>>    create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi
>>>
>>> diff --git a/arch/arm/boot/dts/am335x-igep0033.dtsi
>>> b/arch/arm/boot/dts/am335x-igep0033.dtsi
>>> new file mode 100644
>>> index 0000000..1d70141
>>> --- /dev/null
>>> +++ b/arch/arm/boot/dts/am335x-igep0033.dtsi
>>> @@ -0,0 +1,269 @@
>>> +/*
>>> + * Copyright (C) 2013 ISEE 2007 SL - http://www.isee.biz/
>>> + *
>>> + * This program is free software; you can redistribute it and/or modify
>>> + * it under the terms of the GNU General Public License version 2 as
>>> + * published by the Free Software Foundation.
>>> + */
>>> +
>>> +/dts-v1/;
>>> +
>>> +#include "am33xx.dtsi"
>>> +
>>> +/ {
>>> +       cpus {
>>> +               cpu@0 {
>>> +                       cpu0-supply = <&vdd1_reg>;
>>> +               };
>>> +       };
>>> +
>>> +       memory {
>>> +               device_type = "memory";
>>> +               reg = <0x80000000 0x10000000>; /* 256 MB */
>>> +       };
>>> +
>>> +       am33xx_pinmux: pinmux@44e10800 {
>>
>>
>> That node should be inside the ocp one since the control module is a regular
>> IP connected to the OCP interconnect.
>>
>>
>>
>
> In fact, I don't think that there should be a definition of the On
> Chip Peripherals interconnect or any child node of the ocp in a DTS
> file.
> These should be defined in the included dtsi file since it will be
> common to all boards based on this SoC.

Well there is nothing wrong with that in the DTS theory, but I do agree 
that keeping these internal SoC details outside the board file is indeed 
much better.

> DTS files that describe a board can reference these nodes defined in
> the dtsi and add their custom peripherals as childs of them.
>
> So, instead defining in the DTS:
>
> am33xx_pinmux: pinmux@44e10800 {
> ...
> };
>
> gpmc: gpmc@50000000 {
> ...
> };
>
> i2c0: i2c@44e0b000 {
> ...
> };
>
> uart0: serial@44e09000 {
> ..
> };
>
> It has to be defined as:
>
> &am33xx_pinmux {
> ...
> };
>
> &gpmc {
> ...
> };
>
> &i2c0 {
> ...
> }
>
> I'm looking at other am33xx dts such as am335x-bone.dts and
> am335x-evm.dts and I see that these define device nodes already
> defined in the included .dtsi which is wrong in my opinion.
>
> The OMAP{3,4,5} dtsi and dts do correctly and can be used as a
> reference on how the device nodes have to be defined and referenced.

Good point, we should potentially clean the am33xx files to make then 
consistent with other OMAP stuff.

Thanks,
Benoit

[PATCHv3 0/2] ARM: dts: Add initial support for IGEP AQUILA

[Enric Balletbo i Serra]

From: Enric Balletbo i Serra <eballetbo@iseebcn.com>

Hi all,

These two patches introduces initial support for the IGEP AM335x-based
platforms. The first patch add support for IGEP COM AQUILA products, and the
second patch add support for the development board.

These patches apply on top of bcousson/for_3.12/dts repository.

Changes since v2:
  * Make it compatible with "isee,am335x-base0033", "isee,am335x-igep0033",
    "ti,am33xx" since these boards are manufactured by ISEE not TI. (Javier)
Changes since v1:
  * Use &node to reference the nodes already defined in dtsi files. (Javier)

Best regards,

Enric Balletbo i Serra (2):
  ARM: dts: AM33XX: Add support for IGEP COM AQUILA
  ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board.

 arch/arm/boot/dts/Makefile             |   1 +
 arch/arm/boot/dts/am335x-base0033.dts  |  16 ++
 arch/arm/boot/dts/am335x-igep0033.dtsi | 265 +++++++++++++++++++++++++++++++++
 3 files changed, 282 insertions(+)
 create mode 100644 arch/arm/boot/dts/am335x-base0033.dts
 create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi

-- 
1.8.1.2

[PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA

[Enric Balletbo i Serra]

The IGEP COM AQUILA is industrial processors SODIMM module with
following highlights:

   o AM3352/AM3354/AM3358/AM3359 Texas Instruments processor
   o Cortex-A8 ARM CPU
   o 3.3 volts Inputs / Outputs use industrial
   o 256 MB DDR3 SDRAM / 128 Megabytes FLASH
   o MicroSD card reader on-board
   o Ethernet controller on-board
   o JTAG debug connector available
   o Designed for industrial range purposes

Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
---
 arch/arm/boot/dts/am335x-igep0033.dtsi | 265 +++++++++++++++++++++++++++++++++
 1 file changed, 265 insertions(+)
 create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi

diff --git a/arch/arm/boot/dts/am335x-igep0033.dtsi b/arch/arm/boot/dts/am335x-igep0033.dtsi
new file mode 100644
index 0000000..06eba07
--- /dev/null
+++ b/arch/arm/boot/dts/am335x-igep0033.dtsi
@@ -0,0 +1,265 @@
+/*
+ * am335x-igep0033.dtsi - Device Tree file for IGEP COM AQUILA AM335x
+ *
+ * Copyright (C) 2013 ISEE 2007 SL - http://www.isee.biz
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+/dts-v1/;
+
+#include "am33xx.dtsi"
+
+/ {
+	cpus {
+		cpu@0 {
+			cpu0-supply = <&vdd1_reg>;
+		};
+	};
+
+	memory {
+		device_type = "memory";
+		reg = <0x80000000 0x10000000>; /* 256 MB */
+	};
+
+	leds {
+		pinctrl-names = "default";
+		pinctrl-0 = <&leds_pins>;
+
+		compatible = "gpio-leds";
+
+		led@0 {
+			label = "com:green:user";
+			gpios = <&gpio1 23 GPIO_ACTIVE_HIGH>;
+			default-state = "on";
+		};
+	};
+
+	vbat: fixedregulator@0 {
+		compatible = "regulator-fixed";
+		regulator-name = "vbat";
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+		regulator-boot-on;
+	};
+};
+
+&am33xx_pinmux {
+	i2c0_pins: pinmux_i2c0_pins {
+		pinctrl-single,pins = <
+			0x188 (PIN_INPUT_PULLUP | MUX_MODE0)	/* i2c0_sda.i2c0_sda */
+			0x18c (PIN_INPUT_PULLUP | MUX_MODE0)	/* i2c0_scl.i2c0_scl */
+		>;
+	};
+
+	nandflash_pins: pinmux_nandflash_pins {
+		pinctrl-single,pins = <
+			0x0 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad0.gpmc_ad0 */
+			0x4 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad1.gpmc_ad1 */
+			0x8 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad2.gpmc_ad2 */
+			0xc (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad3.gpmc_ad3 */
+			0x10 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad4.gpmc_ad4 */
+			0x14 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad5.gpmc_ad5 */
+			0x18 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad6.gpmc_ad6 */
+			0x1c (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_ad7.gpmc_ad7 */
+			0x70 (PIN_INPUT_PULLUP | MUX_MODE0)	/* gpmc_wait0.gpmc_wait0 */
+			0x74 (PIN_INPUT_PULLUP | MUX_MODE7)	/* gpmc_wpn.gpio0_30 */
+			0x7c (PIN_OUTPUT | MUX_MODE0)		/* gpmc_csn0.gpmc_csn0 */
+			0x90 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_advn_ale.gpmc_advn_ale */
+			0x94 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_oen_ren.gpmc_oen_ren */
+			0x98 (PIN_OUTPUT | MUX_MODE0)		/* gpmc_wen.gpmc_wen */
+			0x9c (PIN_OUTPUT | MUX_MODE0)		/* gpmc_be0n_cle.gpmc_be0n_cle */
+		>;
+	};
+
+	uart0_pins: pinmux_uart0_pins {
+		pinctrl-single,pins = <
+			0x170 (PIN_INPUT_PULLUP | MUX_MODE0)	/* uart0_rxd.uart0_rxd */
+			0x174 (PIN_OUTPUT_PULLDOWN | MUX_MODE0)	/* uart0_txd.uart0_txd */
+		>;
+	};
+
+	leds_pins: pinmux_leds_pins {
+		pinctrl-single,pins = <
+			0x5c (PIN_OUTPUT_PULLDOWN | MUX_MODE7)	/* gpmc_a7.gpio1_23 */
+		>;
+	};
+};
+
+&cpsw_emac0 {
+	phy_id = <&davinci_mdio>, <0>;
+};
+
+&cpsw_emac1 {
+	phy_id = <&davinci_mdio>, <1>;
+};
+
+&elm {
+	status = "okay";
+};
+
+&gpmc {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&nandflash_pins>;
+
+	ranges = <0 0 0x08000000 0x10000000>;	/* CS0: NAND */
+
+	nand@0,0 {
+		reg = <0 0 0>; /* CS0, offset 0 */
+		nand-bus-width = <8>;
+		ti,nand-ecc-opt = "bch8";
+		gpmc,device-nand = "true";
+		gpmc,device-width = <1>;
+		gpmc,sync-clk-ps = <0>;
+		gpmc,cs-on-ns = <0>;
+		gpmc,cs-rd-off-ns = <44>;
+		gpmc,cs-wr-off-ns = <44>;
+		gpmc,adv-on-ns = <6>;
+		gpmc,adv-rd-off-ns = <34>;
+		gpmc,adv-wr-off-ns = <44>;
+		gpmc,we-on-ns = <0>;
+		gpmc,we-off-ns = <40>;
+		gpmc,oe-on-ns = <0>;
+		gpmc,oe-off-ns = <54>;
+		gpmc,access-ns = <64>;
+		gpmc,rd-cycle-ns = <82>;
+		gpmc,wr-cycle-ns = <82>;
+		gpmc,wait-on-read = "true";
+		gpmc,wait-on-write = "true";
+		gpmc,bus-turnaround-ns = <0>;
+		gpmc,cycle2cycle-delay-ns = <0>;
+		gpmc,clk-activation-ns = <0>;
+		gpmc,wait-monitoring-ns = <0>;
+		gpmc,wr-access-ns = <40>;
+		gpmc,wr-data-mux-bus-ns = <0>;
+
+		#address-cells = <1>;
+		#size-cells = <1>;
+		elm_id = <&elm>;
+
+		/* MTD partition table */
+		partition@0 {
+			label = "SPL";
+			reg = <0x00000000 0x000080000>;
+		};
+
+		partition@1 {
+			label = "U-boot";
+			reg = <0x00080000 0x001e0000>;
+		};
+
+		partition@2 {
+			label = "U-Boot Env";
+			reg = <0x00260000 0x00020000>;
+		};
+
+		partition@3 {
+			label = "Kernel";
+			reg = <0x00280000 0x00500000>;
+		};
+
+		partition@4 {
+			label = "File System";
+			reg = <0x00780000 0x007880000>;
+		};
+	};
+};
+
+&i2c0 {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&i2c0_pins>;
+
+	clock-frequency = <400000>;
+
+	tps: tps@2d {
+		reg = <0x2d>;
+	};
+};
+
+&uart0 {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart0_pins>;
+};
+
+#include "tps65910.dtsi"
+
+&tps {
+	vcc1-supply = <&vbat>;
+	vcc2-supply = <&vbat>;
+	vcc3-supply = <&vbat>;
+	vcc4-supply = <&vbat>;
+	vcc5-supply = <&vbat>;
+	vcc6-supply = <&vbat>;
+	vcc7-supply = <&vbat>;
+	vccio-supply = <&vbat>;
+
+	regulators {
+		vrtc_reg: regulator@0 {
+			regulator-always-on;
+		};
+
+		vio_reg: regulator@1 {
+			regulator-always-on;
+		};
+
+		vdd1_reg: regulator@2 {
+			/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
+			regulator-name = "vdd_mpu";
+			regulator-min-microvolt = <912500>;
+			regulator-max-microvolt = <1312500>;
+			regulator-boot-on;
+			regulator-always-on;
+		};
+
+		vdd2_reg: regulator@3 {
+			/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
+			regulator-name = "vdd_core";
+			regulator-min-microvolt = <912500>;
+			regulator-max-microvolt = <1150000>;
+			regulator-boot-on;
+			regulator-always-on;
+		};
+
+		vdd3_reg: regulator@4 {
+			regulator-always-on;
+		};
+
+		vdig1_reg: regulator@5 {
+			regulator-always-on;
+		};
+
+		vdig2_reg: regulator@6 {
+			regulator-always-on;
+		};
+
+		vpll_reg: regulator@7 {
+			regulator-always-on;
+		};
+
+		vdac_reg: regulator@8 {
+			regulator-always-on;
+		};
+
+		vaux1_reg: regulator@9 {
+			regulator-always-on;
+		};
+
+		vaux2_reg: regulator@10 {
+			regulator-always-on;
+		};
+
+		vaux33_reg: regulator@11 {
+			regulator-always-on;
+		};
+
+		vmmc_reg: regulator@12 {
+			regulator-always-on;
+		};
+	};
+};
+
-- 
1.8.1.2

[PATCH 2/2] ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board.

[Enric Balletbo i Serra]

The IGEP AQUILA EXPANSION board is a development platform for the IGEP COM
AQUILA AM335x boards.

The board adds the following connectivity:

    o USB OTG
    o USB HOST
    o HDMI
    o Ethernet
    o Serial Debug (3.3V)
    o 2x46 pin headers
    o EEPROM

Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
---
 arch/arm/boot/dts/Makefile            |  1 +
 arch/arm/boot/dts/am335x-base0033.dts | 16 ++++++++++++++++
 2 files changed, 17 insertions(+)
 create mode 100644 arch/arm/boot/dts/am335x-base0033.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 641b3c9..12c2949 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -171,6 +171,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \
 	am335x-evm.dtb \
 	am335x-evmsk.dtb \
 	am335x-bone.dtb \
+	am335x-base0033.dtb \
 	am3517-evm.dtb \
 	am3517_mt_ventoux.dtb \
 	am43x-epos-evm.dtb
diff --git a/arch/arm/boot/dts/am335x-base0033.dts b/arch/arm/boot/dts/am335x-base0033.dts
new file mode 100644
index 0000000..7a079db
--- /dev/null
+++ b/arch/arm/boot/dts/am335x-base0033.dts
@@ -0,0 +1,16 @@
+/*
+ * am335x-base0033.dtsi - Device Tree file for IGEP AQUILA EXPANSION
+ *
+ * Copyright (C) 2013 ISEE 2007 SL - http://www.isee.biz
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+#include "am335x-igep0033.dtsi"
+
+/ {
+	model = "IGEP COM AM335x on AQUILA Expansion";
+	compatible = "isee,am335x-base0033", "isee,am335x-igep0033", "ti,am33xx";
+};
-- 
1.8.1.2

Re: [PATCHv3 0/2] ARM: dts: Add initial support for IGEP AQUILA

[Javier Martinez Canillas]

On Tue, Sep 10, 2013 at 4:55 PM, Enric Balletbo i Serra
<eballetbo@gmail.com> wrote:
> From: Enric Balletbo i Serra <eballetbo@iseebcn.com>
>
> Hi all,
>
> These two patches introduces initial support for the IGEP AM335x-based
> platforms. The first patch add support for IGEP COM AQUILA products, and the
> second patch add support for the development board.
>
> These patches apply on top of bcousson/for_3.12/dts repository.
>
> Changes since v2:
>   * Make it compatible with "isee,am335x-base0033", "isee,am335x-igep0033",
>     "ti,am33xx" since these boards are manufactured by ISEE not TI. (Javier)
> Changes since v1:
>   * Use &node to reference the nodes already defined in dtsi files. (Javier)
>
> Best regards,
>
> Enric Balletbo i Serra (2):
>   ARM: dts: AM33XX: Add support for IGEP COM AQUILA
>   ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board.
>
>  arch/arm/boot/dts/Makefile             |   1 +
>  arch/arm/boot/dts/am335x-base0033.dts  |  16 ++
>  arch/arm/boot/dts/am335x-igep0033.dtsi | 265 +++++++++++++++++++++++++++++++++
>  3 files changed, 282 insertions(+)
>  create mode 100644 arch/arm/boot/dts/am335x-base0033.dts
>  create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi
>
> --
> 1.8.1.2
>

Hi Benoit and Tony,

Any comments on this series? I had already reviewed previous versions
of the DT and it looks good to me now.

It would be great if this can make it for 3.13.

Thanks a lot and best regards,
Javier

Re: [PATCHv3 0/2] ARM: dts: Add initial support for IGEP AQUILA

[Tony Lindgren]

* Javier Martinez Canillas <martinez.javier@gmail.com> [130925 06:18]:
> On Tue, Sep 10, 2013 at 4:55 PM, Enric Balletbo i Serra
> <eballetbo@gmail.com> wrote:
> > From: Enric Balletbo i Serra <eballetbo@iseebcn.com>
> >
> > Hi all,
> >
> > These two patches introduces initial support for the IGEP AM335x-based
> > platforms. The first patch add support for IGEP COM AQUILA products, and the
> > second patch add support for the development board.
> >
> > These patches apply on top of bcousson/for_3.12/dts repository.
> >
> > Changes since v2:
> >   * Make it compatible with "isee,am335x-base0033", "isee,am335x-igep0033",
> >     "ti,am33xx" since these boards are manufactured by ISEE not TI. (Javier)
> > Changes since v1:
> >   * Use &node to reference the nodes already defined in dtsi files. (Javier)
> >
> > Best regards,
> >
> > Enric Balletbo i Serra (2):
> >   ARM: dts: AM33XX: Add support for IGEP COM AQUILA
> >   ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board.
> >
> >  arch/arm/boot/dts/Makefile             |   1 +
> >  arch/arm/boot/dts/am335x-base0033.dts  |  16 ++
> >  arch/arm/boot/dts/am335x-igep0033.dtsi | 265 +++++++++++++++++++++++++++++++++
> >  3 files changed, 282 insertions(+)
> >  create mode 100644 arch/arm/boot/dts/am335x-base0033.dts
> >  create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi
> >
> > --
> > 1.8.1.2
> >
> 
> Hi Benoit and Tony,
> 
> Any comments on this series? I had already reviewed previous versions
> of the DT and it looks good to me now.
> 
> It would be great if this can make it for 3.13.

Looks good to me, I would assume Benoit will be looking at this
shortly.

Tony

Re: [PATCHv3 0/2] ARM: dts: Add initial support for IGEP AQUILA

[Benoit Cousson]

Hi Enric,

On 03/10/2013 20:17, Tony Lindgren wrote:
> * Javier Martinez Canillas <martinez.javier@gmail.com> [130925 06:18]:
>> On Tue, Sep 10, 2013 at 4:55 PM, Enric Balletbo i Serra
>> <eballetbo@gmail.com> wrote:
>>> From: Enric Balletbo i Serra <eballetbo@iseebcn.com>
>>>
>>> Hi all,
>>>
>>> These two patches introduces initial support for the IGEP AM335x-based
>>> platforms. The first patch add support for IGEP COM AQUILA products, and the
>>> second patch add support for the development board.
>>>
>>> These patches apply on top of bcousson/for_3.12/dts repository.
>>>
>>> Changes since v2:
>>>    * Make it compatible with "isee,am335x-base0033", "isee,am335x-igep0033",
>>>      "ti,am33xx" since these boards are manufactured by ISEE not TI. (Javier)
>>> Changes since v1:
>>>    * Use &node to reference the nodes already defined in dtsi files. (Javier)
>>>
>>> Best regards,
>>>
>>> Enric Balletbo i Serra (2):
>>>    ARM: dts: AM33XX: Add support for IGEP COM AQUILA
>>>    ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board.
>>>
>>>   arch/arm/boot/dts/Makefile             |   1 +
>>>   arch/arm/boot/dts/am335x-base0033.dts  |  16 ++
>>>   arch/arm/boot/dts/am335x-igep0033.dtsi | 265 +++++++++++++++++++++++++++++++++
>>>   3 files changed, 282 insertions(+)
>>>   create mode 100644 arch/arm/boot/dts/am335x-base0033.dts
>>>   create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi
>>>
>>> --
>>> 1.8.1.2
>>>
>>
>> Hi Benoit and Tony,
>>
>> Any comments on this series? I had already reviewed previous versions
>> of the DT and it looks good to me now.
>>
>> It would be great if this can make it for 3.13.
>
> Looks good to me, I would assume Benoit will be looking at this
> shortly.

I did :-)

It looks good to me too, I've just fixed a conflict with my current 
branch in the Makefile and fix a typo in the header:

+++ b/arch/arm/boot/dts/am335x-base0033.dts
@@ -0,0 +1,16 @@
+/*
+ * am335x-base0033.dtsi - Device Tree file for IGEP AQUILA EXPANSION

Should be am335x-base0033.dts. I fixed it for you. Please check if the 
fix is ok in the branch.

Pushed on my for_3.13/dts branch.

Thanks,
Benoit

Re: [PATCHv3 0/2] ARM: dts: Add initial support for IGEP AQUILA

[Enric Balletbo Serra]

Hi Benoit,

2013/10/4 Benoit Cousson <bcousson@baylibre.com>:
> Hi Enric,
>
>
> On 03/10/2013 20:17, Tony Lindgren wrote:
>>
>> * Javier Martinez Canillas <martinez.javier@gmail.com> [130925 06:18]:
>>>
>>> On Tue, Sep 10, 2013 at 4:55 PM, Enric Balletbo i Serra
>>> <eballetbo@gmail.com> wrote:
>>>>
>>>> From: Enric Balletbo i Serra <eballetbo@iseebcn.com>
>>>>
>>>> Hi all,
>>>>
>>>> These two patches introduces initial support for the IGEP AM335x-based
>>>> platforms. The first patch add support for IGEP COM AQUILA products, and
>>>> the
>>>> second patch add support for the development board.
>>>>
>>>> These patches apply on top of bcousson/for_3.12/dts repository.
>>>>
>>>> Changes since v2:
>>>>    * Make it compatible with "isee,am335x-base0033",
>>>> "isee,am335x-igep0033",
>>>>      "ti,am33xx" since these boards are manufactured by ISEE not TI.
>>>> (Javier)
>>>> Changes since v1:
>>>>    * Use &node to reference the nodes already defined in dtsi files.
>>>> (Javier)
>>>>
>>>> Best regards,
>>>>
>>>> Enric Balletbo i Serra (2):
>>>>    ARM: dts: AM33XX: Add support for IGEP COM AQUILA
>>>>    ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board.
>>>>
>>>>   arch/arm/boot/dts/Makefile             |   1 +
>>>>   arch/arm/boot/dts/am335x-base0033.dts  |  16 ++
>>>>   arch/arm/boot/dts/am335x-igep0033.dtsi | 265
>>>> +++++++++++++++++++++++++++++++++
>>>>   3 files changed, 282 insertions(+)
>>>>   create mode 100644 arch/arm/boot/dts/am335x-base0033.dts
>>>>   create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi
>>>>
>>>> --
>>>> 1.8.1.2
>>>>
>>>
>>> Hi Benoit and Tony,
>>>
>>> Any comments on this series? I had already reviewed previous versions
>>> of the DT and it looks good to me now.
>>>
>>> It would be great if this can make it for 3.13.
>>
>>
>> Looks good to me, I would assume Benoit will be looking at this
>> shortly.
>
>
> I did :-)
>
> It looks good to me too, I've just fixed a conflict with my current branch
> in the Makefile and fix a typo in the header:
>
> +++ b/arch/arm/boot/dts/am335x-base0033.dts
> @@ -0,0 +1,16 @@
> +/*
> + * am335x-base0033.dtsi - Device Tree file for IGEP AQUILA EXPANSION
>
> Should be am335x-base0033.dts. I fixed it for you. Please check if the fix
> is ok in the branch.
>

Checked.

> Pushed on my for_3.13/dts branch.
>
> Thanks,
> Benoit
>

Thanks,
   Enric