Skip to contentSkip to conversation

ARM: LPC32xx: Device tree updates

GIT PULL Roland Stigge 3 participants 3 messages Sep 10, 2014 – Sep 11, 2014

Subsystems (5)
lore
Share & tools
lore
#0 lore raw

The following changes since commit 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9:

Linux 3.17-rc1 (2014-08-16 10:40:26 -0600)

are available in the git repository at:

git://git.antcom.de/linux-2.6 lpc32xx/dt

for you to fetch changes up to 280f965674dbc5090147724d90b701f2b81f07e3:

ARM: LPC32xx: Use DMA for both SSP channels (2014-09-10 15:41:30 +0200)

----------------------------------------------------------------
Roland Stigge (1):
ARM: LPC32xx: Use DMA for both SSP channels

 arch/arm/boot/dts/lpc32xx.dtsi | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)
#1 lore raw
Quoted 10 lines · On Wednesday 10 September 2014 15:58:15 Roland Stigge wrote:

The following changes since commit 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9:

Linux 3.17-rc1 (2014-08-16 10:40:26 -0600)

are available in the git repository at:

git://git.antcom.de/linux-2.6 lpc32xx/dt

for you to fetch changes up to 280f965674dbc5090147724d90b701f2b81f07e3:

Quoted 8 lines

ARM: LPC32xx: Use DMA for both SSP channels (2014-09-10 15:41:30 +0200)

----------------------------------------------------------------
Roland Stigge (1):
ARM: LPC32xx: Use DMA for both SSP channels

 arch/arm/boot/dts/lpc32xx.dtsi | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

This unfortunately uses a binding for pl08x that hasn't been reviewed yet, so I can't pull it. See below for an illustration of how I think the binding should work. There are probably bugs in that code, but hopefully it's not too far off.

Arnd

arch/arm/boot/dts/lpc32xx.dtsi +12 -1
index 3abebb75fc57..72b1923d6ad3 100644
@@ -40,6 +40,8 @@
 			compatible = "nxp,lpc3220-slc";
 			reg = <0x20020000 0x1000>;
 			status = "disabled";
+			dmas = <&pl080 1 1>
+			dma-names = "data";
 		};
 
 		mlc: flash at 200a8000 {
@@ -47,12 +49,17 @@
 			reg = <0x200a8000 0x11000>;
 			interrupts = <11 0>;
 			status = "disabled";
+			dmas = <&pl080 12 1>;
+			dma-names = "data";
 		};
 
-		dma at 31000000 {
+		pl080: dma at 31000000 {
 			compatible = "arm,pl080", "arm,primecell";
 			reg = <0x31000000 0x1000>;
 			interrupts = <0x1c 0>;
+			#dma-cells = <2>;
+			dma-channels = <10>;
+			dma-requests = <32>;
 		};
 
 		/*
@@ -95,11 +102,15 @@
 				compatible = "arm,pl022", "arm,primecell";
 				reg = <0x20084000 0x1000>;
 				interrupts = <0x14 0>;
+				dmas = <&pl080 3 1>, <&pl080 11 1>;
+				dma-names = "rx", "tx";
 			};
 
 			spi1: spi at 20088000 {
 				compatible = "nxp,lpc3220-spi";
 				reg = <0x20088000 0x1000>;
+				dmas = <&pl080 14 1>, <&pl080 15 1>;
+				dma-names = "rx", "tx";
 			};
 
 			ssp1: ssp at 2008c000 {
arch/arm/mach-lpc32xx/phy3250.c +3 -40

62 lines not on this page · raw message

drivers/dma/amba-pl08x.c +53 -0

65 lines not on this page · raw message

#2 lore raw
Quoted 17 lines · Arnd Bergmann
Quoted 16 lines · On Wednesday 10 September 2014 15:58:15 Roland Stigge wrote:

The following changes since commit 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9:

Linux 3.17-rc1 (2014-08-16 10:40:26 -0600)

are available in the git repository at:

git://git.antcom.de/linux-2.6 lpc32xx/dt

for you to fetch changes up to 280f965674dbc5090147724d90b701f2b81f07e3:

ARM: LPC32xx: Use DMA for both SSP channels (2014-09-10 15:41:30 +0200)

----------------------------------------------------------------
Roland Stigge (1):
ARM: LPC32xx: Use DMA for both SSP channels

Quoted 7 lines
Quoted 2 lines
 arch/arm/boot/dts/lpc32xx.dtsi | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

This unfortunately uses a binding for pl08x that hasn't been reviewed yet, so I can't pull it. See below for an illustration of how I think the binding should work. There are probably bugs in that code, but hopefully it's not too far off.

I have a more complete PL08x DT binding which includes defining all signals etc. I will kick my own butt to get it out tonight or so so you can compare it with this.

I think Russell want to be involved in reviewing this, his input was very important when I defined my bindings.

Yours,
Linus Walleij