* [PATCH 0/2] ARM: shmobile: r8a7793: Increase scif support @ 2015-11-27 1:52 ` Simon Horman 0 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-11-27 1:52 UTC (permalink / raw) To: linux-arm-kernel Hi, this short series enables DMA for serial devices already present and then adds missing serial ports to the r8a7793 device tree. This is based on prior work for the r8a7791 SoC by Laurent Pinchart. Lightly tested on top of renesas-devel-20151126-v4.4-rc2. Simon Horman (2): ARM: shmobile: r8a7793: Add missing serial devices to DT ARM: shmobile: r8a7793: Describe DMA for the serial ports arch/arm/boot/dts/r8a7793.dtsi | 215 ++++++++++++++++++++++++++++++++++++++++- 1 file changed, 213 insertions(+), 2 deletions(-) -- 2.1.4 ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 0/2] ARM: shmobile: r8a7793: Increase scif support @ 2015-11-27 1:52 ` Simon Horman 0 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-11-27 1:52 UTC (permalink / raw) To: linux-arm-kernel Hi, this short series enables DMA for serial devices already present and then adds missing serial ports to the r8a7793 device tree. This is based on prior work for the r8a7791 SoC by Laurent Pinchart. Lightly tested on top of renesas-devel-20151126-v4.4-rc2. Simon Horman (2): ARM: shmobile: r8a7793: Add missing serial devices to DT ARM: shmobile: r8a7793: Describe DMA for the serial ports arch/arm/boot/dts/r8a7793.dtsi | 215 ++++++++++++++++++++++++++++++++++++++++- 1 file changed, 213 insertions(+), 2 deletions(-) -- 2.1.4 ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 1/2] ARM: shmobile: r8a7793: Add missing serial devices to DT 2015-11-27 1:52 ` Simon Horman @ 2015-11-27 1:52 ` Simon Horman -1 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-11-27 1:52 UTC (permalink / raw) To: linux-arm-kernel Instantiate all serial devices in r8a7793 device tree and set them as disabled by default. Based on similar work for the r8a7791 by Laurent Pinchart. Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- arch/arm/boot/dts/r8a7793.dtsi | 179 ++++++++++++++++++++++++++++++++++++++++- 1 file changed, 177 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi index cbcda80f4658..3ac2ec0414a0 100644 --- a/arch/arm/boot/dts/r8a7793.dtsi +++ b/arch/arm/boot/dts/r8a7793.dtsi @@ -298,6 +298,96 @@ dma-channels = <15>; }; + scifa0: serial@e6c40000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c40000 0 64>; + interrupts = <0 144 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFA0>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa1: serial@e6c50000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c50000 0 64>; + interrupts = <0 145 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFA1>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa2: serial@e6c60000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c60000 0 64>; + interrupts = <0 151 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFA2>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa3: serial@e6c70000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c70000 0 64>; + interrupts = <0 29 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp11_clks R8A7793_CLK_SCIFA3>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa4: serial@e6c78000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c78000 0 64>; + interrupts = <0 30 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp11_clks R8A7793_CLK_SCIFA4>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa5: serial@e6c80000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c80000 0 64>; + interrupts = <0 31 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp11_clks R8A7793_CLK_SCIFA5>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifb0: serial@e6c20000 { + compatible = "renesas,scifb-r8a7793", "renesas,scifb"; + reg = <0 0xe6c20000 0 64>; + interrupts = <0 148 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFB0>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifb1: serial@e6c30000 { + compatible = "renesas,scifb-r8a7793", "renesas,scifb"; + reg = <0 0xe6c30000 0 64>; + interrupts = <0 149 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFB1>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifb2: serial@e6ce0000 { + compatible = "renesas,scifb-r8a7793", "renesas,scifb"; + reg = <0 0xe6ce0000 0 64>; + interrupts = <0 150 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFB2>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + scif0: serial@e6e60000 { compatible = "renesas,scif-r8a7793", "renesas,scif"; reg = <0 0xe6e60000 0 64>; @@ -318,6 +408,76 @@ status = "disabled"; }; + scif2: serial@e6e58000 { + compatible = "renesas,scif-r8a7793", "renesas,scif"; + reg = <0 0xe6e58000 0 64>; + interrupts = <0 22 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_SCIF2>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scif3: serial@e6ea8000 { + compatible = "renesas,scif-r8a7793", "renesas,scif"; + reg = <0 0xe6ea8000 0 64>; + interrupts = <0 23 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_SCIF3>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scif4: serial@e6ee0000 { + compatible = "renesas,scif-r8a7793", "renesas,scif"; + reg = <0 0xe6ee0000 0 64>; + interrupts = <0 24 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_SCIF4>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scif5: serial@e6ee8000 { + compatible = "renesas,scif-r8a7793", "renesas,scif"; + reg = <0 0xe6ee8000 0 64>; + interrupts = <0 25 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_SCIF5>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + hscif0: serial@e62c0000 { + compatible = "renesas,hscif-r8a7793", "renesas,hscif"; + reg = <0 0xe62c0000 0 96>; + interrupts = <0 154 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_HSCIF0>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + hscif1: serial@e62c8000 { + compatible = "renesas,hscif-r8a7793", "renesas,hscif"; + reg = <0 0xe62c8000 0 96>; + interrupts = <0 155 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_HSCIF1>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + hscif2: serial@e62d0000 { + compatible = "renesas,hscif-r8a7793", "renesas,hscif"; + reg = <0 0xe62d0000 0 96>; + interrupts = <0 21 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_HSCIF2>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + ether: ethernet@ee700000 { compatible = "renesas,ether-r8a7793"; reg = <0 0xee700000 0 0x400>; @@ -530,12 +690,17 @@ mstp2_clks: mstp2_clks@e6150138 { compatible = "renesas,r8a7793-mstp-clocks", "renesas,cpg-mstp-clocks"; reg = <0 0xe6150138 0 4>, <0 0xe6150040 0 4>; - clocks = <&zs_clk>, <&zs_clk>; + clocks = <&mp_clk>, <&mp_clk>, <&mp_clk>, <&mp_clk>, + <&mp_clk>, <&mp_clk>, <&zs_clk>, <&zs_clk>; #clock-cells = <1>; clock-indices = < + R8A7793_CLK_SCIFA2 R8A7793_CLK_SCIFA1 R8A7793_CLK_SCIFA0 + R8A7793_CLK_SCIFB0 R8A7793_CLK_SCIFB1 R8A7793_CLK_SCIFB2 R8A7793_CLK_SYS_DMAC1 R8A7793_CLK_SYS_DMAC0 >; - clock-output-names = "sys-dmac1", "sys-dmac0"; + clock-output-names + "scifa2", "scifa1", "scifa0", "scifb0", + "scifb1", "scifb2", "sys-dmac1", "sys-dmac0"; }; mstp3_clks: mstp3_clks@e615013c { compatible = "renesas,r8a7793-mstp-clocks", @@ -634,6 +799,16 @@ "gpio3", "gpio2", "gpio1", "gpio0", "qspi_mod"; }; + mstp11_clks: mstp11_clks@e615099c { + compatible = "renesas,r8a7793-mstp-clocks", "renesas,cpg-mstp-clocks"; + reg = <0 0xe615099c 0 4>, <0 0xe61509ac 0 4>; + clocks = <&mp_clk>, <&mp_clk>, <&mp_clk>; + #clock-cells = <1>; + clock-indices = < + R8A7793_CLK_SCIFA3 R8A7793_CLK_SCIFA4 R8A7793_CLK_SCIFA5 + >; + clock-output-names = "scifa3", "scifa4", "scifa5"; + }; }; ipmmu_sy0: mmu@e6280000 { -- 2.1.4 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 1/2] ARM: shmobile: r8a7793: Add missing serial devices to DT @ 2015-11-27 1:52 ` Simon Horman 0 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-11-27 1:52 UTC (permalink / raw) To: linux-arm-kernel Instantiate all serial devices in r8a7793 device tree and set them as disabled by default. Based on similar work for the r8a7791 by Laurent Pinchart. Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- arch/arm/boot/dts/r8a7793.dtsi | 179 ++++++++++++++++++++++++++++++++++++++++- 1 file changed, 177 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi index cbcda80f4658..3ac2ec0414a0 100644 --- a/arch/arm/boot/dts/r8a7793.dtsi +++ b/arch/arm/boot/dts/r8a7793.dtsi @@ -298,6 +298,96 @@ dma-channels = <15>; }; + scifa0: serial at e6c40000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c40000 0 64>; + interrupts = <0 144 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFA0>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa1: serial at e6c50000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c50000 0 64>; + interrupts = <0 145 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFA1>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa2: serial at e6c60000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c60000 0 64>; + interrupts = <0 151 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFA2>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa3: serial at e6c70000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c70000 0 64>; + interrupts = <0 29 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp11_clks R8A7793_CLK_SCIFA3>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa4: serial at e6c78000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c78000 0 64>; + interrupts = <0 30 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp11_clks R8A7793_CLK_SCIFA4>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifa5: serial at e6c80000 { + compatible = "renesas,scifa-r8a7793", "renesas,scifa"; + reg = <0 0xe6c80000 0 64>; + interrupts = <0 31 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp11_clks R8A7793_CLK_SCIFA5>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifb0: serial at e6c20000 { + compatible = "renesas,scifb-r8a7793", "renesas,scifb"; + reg = <0 0xe6c20000 0 64>; + interrupts = <0 148 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFB0>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifb1: serial at e6c30000 { + compatible = "renesas,scifb-r8a7793", "renesas,scifb"; + reg = <0 0xe6c30000 0 64>; + interrupts = <0 149 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFB1>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scifb2: serial at e6ce0000 { + compatible = "renesas,scifb-r8a7793", "renesas,scifb"; + reg = <0 0xe6ce0000 0 64>; + interrupts = <0 150 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp2_clks R8A7793_CLK_SCIFB2>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + scif0: serial at e6e60000 { compatible = "renesas,scif-r8a7793", "renesas,scif"; reg = <0 0xe6e60000 0 64>; @@ -318,6 +408,76 @@ status = "disabled"; }; + scif2: serial at e6e58000 { + compatible = "renesas,scif-r8a7793", "renesas,scif"; + reg = <0 0xe6e58000 0 64>; + interrupts = <0 22 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_SCIF2>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scif3: serial at e6ea8000 { + compatible = "renesas,scif-r8a7793", "renesas,scif"; + reg = <0 0xe6ea8000 0 64>; + interrupts = <0 23 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_SCIF3>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scif4: serial at e6ee0000 { + compatible = "renesas,scif-r8a7793", "renesas,scif"; + reg = <0 0xe6ee0000 0 64>; + interrupts = <0 24 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_SCIF4>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + scif5: serial at e6ee8000 { + compatible = "renesas,scif-r8a7793", "renesas,scif"; + reg = <0 0xe6ee8000 0 64>; + interrupts = <0 25 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_SCIF5>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + hscif0: serial at e62c0000 { + compatible = "renesas,hscif-r8a7793", "renesas,hscif"; + reg = <0 0xe62c0000 0 96>; + interrupts = <0 154 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_HSCIF0>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + hscif1: serial at e62c8000 { + compatible = "renesas,hscif-r8a7793", "renesas,hscif"; + reg = <0 0xe62c8000 0 96>; + interrupts = <0 155 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_HSCIF1>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + + hscif2: serial at e62d0000 { + compatible = "renesas,hscif-r8a7793", "renesas,hscif"; + reg = <0 0xe62d0000 0 96>; + interrupts = <0 21 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp7_clks R8A7793_CLK_HSCIF2>; + clock-names = "sci_ick"; + power-domains = <&cpg_clocks>; + status = "disabled"; + }; + ether: ethernet at ee700000 { compatible = "renesas,ether-r8a7793"; reg = <0 0xee700000 0 0x400>; @@ -530,12 +690,17 @@ mstp2_clks: mstp2_clks at e6150138 { compatible = "renesas,r8a7793-mstp-clocks", "renesas,cpg-mstp-clocks"; reg = <0 0xe6150138 0 4>, <0 0xe6150040 0 4>; - clocks = <&zs_clk>, <&zs_clk>; + clocks = <&mp_clk>, <&mp_clk>, <&mp_clk>, <&mp_clk>, + <&mp_clk>, <&mp_clk>, <&zs_clk>, <&zs_clk>; #clock-cells = <1>; clock-indices = < + R8A7793_CLK_SCIFA2 R8A7793_CLK_SCIFA1 R8A7793_CLK_SCIFA0 + R8A7793_CLK_SCIFB0 R8A7793_CLK_SCIFB1 R8A7793_CLK_SCIFB2 R8A7793_CLK_SYS_DMAC1 R8A7793_CLK_SYS_DMAC0 >; - clock-output-names = "sys-dmac1", "sys-dmac0"; + clock-output-names = + "scifa2", "scifa1", "scifa0", "scifb0", + "scifb1", "scifb2", "sys-dmac1", "sys-dmac0"; }; mstp3_clks: mstp3_clks at e615013c { compatible = "renesas,r8a7793-mstp-clocks", @@ -634,6 +799,16 @@ "gpio3", "gpio2", "gpio1", "gpio0", "qspi_mod"; }; + mstp11_clks: mstp11_clks at e615099c { + compatible = "renesas,r8a7793-mstp-clocks", "renesas,cpg-mstp-clocks"; + reg = <0 0xe615099c 0 4>, <0 0xe61509ac 0 4>; + clocks = <&mp_clk>, <&mp_clk>, <&mp_clk>; + #clock-cells = <1>; + clock-indices = < + R8A7793_CLK_SCIFA3 R8A7793_CLK_SCIFA4 R8A7793_CLK_SCIFA5 + >; + clock-output-names = "scifa3", "scifa4", "scifa5"; + }; }; ipmmu_sy0: mmu at e6280000 { -- 2.1.4 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH 1/2] ARM: shmobile: r8a7793: Add missing serial devices to DT 2015-11-27 1:52 ` Simon Horman @ 2015-12-09 10:42 ` Geert Uytterhoeven -1 siblings, 0 replies; 18+ messages in thread From: Geert Uytterhoeven @ 2015-12-09 10:42 UTC (permalink / raw) To: linux-arm-kernel On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman <horms+renesas@verge.net.au> wrote: > Instantiate all serial devices in r8a7793 device tree > and set them as disabled by default. > > Based on similar work for the r8a7791 by Laurent Pinchart. > > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 1/2] ARM: shmobile: r8a7793: Add missing serial devices to DT @ 2015-12-09 10:42 ` Geert Uytterhoeven 0 siblings, 0 replies; 18+ messages in thread From: Geert Uytterhoeven @ 2015-12-09 10:42 UTC (permalink / raw) To: linux-arm-kernel On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman <horms+renesas@verge.net.au> wrote: > Instantiate all serial devices in r8a7793 device tree > and set them as disabled by default. > > Based on similar work for the r8a7791 by Laurent Pinchart. > > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/2] ARM: shmobile: r8a7793: Add missing serial devices to DT 2015-12-09 10:42 ` Geert Uytterhoeven @ 2015-12-10 4:48 ` Simon Horman -1 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-12-10 4:48 UTC (permalink / raw) To: linux-arm-kernel On Wed, Dec 09, 2015 at 11:42:35AM +0100, Geert Uytterhoeven wrote: > On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman > <horms+renesas@verge.net.au> wrote: > > Instantiate all serial devices in r8a7793 device tree > > and set them as disabled by default. > > > > Based on similar work for the r8a7791 by Laurent Pinchart. > > > > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> Thanks, I have queued this up. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 1/2] ARM: shmobile: r8a7793: Add missing serial devices to DT @ 2015-12-10 4:48 ` Simon Horman 0 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-12-10 4:48 UTC (permalink / raw) To: linux-arm-kernel On Wed, Dec 09, 2015 at 11:42:35AM +0100, Geert Uytterhoeven wrote: > On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman > <horms+renesas@verge.net.au> wrote: > > Instantiate all serial devices in r8a7793 device tree > > and set them as disabled by default. > > > > Based on similar work for the r8a7791 by Laurent Pinchart. > > > > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> Thanks, I have queued this up. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports 2015-11-27 1:52 ` Simon Horman @ 2015-11-27 1:52 ` Simon Horman -1 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-11-27 1:52 UTC (permalink / raw) To: linux-arm-kernel Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. Based on similar work for the r8a7791 by Geert Uytterhoeven. Cc: Geert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ 1 file changed, 36 insertions(+) diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi index 3ac2ec0414a0..8d5cbdfc3d5d 100644 --- a/arch/arm/boot/dts/r8a7793.dtsi +++ b/arch/arm/boot/dts/r8a7793.dtsi @@ -304,6 +304,8 @@ interrupts = <0 144 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFA0>; clock-names = "sci_ick"; + dmas = <&dmac0 0x21>, <&dmac0 0x22>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -314,6 +316,8 @@ interrupts = <0 145 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFA1>; clock-names = "sci_ick"; + dmas = <&dmac0 0x25>, <&dmac0 0x26>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -324,6 +328,8 @@ interrupts = <0 151 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFA2>; clock-names = "sci_ick"; + dmas = <&dmac0 0x27>, <&dmac0 0x28>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -334,6 +340,8 @@ interrupts = <0 29 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp11_clks R8A7793_CLK_SCIFA3>; clock-names = "sci_ick"; + dmas = <&dmac0 0x1b>, <&dmac0 0x1c>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -344,6 +352,8 @@ interrupts = <0 30 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp11_clks R8A7793_CLK_SCIFA4>; clock-names = "sci_ick"; + dmas = <&dmac0 0x1f>, <&dmac0 0x20>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -354,6 +364,8 @@ interrupts = <0 31 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp11_clks R8A7793_CLK_SCIFA5>; clock-names = "sci_ick"; + dmas = <&dmac0 0x23>, <&dmac0 0x24>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -364,6 +376,8 @@ interrupts = <0 148 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFB0>; clock-names = "sci_ick"; + dmas = <&dmac0 0x3d>, <&dmac0 0x3e>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -374,6 +388,8 @@ interrupts = <0 149 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFB1>; clock-names = "sci_ick"; + dmas = <&dmac0 0x19>, <&dmac0 0x1a>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -384,6 +400,8 @@ interrupts = <0 150 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFB2>; clock-names = "sci_ick"; + dmas = <&dmac0 0x1d>, <&dmac0 0x1e>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -394,6 +412,8 @@ interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; clock-names = "sci_ick"; + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -404,6 +424,8 @@ interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; clock-names = "sci_ick"; + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -414,6 +436,8 @@ interrupts = <0 22 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF2>; clock-names = "sci_ick"; + dmas = <&dmac0 0x2b>, <&dmac0 0x2c>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -424,6 +448,8 @@ interrupts = <0 23 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF3>; clock-names = "sci_ick"; + dmas = <&dmac0 0x2f>, <&dmac0 0x30>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -434,6 +460,8 @@ interrupts = <0 24 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF4>; clock-names = "sci_ick"; + dmas = <&dmac0 0xfb>, <&dmac0 0xfc>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -444,6 +472,8 @@ interrupts = <0 25 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF5>; clock-names = "sci_ick"; + dmas = <&dmac0 0xfd>, <&dmac0 0xfe>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -454,6 +484,8 @@ interrupts = <0 154 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_HSCIF0>; clock-names = "sci_ick"; + dmas = <&dmac0 0x39>, <&dmac0 0x3a>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -464,6 +496,8 @@ interrupts = <0 155 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_HSCIF1>; clock-names = "sci_ick"; + dmas = <&dmac0 0x4d>, <&dmac0 0x4e>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -474,6 +508,8 @@ interrupts = <0 21 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_HSCIF2>; clock-names = "sci_ick"; + dmas = <&dmac0 0x3b>, <&dmac0 0x3c>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; -- 2.1.4 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports @ 2015-11-27 1:52 ` Simon Horman 0 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-11-27 1:52 UTC (permalink / raw) To: linux-arm-kernel Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. Based on similar work for the r8a7791 by Geert Uytterhoeven. Cc: Geert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ 1 file changed, 36 insertions(+) diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi index 3ac2ec0414a0..8d5cbdfc3d5d 100644 --- a/arch/arm/boot/dts/r8a7793.dtsi +++ b/arch/arm/boot/dts/r8a7793.dtsi @@ -304,6 +304,8 @@ interrupts = <0 144 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFA0>; clock-names = "sci_ick"; + dmas = <&dmac0 0x21>, <&dmac0 0x22>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -314,6 +316,8 @@ interrupts = <0 145 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFA1>; clock-names = "sci_ick"; + dmas = <&dmac0 0x25>, <&dmac0 0x26>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -324,6 +328,8 @@ interrupts = <0 151 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFA2>; clock-names = "sci_ick"; + dmas = <&dmac0 0x27>, <&dmac0 0x28>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -334,6 +340,8 @@ interrupts = <0 29 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp11_clks R8A7793_CLK_SCIFA3>; clock-names = "sci_ick"; + dmas = <&dmac0 0x1b>, <&dmac0 0x1c>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -344,6 +352,8 @@ interrupts = <0 30 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp11_clks R8A7793_CLK_SCIFA4>; clock-names = "sci_ick"; + dmas = <&dmac0 0x1f>, <&dmac0 0x20>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -354,6 +364,8 @@ interrupts = <0 31 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp11_clks R8A7793_CLK_SCIFA5>; clock-names = "sci_ick"; + dmas = <&dmac0 0x23>, <&dmac0 0x24>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -364,6 +376,8 @@ interrupts = <0 148 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFB0>; clock-names = "sci_ick"; + dmas = <&dmac0 0x3d>, <&dmac0 0x3e>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -374,6 +388,8 @@ interrupts = <0 149 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFB1>; clock-names = "sci_ick"; + dmas = <&dmac0 0x19>, <&dmac0 0x1a>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -384,6 +400,8 @@ interrupts = <0 150 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp2_clks R8A7793_CLK_SCIFB2>; clock-names = "sci_ick"; + dmas = <&dmac0 0x1d>, <&dmac0 0x1e>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -394,6 +412,8 @@ interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; clock-names = "sci_ick"; + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -404,6 +424,8 @@ interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; clock-names = "sci_ick"; + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -414,6 +436,8 @@ interrupts = <0 22 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF2>; clock-names = "sci_ick"; + dmas = <&dmac0 0x2b>, <&dmac0 0x2c>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -424,6 +448,8 @@ interrupts = <0 23 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF3>; clock-names = "sci_ick"; + dmas = <&dmac0 0x2f>, <&dmac0 0x30>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -434,6 +460,8 @@ interrupts = <0 24 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF4>; clock-names = "sci_ick"; + dmas = <&dmac0 0xfb>, <&dmac0 0xfc>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -444,6 +472,8 @@ interrupts = <0 25 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_SCIF5>; clock-names = "sci_ick"; + dmas = <&dmac0 0xfd>, <&dmac0 0xfe>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -454,6 +484,8 @@ interrupts = <0 154 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_HSCIF0>; clock-names = "sci_ick"; + dmas = <&dmac0 0x39>, <&dmac0 0x3a>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -464,6 +496,8 @@ interrupts = <0 155 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_HSCIF1>; clock-names = "sci_ick"; + dmas = <&dmac0 0x4d>, <&dmac0 0x4e>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; @@ -474,6 +508,8 @@ interrupts = <0 21 IRQ_TYPE_LEVEL_HIGH>; clocks = <&mstp7_clks R8A7793_CLK_HSCIF2>; clock-names = "sci_ick"; + dmas = <&dmac0 0x3b>, <&dmac0 0x3c>; + dma-names = "tx", "rx"; power-domains = <&cpg_clocks>; status = "disabled"; }; -- 2.1.4 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports 2015-11-27 1:52 ` Simon Horman @ 2015-12-09 10:47 ` Geert Uytterhoeven -1 siblings, 0 replies; 18+ messages in thread From: Geert Uytterhoeven @ 2015-12-09 10:47 UTC (permalink / raw) To: linux-arm-kernel Hi Simon, On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman <horms+renesas@verge.net.au> wrote: > Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. > > Based on similar work for the r8a7791 by Geert Uytterhoeven. > > Cc: Geert Uytterhoeven <geert+renesas@glider.be> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > --- > arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ > 1 file changed, 36 insertions(+) > > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi > index 3ac2ec0414a0..8d5cbdfc3d5d 100644 > --- a/arch/arm/boot/dts/r8a7793.dtsi > +++ b/arch/arm/boot/dts/r8a7793.dtsi > @@ -394,6 +412,8 @@ > interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; > clock-names = "sci_ick"; > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > + dma-names = "tx", "rx"; > power-domains = <&cpg_clocks>; > status = "disabled"; > }; > @@ -404,6 +424,8 @@ > interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; > clock-names = "sci_ick"; > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; This is duplicated from scif0, and should be: dmas = <&dmac0 0x2d>, <&dmac0 0x2e>; > + dma-names = "tx", "rx"; After sed -i -e 's/7791/779x/g' arch/arm/boot/dts/r8a7791.dtsi sed -i -e 's/7793/779x/g' arch/arm/boot/dts/r8a7793.dtsi arch/arm/boot/dts/r8a7791.dtsi and arch/arm/boot/dts/r8a7793.dtsi should be identical ;-) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports @ 2015-12-09 10:47 ` Geert Uytterhoeven 0 siblings, 0 replies; 18+ messages in thread From: Geert Uytterhoeven @ 2015-12-09 10:47 UTC (permalink / raw) To: linux-arm-kernel Hi Simon, On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman <horms+renesas@verge.net.au> wrote: > Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. > > Based on similar work for the r8a7791 by Geert Uytterhoeven. > > Cc: Geert Uytterhoeven <geert+renesas@glider.be> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > --- > arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ > 1 file changed, 36 insertions(+) > > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi > index 3ac2ec0414a0..8d5cbdfc3d5d 100644 > --- a/arch/arm/boot/dts/r8a7793.dtsi > +++ b/arch/arm/boot/dts/r8a7793.dtsi > @@ -394,6 +412,8 @@ > interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; > clock-names = "sci_ick"; > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > + dma-names = "tx", "rx"; > power-domains = <&cpg_clocks>; > status = "disabled"; > }; > @@ -404,6 +424,8 @@ > interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; > clock-names = "sci_ick"; > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; This is duplicated from scif0, and should be: dmas = <&dmac0 0x2d>, <&dmac0 0x2e>; > + dma-names = "tx", "rx"; After sed -i -e 's/7791/779x/g' arch/arm/boot/dts/r8a7791.dtsi sed -i -e 's/7793/779x/g' arch/arm/boot/dts/r8a7793.dtsi arch/arm/boot/dts/r8a7791.dtsi and arch/arm/boot/dts/r8a7793.dtsi should be identical ;-) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports 2015-12-09 10:47 ` Geert Uytterhoeven @ 2015-12-10 4:54 ` Simon Horman -1 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-12-10 4:54 UTC (permalink / raw) To: linux-arm-kernel On Wed, Dec 09, 2015 at 11:47:36AM +0100, Geert Uytterhoeven wrote: > Hi Simon, > > On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman > <horms+renesas@verge.net.au> wrote: > > Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. > > > > Based on similar work for the r8a7791 by Geert Uytterhoeven. > > > > Cc: Geert Uytterhoeven <geert+renesas@glider.be> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > > --- > > arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ > > 1 file changed, 36 insertions(+) > > > > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi > > index 3ac2ec0414a0..8d5cbdfc3d5d 100644 > > --- a/arch/arm/boot/dts/r8a7793.dtsi > > +++ b/arch/arm/boot/dts/r8a7793.dtsi > > > @@ -394,6 +412,8 @@ > > interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; > > clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; > > clock-names = "sci_ick"; > > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > > + dma-names = "tx", "rx"; > > power-domains = <&cpg_clocks>; > > status = "disabled"; > > }; > > @@ -404,6 +424,8 @@ > > interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; > > clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; > > clock-names = "sci_ick"; > > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > > This is duplicated from scif0, and should be: > > dmas = <&dmac0 0x2d>, <&dmac0 0x2e>; > > > + dma-names = "tx", "rx"; > > After > > sed -i -e 's/7791/779x/g' arch/arm/boot/dts/r8a7791.dtsi > sed -i -e 's/7793/779x/g' arch/arm/boot/dts/r8a7793.dtsi > > arch/arm/boot/dts/r8a7791.dtsi and arch/arm/boot/dts/r8a7793.dtsi should > be identical ;-) Thanks, I have fixed that in v2. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports @ 2015-12-10 4:54 ` Simon Horman 0 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-12-10 4:54 UTC (permalink / raw) To: linux-arm-kernel On Wed, Dec 09, 2015 at 11:47:36AM +0100, Geert Uytterhoeven wrote: > Hi Simon, > > On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman > <horms+renesas@verge.net.au> wrote: > > Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. > > > > Based on similar work for the r8a7791 by Geert Uytterhoeven. > > > > Cc: Geert Uytterhoeven <geert+renesas@glider.be> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > > --- > > arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ > > 1 file changed, 36 insertions(+) > > > > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi > > index 3ac2ec0414a0..8d5cbdfc3d5d 100644 > > --- a/arch/arm/boot/dts/r8a7793.dtsi > > +++ b/arch/arm/boot/dts/r8a7793.dtsi > > > @@ -394,6 +412,8 @@ > > interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; > > clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; > > clock-names = "sci_ick"; > > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > > + dma-names = "tx", "rx"; > > power-domains = <&cpg_clocks>; > > status = "disabled"; > > }; > > @@ -404,6 +424,8 @@ > > interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; > > clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; > > clock-names = "sci_ick"; > > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > > This is duplicated from scif0, and should be: > > dmas = <&dmac0 0x2d>, <&dmac0 0x2e>; > > > + dma-names = "tx", "rx"; > > After > > sed -i -e 's/7791/779x/g' arch/arm/boot/dts/r8a7791.dtsi > sed -i -e 's/7793/779x/g' arch/arm/boot/dts/r8a7793.dtsi > > arch/arm/boot/dts/r8a7791.dtsi and arch/arm/boot/dts/r8a7793.dtsi should > be identical ;-) Thanks, I have fixed that in v2. ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports 2015-12-10 4:54 ` Simon Horman @ 2015-12-10 10:14 ` Geert Uytterhoeven -1 siblings, 0 replies; 18+ messages in thread From: Geert Uytterhoeven @ 2015-12-10 10:14 UTC (permalink / raw) To: linux-arm-kernel On Thu, Dec 10, 2015 at 5:54 AM, Simon Horman <horms@verge.net.au> wrote: > On Wed, Dec 09, 2015 at 11:47:36AM +0100, Geert Uytterhoeven wrote: >> Hi Simon, >> >> On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman >> <horms+renesas@verge.net.au> wrote: >> > Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. >> > >> > Based on similar work for the r8a7791 by Geert Uytterhoeven. >> > >> > Cc: Geert Uytterhoeven <geert+renesas@glider.be> >> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> >> > --- >> > arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ >> > 1 file changed, 36 insertions(+) >> > >> > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi >> > index 3ac2ec0414a0..8d5cbdfc3d5d 100644 >> > --- a/arch/arm/boot/dts/r8a7793.dtsi >> > +++ b/arch/arm/boot/dts/r8a7793.dtsi >> >> > @@ -394,6 +412,8 @@ >> > interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; >> > clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; >> > clock-names = "sci_ick"; >> > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; >> > + dma-names = "tx", "rx"; >> > power-domains = <&cpg_clocks>; >> > status = "disabled"; >> > }; >> > @@ -404,6 +424,8 @@ >> > interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; >> > clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; >> > clock-names = "sci_ick"; >> > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; >> >> This is duplicated from scif0, and should be: >> >> dmas = <&dmac0 0x2d>, <&dmac0 0x2e>; >> >> > + dma-names = "tx", "rx"; >> >> After >> >> sed -i -e 's/7791/779x/g' arch/arm/boot/dts/r8a7791.dtsi >> sed -i -e 's/7793/779x/g' arch/arm/boot/dts/r8a7793.dtsi >> >> arch/arm/boot/dts/r8a7791.dtsi and arch/arm/boot/dts/r8a7793.dtsi should >> be identical ;-) > > Thanks, I have fixed that in v2. Haven't seen that patch, but feel free to add my Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> after having corrected the scif1 DMA names. The sooner it arrives in renesas-devel, the better (I have patches depending on it). Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports @ 2015-12-10 10:14 ` Geert Uytterhoeven 0 siblings, 0 replies; 18+ messages in thread From: Geert Uytterhoeven @ 2015-12-10 10:14 UTC (permalink / raw) To: linux-arm-kernel On Thu, Dec 10, 2015 at 5:54 AM, Simon Horman <horms@verge.net.au> wrote: > On Wed, Dec 09, 2015 at 11:47:36AM +0100, Geert Uytterhoeven wrote: >> Hi Simon, >> >> On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman >> <horms+renesas@verge.net.au> wrote: >> > Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. >> > >> > Based on similar work for the r8a7791 by Geert Uytterhoeven. >> > >> > Cc: Geert Uytterhoeven <geert+renesas@glider.be> >> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> >> > --- >> > arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ >> > 1 file changed, 36 insertions(+) >> > >> > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi >> > index 3ac2ec0414a0..8d5cbdfc3d5d 100644 >> > --- a/arch/arm/boot/dts/r8a7793.dtsi >> > +++ b/arch/arm/boot/dts/r8a7793.dtsi >> >> > @@ -394,6 +412,8 @@ >> > interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; >> > clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; >> > clock-names = "sci_ick"; >> > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; >> > + dma-names = "tx", "rx"; >> > power-domains = <&cpg_clocks>; >> > status = "disabled"; >> > }; >> > @@ -404,6 +424,8 @@ >> > interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; >> > clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; >> > clock-names = "sci_ick"; >> > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; >> >> This is duplicated from scif0, and should be: >> >> dmas = <&dmac0 0x2d>, <&dmac0 0x2e>; >> >> > + dma-names = "tx", "rx"; >> >> After >> >> sed -i -e 's/7791/779x/g' arch/arm/boot/dts/r8a7791.dtsi >> sed -i -e 's/7793/779x/g' arch/arm/boot/dts/r8a7793.dtsi >> >> arch/arm/boot/dts/r8a7791.dtsi and arch/arm/boot/dts/r8a7793.dtsi should >> be identical ;-) > > Thanks, I have fixed that in v2. Haven't seen that patch, but feel free to add my Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> after having corrected the scif1 DMA names. The sooner it arrives in renesas-devel, the better (I have patches depending on it). Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports 2015-12-10 10:14 ` Geert Uytterhoeven @ 2015-12-11 1:08 ` Simon Horman -1 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-12-11 1:08 UTC (permalink / raw) To: linux-arm-kernel On Thu, Dec 10, 2015 at 11:14:47AM +0100, Geert Uytterhoeven wrote: > On Thu, Dec 10, 2015 at 5:54 AM, Simon Horman <horms@verge.net.au> wrote: > > On Wed, Dec 09, 2015 at 11:47:36AM +0100, Geert Uytterhoeven wrote: > >> Hi Simon, > >> > >> On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman > >> <horms+renesas@verge.net.au> wrote: > >> > Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. > >> > > >> > Based on similar work for the r8a7791 by Geert Uytterhoeven. > >> > > >> > Cc: Geert Uytterhoeven <geert+renesas@glider.be> > >> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > >> > --- > >> > arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ > >> > 1 file changed, 36 insertions(+) > >> > > >> > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi > >> > index 3ac2ec0414a0..8d5cbdfc3d5d 100644 > >> > --- a/arch/arm/boot/dts/r8a7793.dtsi > >> > +++ b/arch/arm/boot/dts/r8a7793.dtsi > >> > >> > @@ -394,6 +412,8 @@ > >> > interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; > >> > clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; > >> > clock-names = "sci_ick"; > >> > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > >> > + dma-names = "tx", "rx"; > >> > power-domains = <&cpg_clocks>; > >> > status = "disabled"; > >> > }; > >> > @@ -404,6 +424,8 @@ > >> > interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; > >> > clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; > >> > clock-names = "sci_ick"; > >> > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > >> > >> This is duplicated from scif0, and should be: > >> > >> dmas = <&dmac0 0x2d>, <&dmac0 0x2e>; > >> > >> > + dma-names = "tx", "rx"; > >> > >> After > >> > >> sed -i -e 's/7791/779x/g' arch/arm/boot/dts/r8a7791.dtsi > >> sed -i -e 's/7793/779x/g' arch/arm/boot/dts/r8a7793.dtsi > >> > >> arch/arm/boot/dts/r8a7791.dtsi and arch/arm/boot/dts/r8a7793.dtsi should > >> be identical ;-) > > > > Thanks, I have fixed that in v2. > > Haven't seen that patch, but feel free to add my > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> > after having corrected the scif1 DMA names. > The sooner it arrives in renesas-devel, the better (I have patches depending > on it). Sorry for being a bit slow there. I have sent it out with your Ack and plan to queue it up a bit later today. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports @ 2015-12-11 1:08 ` Simon Horman 0 siblings, 0 replies; 18+ messages in thread From: Simon Horman @ 2015-12-11 1:08 UTC (permalink / raw) To: linux-arm-kernel On Thu, Dec 10, 2015 at 11:14:47AM +0100, Geert Uytterhoeven wrote: > On Thu, Dec 10, 2015 at 5:54 AM, Simon Horman <horms@verge.net.au> wrote: > > On Wed, Dec 09, 2015 at 11:47:36AM +0100, Geert Uytterhoeven wrote: > >> Hi Simon, > >> > >> On Fri, Nov 27, 2015 at 2:52 AM, Simon Horman > >> <horms+renesas@verge.net.au> wrote: > >> > Add DMA properties to all SCIF, SCIFA, SCIFB, and HSCIF device nodes. > >> > > >> > Based on similar work for the r8a7791 by Geert Uytterhoeven. > >> > > >> > Cc: Geert Uytterhoeven <geert+renesas@glider.be> > >> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > >> > --- > >> > arch/arm/boot/dts/r8a7793.dtsi | 36 ++++++++++++++++++++++++++++++++++++ > >> > 1 file changed, 36 insertions(+) > >> > > >> > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi > >> > index 3ac2ec0414a0..8d5cbdfc3d5d 100644 > >> > --- a/arch/arm/boot/dts/r8a7793.dtsi > >> > +++ b/arch/arm/boot/dts/r8a7793.dtsi > >> > >> > @@ -394,6 +412,8 @@ > >> > interrupts = <0 152 IRQ_TYPE_LEVEL_HIGH>; > >> > clocks = <&mstp7_clks R8A7793_CLK_SCIF0>; > >> > clock-names = "sci_ick"; > >> > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > >> > + dma-names = "tx", "rx"; > >> > power-domains = <&cpg_clocks>; > >> > status = "disabled"; > >> > }; > >> > @@ -404,6 +424,8 @@ > >> > interrupts = <0 153 IRQ_TYPE_LEVEL_HIGH>; > >> > clocks = <&mstp7_clks R8A7793_CLK_SCIF1>; > >> > clock-names = "sci_ick"; > >> > + dmas = <&dmac0 0x29>, <&dmac0 0x2a>; > >> > >> This is duplicated from scif0, and should be: > >> > >> dmas = <&dmac0 0x2d>, <&dmac0 0x2e>; > >> > >> > + dma-names = "tx", "rx"; > >> > >> After > >> > >> sed -i -e 's/7791/779x/g' arch/arm/boot/dts/r8a7791.dtsi > >> sed -i -e 's/7793/779x/g' arch/arm/boot/dts/r8a7793.dtsi > >> > >> arch/arm/boot/dts/r8a7791.dtsi and arch/arm/boot/dts/r8a7793.dtsi should > >> be identical ;-) > > > > Thanks, I have fixed that in v2. > > Haven't seen that patch, but feel free to add my > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> > after having corrected the scif1 DMA names. > The sooner it arrives in renesas-devel, the better (I have patches depending > on it). Sorry for being a bit slow there. I have sent it out with your Ack and plan to queue it up a bit later today. ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2015-12-11 1:08 UTC | newest] Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-11-27 1:52 [PATCH 0/2] ARM: shmobile: r8a7793: Increase scif support Simon Horman 2015-11-27 1:52 ` Simon Horman 2015-11-27 1:52 ` [PATCH 1/2] ARM: shmobile: r8a7793: Add missing serial devices to DT Simon Horman 2015-11-27 1:52 ` Simon Horman 2015-12-09 10:42 ` Geert Uytterhoeven 2015-12-09 10:42 ` Geert Uytterhoeven 2015-12-10 4:48 ` Simon Horman 2015-12-10 4:48 ` Simon Horman 2015-11-27 1:52 ` [PATCH 2/2] ARM: shmobile: r8a7793: Describe DMA for the serial ports Simon Horman 2015-11-27 1:52 ` Simon Horman 2015-12-09 10:47 ` Geert Uytterhoeven 2015-12-09 10:47 ` Geert Uytterhoeven 2015-12-10 4:54 ` Simon Horman 2015-12-10 4:54 ` Simon Horman 2015-12-10 10:14 ` Geert Uytterhoeven 2015-12-10 10:14 ` Geert Uytterhoeven 2015-12-11 1:08 ` Simon Horman 2015-12-11 1:08 ` Simon Horman
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.