* [PATCH] ARM: dts: rockchip: Add DMA tx and rx channels in UART interfaces for RK3288 SoCs
@ 2015-06-16 17:19 Romain Perier
[not found] ` <1434475166-30338-1-git-send-email-romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
0 siblings, 1 reply; 5+ messages in thread
From: Romain Perier @ 2015-06-16 17:19 UTC (permalink / raw)
To: heiko-4mtYJXux2i+zQB+pC5nmwQ
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA
This enable DMA capability for the corresponding UART interfaces, also it removes
warning "ttyS%d - Failed to request dma" from kernel logs.
Signed-off-by: Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
Changes since v1 ("[PATCH] ARM: dts: rockchip: Add dmac_bus rx and tx for uart2"):
- Add DMA tx and rx channels for all UART interfaces
- Rename commit summary
arch/arm/boot/dts/rk3288.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi
index 22316d0..1996452 100644
--- a/arch/arm/boot/dts/rk3288.dtsi
+++ b/arch/arm/boot/dts/rk3288.dtsi
@@ -359,6 +359,8 @@
clock-names = "baudclk", "apb_pclk";
pinctrl-names = "default";
pinctrl-0 = <&uart0_xfer>;
+ dmas = <&dmac_peri 1>, <&dmac_peri 2>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -372,6 +374,8 @@
clock-names = "baudclk", "apb_pclk";
pinctrl-names = "default";
pinctrl-0 = <&uart1_xfer>;
+ dmas = <&dmac_peri 3>, <&dmac_peri 4>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -385,6 +389,8 @@
clock-names = "baudclk", "apb_pclk";
pinctrl-names = "default";
pinctrl-0 = <&uart2_xfer>;
+ dmas = <&dmac_bus_s 4>, <&dmac_bus_s 5>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -398,6 +404,8 @@
clock-names = "baudclk", "apb_pclk";
pinctrl-names = "default";
pinctrl-0 = <&uart3_xfer>;
+ dmas = <&dmac_peri 7>, <&dmac_peri 8>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -411,6 +419,8 @@
clock-names = "baudclk", "apb_pclk";
pinctrl-names = "default";
pinctrl-0 = <&uart4_xfer>;
+ dmas = <&dmac_peri 9>, <&dmac_peri 10>;
+ dma-names = "tx", "rx";
status = "disabled";
};
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 5+ messages in thread[parent not found: <1434475166-30338-1-git-send-email-romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]
* Re: [PATCH] ARM: dts: rockchip: Add DMA tx and rx channels in UART interfaces for RK3288 SoCs [not found] ` <1434475166-30338-1-git-send-email-romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> @ 2015-06-23 6:53 ` Romain Perier [not found] ` <CABgxDo+8MafT8hOzpG=gjju=S7bVtNBOKxmi_JPHv1bDZSGeCQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 5+ messages in thread From: Romain Perier @ 2015-06-23 6:53 UTC (permalink / raw) To: Heiko Stübner Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, open list:ARM/Rockchip SoC..., devicetree ping :) 2015-06-16 19:19 GMT+02:00 Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>: > This enable DMA capability for the corresponding UART interfaces, also it removes > warning "ttyS%d - Failed to request dma" from kernel logs. > > Signed-off-by: Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > --- > > Changes since v1 ("[PATCH] ARM: dts: rockchip: Add dmac_bus rx and tx for uart2"): > - Add DMA tx and rx channels for all UART interfaces > - Rename commit summary > > arch/arm/boot/dts/rk3288.dtsi | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi > index 22316d0..1996452 100644 > --- a/arch/arm/boot/dts/rk3288.dtsi > +++ b/arch/arm/boot/dts/rk3288.dtsi > @@ -359,6 +359,8 @@ > clock-names = "baudclk", "apb_pclk"; > pinctrl-names = "default"; > pinctrl-0 = <&uart0_xfer>; > + dmas = <&dmac_peri 1>, <&dmac_peri 2>; > + dma-names = "tx", "rx"; > status = "disabled"; > }; > > @@ -372,6 +374,8 @@ > clock-names = "baudclk", "apb_pclk"; > pinctrl-names = "default"; > pinctrl-0 = <&uart1_xfer>; > + dmas = <&dmac_peri 3>, <&dmac_peri 4>; > + dma-names = "tx", "rx"; > status = "disabled"; > }; > > @@ -385,6 +389,8 @@ > clock-names = "baudclk", "apb_pclk"; > pinctrl-names = "default"; > pinctrl-0 = <&uart2_xfer>; > + dmas = <&dmac_bus_s 4>, <&dmac_bus_s 5>; > + dma-names = "tx", "rx"; > status = "disabled"; > }; > > @@ -398,6 +404,8 @@ > clock-names = "baudclk", "apb_pclk"; > pinctrl-names = "default"; > pinctrl-0 = <&uart3_xfer>; > + dmas = <&dmac_peri 7>, <&dmac_peri 8>; > + dma-names = "tx", "rx"; > status = "disabled"; > }; > > @@ -411,6 +419,8 @@ > clock-names = "baudclk", "apb_pclk"; > pinctrl-names = "default"; > pinctrl-0 = <&uart4_xfer>; > + dmas = <&dmac_peri 9>, <&dmac_peri 10>; > + dma-names = "tx", "rx"; > status = "disabled"; > }; > > -- > 1.9.1 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <CABgxDo+8MafT8hOzpG=gjju=S7bVtNBOKxmi_JPHv1bDZSGeCQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH] ARM: dts: rockchip: Add DMA tx and rx channels in UART interfaces for RK3288 SoCs [not found] ` <CABgxDo+8MafT8hOzpG=gjju=S7bVtNBOKxmi_JPHv1bDZSGeCQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2015-06-23 11:09 ` Heiko Stübner 2015-06-23 11:55 ` Romain Perier 2015-06-23 21:11 ` Sonny Rao 0 siblings, 2 replies; 5+ messages in thread From: Heiko Stübner @ 2015-06-23 11:09 UTC (permalink / raw) To: Romain Perier Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, open list:ARM/Rockchip SoC..., devicetree Hi Romain, Am Dienstag, 23. Juni 2015, 08:53:17 schrieb Romain Perier: > ping :) I haven't forgotten this patch :-) I just haven't fully made up my mind concerning the bug in the pl330 implementation on Rockchip SoCs - aka if we'll make it worse by also enabling dma on the uarts. So far the problem reports I read only mentioned degraded functionality like popping sounds in audio, not full system failures or freezes. So I tend to include the patch and see what happens ;-) Heiko > 2015-06-16 19:19 GMT+02:00 Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>: > > This enable DMA capability for the corresponding UART interfaces, also it > > removes warning "ttyS%d - Failed to request dma" from kernel logs. > > > > Signed-off-by: Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > > --- > > > > Changes since v1 ("[PATCH] ARM: dts: rockchip: Add dmac_bus rx and tx for uart2"): > > - Add DMA tx and rx channels for all UART interfaces > > - Rename commit summary > > > > arch/arm/boot/dts/rk3288.dtsi | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi > > index 22316d0..1996452 100644 > > --- a/arch/arm/boot/dts/rk3288.dtsi > > +++ b/arch/arm/boot/dts/rk3288.dtsi > > @@ -359,6 +359,8 @@ > > > > clock-names = "baudclk", "apb_pclk"; > > pinctrl-names = "default"; > > pinctrl-0 = <&uart0_xfer>; > > > > + dmas = <&dmac_peri 1>, <&dmac_peri 2>; > > + dma-names = "tx", "rx"; > > > > status = "disabled"; > > > > }; > > > > @@ -372,6 +374,8 @@ > > > > clock-names = "baudclk", "apb_pclk"; > > pinctrl-names = "default"; > > pinctrl-0 = <&uart1_xfer>; > > > > + dmas = <&dmac_peri 3>, <&dmac_peri 4>; > > + dma-names = "tx", "rx"; > > > > status = "disabled"; > > > > }; > > > > @@ -385,6 +389,8 @@ > > > > clock-names = "baudclk", "apb_pclk"; > > pinctrl-names = "default"; > > pinctrl-0 = <&uart2_xfer>; > > > > + dmas = <&dmac_bus_s 4>, <&dmac_bus_s 5>; > > + dma-names = "tx", "rx"; > > > > status = "disabled"; > > > > }; > > > > @@ -398,6 +404,8 @@ > > > > clock-names = "baudclk", "apb_pclk"; > > pinctrl-names = "default"; > > pinctrl-0 = <&uart3_xfer>; > > > > + dmas = <&dmac_peri 7>, <&dmac_peri 8>; > > + dma-names = "tx", "rx"; > > > > status = "disabled"; > > > > }; > > > > @@ -411,6 +419,8 @@ > > > > clock-names = "baudclk", "apb_pclk"; > > pinctrl-names = "default"; > > pinctrl-0 = <&uart4_xfer>; > > > > + dmas = <&dmac_peri 9>, <&dmac_peri 10>; > > + dma-names = "tx", "rx"; > > > > status = "disabled"; > > > > }; > > > > -- > > 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: rockchip: Add DMA tx and rx channels in UART interfaces for RK3288 SoCs 2015-06-23 11:09 ` Heiko Stübner @ 2015-06-23 11:55 ` Romain Perier 2015-06-23 21:11 ` Sonny Rao 1 sibling, 0 replies; 5+ messages in thread From: Romain Perier @ 2015-06-23 11:55 UTC (permalink / raw) To: Heiko Stübner Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, open list:ARM/Rockchip SoC..., devicetree Hi Heiko, Oh sure, I completly forgot this issue ^^ , no problem. If you want some help feel free to ping on IRC. I did not notice regressions or issues, but my system was not under load, I will try to investigate and/or test with care. Have a nice day, Romain 2015-06-23 13:09 GMT+02:00 Heiko Stübner <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>: > Hi Romain, > > Am Dienstag, 23. Juni 2015, 08:53:17 schrieb Romain Perier: >> ping :) > > I haven't forgotten this patch :-) > > I just haven't fully made up my mind concerning the bug in the pl330 > implementation on Rockchip SoCs - aka if we'll make it worse by also enabling > dma on the uarts. > > So far the problem reports I read only mentioned degraded functionality like > popping sounds in audio, not full system failures or freezes. So I tend to > include the patch and see what happens ;-) > > > Heiko > > > >> 2015-06-16 19:19 GMT+02:00 Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>: >> > This enable DMA capability for the corresponding UART interfaces, also it >> > removes warning "ttyS%d - Failed to request dma" from kernel logs. >> > >> > Signed-off-by: Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> >> > --- >> > >> > Changes since v1 ("[PATCH] ARM: dts: rockchip: Add dmac_bus rx and tx for > uart2"): >> > - Add DMA tx and rx channels for all UART interfaces >> > - Rename commit summary >> > >> > arch/arm/boot/dts/rk3288.dtsi | 10 ++++++++++ >> > 1 file changed, 10 insertions(+) >> > >> > diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi >> > index 22316d0..1996452 100644 >> > --- a/arch/arm/boot/dts/rk3288.dtsi >> > +++ b/arch/arm/boot/dts/rk3288.dtsi >> > @@ -359,6 +359,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart0_xfer>; >> > >> > + dmas = <&dmac_peri 1>, <&dmac_peri 2>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > @@ -372,6 +374,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart1_xfer>; >> > >> > + dmas = <&dmac_peri 3>, <&dmac_peri 4>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > @@ -385,6 +389,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart2_xfer>; >> > >> > + dmas = <&dmac_bus_s 4>, <&dmac_bus_s 5>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > @@ -398,6 +404,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart3_xfer>; >> > >> > + dmas = <&dmac_peri 7>, <&dmac_peri 8>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > @@ -411,6 +419,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart4_xfer>; >> > >> > + dmas = <&dmac_peri 9>, <&dmac_peri 10>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > -- >> > 1.9.1 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: rockchip: Add DMA tx and rx channels in UART interfaces for RK3288 SoCs 2015-06-23 11:09 ` Heiko Stübner 2015-06-23 11:55 ` Romain Perier @ 2015-06-23 21:11 ` Sonny Rao 1 sibling, 0 replies; 5+ messages in thread From: Sonny Rao @ 2015-06-23 21:11 UTC (permalink / raw) To: Heiko Stübner Cc: Romain Perier, open list:ARM/Rockchip SoC..., linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree On Tue, Jun 23, 2015 at 4:09 AM, Heiko Stübner <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org> wrote: > Hi Romain, > > Am Dienstag, 23. Juni 2015, 08:53:17 schrieb Romain Perier: >> ping :) > > I haven't forgotten this patch :-) > > I just haven't fully made up my mind concerning the bug in the pl330 > implementation on Rockchip SoCs - aka if we'll make it worse by also enabling > dma on the uarts. > > So far the problem reports I read only mentioned degraded functionality like > popping sounds in audio, not full system failures or freezes. So I tend to > include the patch and see what happens ;-) You should look out for corrupted data -- like zeros or repeated data in the serial input or output. That was what was causing the pops -- a sudden sample of zeros, or buzzing from repeated samples. It may not happen at low speeds, but might happen if you tried doing something like PPP over the UART. > > > Heiko > > > >> 2015-06-16 19:19 GMT+02:00 Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>: >> > This enable DMA capability for the corresponding UART interfaces, also it >> > removes warning "ttyS%d - Failed to request dma" from kernel logs. >> > >> > Signed-off-by: Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> >> > --- >> > >> > Changes since v1 ("[PATCH] ARM: dts: rockchip: Add dmac_bus rx and tx for > uart2"): >> > - Add DMA tx and rx channels for all UART interfaces >> > - Rename commit summary >> > >> > arch/arm/boot/dts/rk3288.dtsi | 10 ++++++++++ >> > 1 file changed, 10 insertions(+) >> > >> > diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi >> > index 22316d0..1996452 100644 >> > --- a/arch/arm/boot/dts/rk3288.dtsi >> > +++ b/arch/arm/boot/dts/rk3288.dtsi >> > @@ -359,6 +359,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart0_xfer>; >> > >> > + dmas = <&dmac_peri 1>, <&dmac_peri 2>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > @@ -372,6 +374,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart1_xfer>; >> > >> > + dmas = <&dmac_peri 3>, <&dmac_peri 4>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > @@ -385,6 +389,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart2_xfer>; >> > >> > + dmas = <&dmac_bus_s 4>, <&dmac_bus_s 5>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > @@ -398,6 +404,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart3_xfer>; >> > >> > + dmas = <&dmac_peri 7>, <&dmac_peri 8>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > @@ -411,6 +419,8 @@ >> > >> > clock-names = "baudclk", "apb_pclk"; >> > pinctrl-names = "default"; >> > pinctrl-0 = <&uart4_xfer>; >> > >> > + dmas = <&dmac_peri 9>, <&dmac_peri 10>; >> > + dma-names = "tx", "rx"; >> > >> > status = "disabled"; >> > >> > }; >> > >> > -- >> > 1.9.1 > > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-06-23 21:11 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-16 17:19 [PATCH] ARM: dts: rockchip: Add DMA tx and rx channels in UART interfaces for RK3288 SoCs Romain Perier
[not found] ` <1434475166-30338-1-git-send-email-romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-06-23 6:53 ` Romain Perier
[not found] ` <CABgxDo+8MafT8hOzpG=gjju=S7bVtNBOKxmi_JPHv1bDZSGeCQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-06-23 11:09 ` Heiko Stübner
2015-06-23 11:55 ` Romain Perier
2015-06-23 21:11 ` Sonny Rao
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).