* [PATCH 0/2] ARM: dts: Add initial support for IGEP AQUILA @ 2013-06-19 8:27 Enric Balletbo i Serra 2013-06-19 8:27 ` [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA Enric Balletbo i Serra 2013-06-19 8:27 ` [PATCH 2/2] ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board Enric Balletbo i Serra 0 siblings, 2 replies; 7+ messages in thread From: Enric Balletbo i Serra @ 2013-06-19 8:27 UTC (permalink / raw) To: linux-omap, Benoit Cousson, Tony Lindgren, Javier Martinez Canillas Cc: Enric Balletbo i Serra 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.11/dts repository. 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 | 3 +- arch/arm/boot/dts/am335x-base0033.dts | 17 +++ arch/arm/boot/dts/am335x-igep0033.dtsi | 269 +++++++++++++++++++++++++++++++++ 3 files changed, 288 insertions(+), 1 deletion(-) create mode 100644 arch/arm/boot/dts/am335x-base0033.dts create mode 100644 arch/arm/boot/dts/am335x-igep0033.dtsi -- 1.8.1.2 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA 2013-06-19 8:27 [PATCH 0/2] ARM: dts: Add initial support for IGEP AQUILA Enric Balletbo i Serra @ 2013-06-19 8:27 ` Enric Balletbo i Serra 2013-06-19 10:46 ` Benoit Cousson 2013-06-19 8:27 ` [PATCH 2/2] ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board Enric Balletbo i Serra 1 sibling, 1 reply; 7+ messages in thread From: Enric Balletbo i Serra @ 2013-06-19 8:27 UTC (permalink / raw) To: linux-omap, Benoit Cousson, Tony Lindgren, Javier Martinez Canillas Cc: 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 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA 2013-06-19 8:27 ` [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA Enric Balletbo i Serra @ 2013-06-19 10:46 ` Benoit Cousson 2013-06-19 13:32 ` Javier Martinez Canillas 0 siblings, 1 reply; 7+ messages in thread From: Benoit Cousson @ 2013-06-19 10:46 UTC (permalink / raw) To: Enric Balletbo i Serra Cc: linux-omap, Tony Lindgren, Javier Martinez Canillas 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA 2013-06-19 10:46 ` Benoit Cousson @ 2013-06-19 13:32 ` Javier Martinez Canillas 2013-06-20 14:16 ` Cousson, Benoit 0 siblings, 1 reply; 7+ messages in thread From: Javier Martinez Canillas @ 2013-06-19 13:32 UTC (permalink / raw) To: Benoit Cousson; +Cc: Enric Balletbo i Serra, linux-omap, Tony Lindgren 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA 2013-06-19 13:32 ` Javier Martinez Canillas @ 2013-06-20 14:16 ` Cousson, Benoit 0 siblings, 0 replies; 7+ messages in thread From: Cousson, Benoit @ 2013-06-20 14:16 UTC (permalink / raw) To: Javier Martinez Canillas Cc: Enric Balletbo i Serra, linux-omap, Tony Lindgren 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board. 2013-06-19 8:27 [PATCH 0/2] ARM: dts: Add initial support for IGEP AQUILA Enric Balletbo i Serra 2013-06-19 8:27 ` [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA Enric Balletbo i Serra @ 2013-06-19 8:27 ` Enric Balletbo i Serra 1 sibling, 0 replies; 7+ messages in thread From: Enric Balletbo i Serra @ 2013-06-19 8:27 UTC (permalink / raw) To: linux-omap, Benoit Cousson, Tony Lindgren, Javier Martinez Canillas Cc: 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> --- arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/am335x-base0033.dts | 17 +++++++++++++++++ 2 files changed, 18 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 8e50761..cb675d2 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -154,6 +154,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 dtb-$(CONFIG_ARCH_ORION5X) += orion5x-lacie-ethernet-disk-mini-v2.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..a365f11 --- /dev/null +++ b/arch/arm/boot/dts/am335x-base0033.dts @@ -0,0 +1,17 @@ +/* + * 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. + * + * IGEP AQUILA Expansion Board. + * + */ + +#include "am335x-igep0033.dtsi" + +/ { + model = "IGEP COM AM335x on AQUILA Expansion"; + compatible = "ti,am335x-base0033", "ti,am335x-igep0033", "ti,am33xx"; +}; -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCHv3 0/2] ARM: dts: Add initial support for IGEP AQUILA
@ 2013-09-10 14:55 Enric Balletbo i Serra
2013-09-10 14:55 ` [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA Enric Balletbo i Serra
0 siblings, 1 reply; 7+ messages in thread
From: Enric Balletbo i Serra @ 2013-09-10 14:55 UTC (permalink / raw)
To: linux-omap, Benoit Cousson, Tony Lindgren,
Javier Martinez Canillas
Cc: 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
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA 2013-09-10 14:55 [PATCHv3 0/2] ARM: dts: Add initial support for IGEP AQUILA Enric Balletbo i Serra @ 2013-09-10 14:55 ` Enric Balletbo i Serra 0 siblings, 0 replies; 7+ messages in thread From: Enric Balletbo i Serra @ 2013-09-10 14:55 UTC (permalink / raw) To: linux-omap, Benoit Cousson, Tony Lindgren, Javier Martinez Canillas Cc: Enric Balletbo i Serra, 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 ^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-09-10 14:57 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-06-19 8:27 [PATCH 0/2] ARM: dts: Add initial support for IGEP AQUILA Enric Balletbo i Serra 2013-06-19 8:27 ` [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA Enric Balletbo i Serra 2013-06-19 10:46 ` Benoit Cousson 2013-06-19 13:32 ` Javier Martinez Canillas 2013-06-20 14:16 ` Cousson, Benoit 2013-06-19 8:27 ` [PATCH 2/2] ARM: dts: AM33XX: Add support for IGEP AQUILA EXPANSION board Enric Balletbo i Serra -- strict thread matches above, loose matches on Subject: below -- 2013-09-10 14:55 [PATCHv3 0/2] ARM: dts: Add initial support for IGEP AQUILA Enric Balletbo i Serra 2013-09-10 14:55 ` [PATCH 1/2] ARM: dts: AM33XX: Add support for IGEP COM AQUILA Enric Balletbo i Serra
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).