[v2 PATCH 13/14] ARM: dts: add spi nodes for atmel SoC
From: Yang, Wenyou <hidden>
Date: 2012-12-07 01:37:07
Also in:
lkml
Hi Ludovic,
-----Original Message----- From: Desroches, Ludovic Sent: 2012?12?7? 0:37 To: Yang, Wenyou Cc: linux-arm-kernel at lists.infradead.org; linux at arm.linux.org.uk; richard.genoud at gmail.com; Lin, JM; Ferre, Nicolas; linux-kernel at vger.kernel.org; plagnioj at jcrosoft.com; Desroches, Ludovic Subject: Re: [v2 PATCH 13/14] ARM: dts: add spi nodes for atmel SoC Hi, On 12/04/2012 08:53 AM, Wenyou Yang wrote:quoted
From: Richard Genoud <redacted> Signed-off-by: Richard Genoud <redacted> [wenyou.yang at atmel.com: add spi nodes for other atmel SOC] Signed-off-by: Wenyou Yang <redacted> Cc: linux at arm.linux.org.uk --- arch/arm/boot/dts/at91sam9260.dtsi | 30++++++++++++++++++++++++++++++quoted
arch/arm/boot/dts/at91sam9263.dtsi | 30++++++++++++++++++++++++++++++quoted
arch/arm/boot/dts/at91sam9g45.dtsi | 30++++++++++++++++++++++++++++++quoted
arch/arm/boot/dts/at91sam9n12.dtsi | 30++++++++++++++++++++++++++++++quoted
arch/arm/boot/dts/at91sam9x5.dtsi | 30++++++++++++++++++++++++++++++quoted
5 files changed, 150 insertions(+)diff --git a/arch/arm/boot/dts/at91sam9260.dtsib/arch/arm/boot/dts/at91sam9260.dtsiquoted
index d410581..87254aa 100644--- a/arch/arm/boot/dts/at91sam9260.dtsi +++ b/arch/arm/boot/dts/at91sam9260.dtsi@@ -212,6 +212,36 @@ status = "disabled"; }; + spi0: spi at fffc8000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9260-spi"; + reg = <0xfffc8000 0x200>; + interrupts = <12 4 3>; + cs-gpios = <&pioA 3 0 + &pioC 11 0 + &pioC 16 0 + &pioC 17 0 + >;From my point of view, the cs-gpios stuff should be in board dts files instead of the SoC dtsi file. These pios can be shared with other devices. If only one chip select gpio is used on the board, it could avoid some useless pin conflicts.
Thanks These pins are defined as CS0, CS1, CS2 and CS3 for SoC. it is selected for the board. And it will not be configured unless it is selected. So it doesn't cause pin conficts.
Regards Ludovic
Best Regards, Wenyou Yang
quoted
+ dma-mask = <0xffffffff>; + status = "disabled"; + }; + + spi1: spi at fffcc000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9260-spi"; + reg = <0xfffcc000 0x200>; + interrupts = <13 4 3>; + cs-gpios = <&pioB 3 0 + &pioC 5 0 + &pioC 4 0 + &pioC 3 0 + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; + adc0: adc at fffe0000 { compatible = "atmel,at91sam9260-adc"; reg = <0xfffe0000 0x100>;diff --git a/arch/arm/boot/dts/at91sam9263.dtsib/arch/arm/boot/dts/at91sam9263.dtsiquoted
index 3e6e5c1..528395d 100644--- a/arch/arm/boot/dts/at91sam9263.dtsi +++ b/arch/arm/boot/dts/at91sam9263.dtsi@@ -195,6 +195,36 @@ #size-cells = <0>; status = "disabled"; }; + + spi0: spi at fffa4000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9260-spi"; + reg = <0xfffa4000 0x200>; + interrupts = <14 4 3>; + cs-gpios = <&pioA 5 0 + &pioA 3 0 + &pioA 4 0 + &pioB 11 0 + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; + + spi1: spi at fffa8000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9260-spi"; + reg = <0xfffa8000 0x200>; + interrupts = <15 4 3>; + cs-gpios = <&pioB 15 0 + &pioB 16 0 + &pioB 17 0 + &pioB 18 0 + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; }; nand0: nand at 40000000 {diff --git a/arch/arm/boot/dts/at91sam9g45.dtsib/arch/arm/boot/dts/at91sam9g45.dtsiquoted
index 3add030..8a6aae92 100644--- a/arch/arm/boot/dts/at91sam9g45.dtsi +++ b/arch/arm/boot/dts/at91sam9g45.dtsi@@ -262,6 +262,36 @@ trigger-value = <0x6>; }; }; + + spi0: spi at fffa4000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9260-spi"; + reg = <0xfffa4000 0x200>; + interrupts = <14 4 3>; + cs-gpios = <&pioB 3 0 + &pioB 18 0 + &pioB 19 0 + &pioD 27 0 + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; + + spi1: spi at fffa8000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9260-spi"; + reg = <0xfffa8000 0x200>; + interrupts = <15 4 3>; + cs-gpios = <&pioB 17 0 + &pioD 28 0 + &pioD 18 0 + &pioD 19 0 + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; }; nand0: nand at 40000000 {diff --git a/arch/arm/boot/dts/at91sam9n12.dtsib/arch/arm/boot/dts/at91sam9n12.dtsiquoted
index 82508d6..183cacf 100644--- a/arch/arm/boot/dts/at91sam9n12.dtsi +++ b/arch/arm/boot/dts/at91sam9n12.dtsi@@ -202,6 +202,36 @@ #size-cells = <0>; status = "disabled"; }; + + spi0: spi at f0000000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9x5-spi"; + reg = <0xf0000000 0x100>; + interrupts = <13 4 3>; + cs-gpios = <&pioA 14 0 + &pioA 7 0 + &pioA 1 0 + &pioB 3 0 + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; + + spi1: spi at f0004000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9x5-spi"; + reg = <0xf0004000 0x100>; + interrupts = <14 4 3>; + cs-gpios = <&pioA 8 0 + &pioA 0 0 + &pioA 31 0 + &pioA 30 0 + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; }; nand0: nand at 40000000 {diff --git a/arch/arm/boot/dts/at91sam9x5.dtsib/arch/arm/boot/dts/at91sam9x5.dtsiquoted
index 03fc136..95b55fd 100644--- a/arch/arm/boot/dts/at91sam9x5.dtsi +++ b/arch/arm/boot/dts/at91sam9x5.dtsi@@ -263,6 +263,36 @@ trigger-value = <0x6>; }; }; + + spi0: spi at f0000000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9x5-spi"; + reg = <0xf0000000 0x100>; + interrupts = <13 4 3>; + cs-gpios = <&pioA 14 0 + &pioA 7 0 /* conflicts with TXD2 */ + &pioA 1 0 /* conflicts with RXD0 */ + &pioB 3 0 /* conflicts with ERXDV */ + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; + + spi1: spi at f0004000 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "atmel,at91sam9x5-spi"; + reg = <0xf0004000 0x100>; + interrupts = <14 4 3>; + cs-gpios = <&pioA 8 0 /* conflitcs with RXD2 */ + &pioA 0 0 /* conflitcs with TXD0 */ + &pioA 31 0 /* conflitcs with TWCK0 */ + &pioA 30 0 /* conflitcs with TWD0 */ + >; + dma-mask = <0xffffffff>; + status = "disabled"; + }; }; nand0: nand at 40000000 {