* [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-21 16:22 ` Katsuhiro Suzuki
0 siblings, 0 replies; 22+ messages in thread
From: Katsuhiro Suzuki @ 2019-03-21 16:22 UTC (permalink / raw)
To: Heiko Stuebner, linux-rockchip
Cc: linux-arm-kernel, linux-kernel, Katsuhiro Suzuki
Add UART dma channels as specified by the rk3399 TRM.
Refer:
RK3399 TRM V1.4: Chapter 12 DMA Controller
Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
---
Changes from v1:
- Add dma property for UART4
---
arch/arm64/boot/dts/rockchip/rk3399.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
index 0301e3e01b38..31e487202676 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
@@ -635,6 +635,8 @@
clocks = <&cru SCLK_UART0>, <&cru PCLK_UART0>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 99 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 0>, <&dmac_peri 1>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -648,6 +650,8 @@
clocks = <&cru SCLK_UART1>, <&cru PCLK_UART1>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 98 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 2>, <&dmac_peri 3>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -661,6 +665,8 @@
clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 100 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 4>, <&dmac_peri 5>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -674,6 +680,8 @@
clocks = <&cru SCLK_UART3>, <&cru PCLK_UART3>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 101 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 6>, <&dmac_peri 7>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -1152,6 +1160,8 @@
clocks = <&pmucru SCLK_UART4_PMU>, <&pmucru PCLK_UART4_PMU>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 8>, <&dmac_peri 9>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
--
2.20.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-21 16:22 ` Katsuhiro Suzuki
0 siblings, 0 replies; 22+ messages in thread
From: Katsuhiro Suzuki @ 2019-03-21 16:22 UTC (permalink / raw)
To: Heiko Stuebner, linux-rockchip
Cc: Katsuhiro Suzuki, linux-kernel, linux-arm-kernel
Add UART dma channels as specified by the rk3399 TRM.
Refer:
RK3399 TRM V1.4: Chapter 12 DMA Controller
Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
---
Changes from v1:
- Add dma property for UART4
---
arch/arm64/boot/dts/rockchip/rk3399.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
index 0301e3e01b38..31e487202676 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
@@ -635,6 +635,8 @@
clocks = <&cru SCLK_UART0>, <&cru PCLK_UART0>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 99 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 0>, <&dmac_peri 1>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -648,6 +650,8 @@
clocks = <&cru SCLK_UART1>, <&cru PCLK_UART1>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 98 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 2>, <&dmac_peri 3>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -661,6 +665,8 @@
clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 100 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 4>, <&dmac_peri 5>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -674,6 +680,8 @@
clocks = <&cru SCLK_UART3>, <&cru PCLK_UART3>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 101 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 6>, <&dmac_peri 7>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -1152,6 +1160,8 @@
clocks = <&pmucru SCLK_UART4_PMU>, <&pmucru PCLK_UART4_PMU>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 8>, <&dmac_peri 9>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
--
2.20.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-21 16:22 ` Katsuhiro Suzuki
0 siblings, 0 replies; 22+ messages in thread
From: Katsuhiro Suzuki @ 2019-03-21 16:22 UTC (permalink / raw)
To: Heiko Stuebner, linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Cc: Katsuhiro Suzuki, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Add UART dma channels as specified by the rk3399 TRM.
Refer:
RK3399 TRM V1.4: Chapter 12 DMA Controller
Signed-off-by: Katsuhiro Suzuki <katsuhiro-WKCMddiH/C4xsqv6Oivclw@public.gmane.org>
---
Changes from v1:
- Add dma property for UART4
---
arch/arm64/boot/dts/rockchip/rk3399.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
index 0301e3e01b38..31e487202676 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
@@ -635,6 +635,8 @@
clocks = <&cru SCLK_UART0>, <&cru PCLK_UART0>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 99 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 0>, <&dmac_peri 1>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -648,6 +650,8 @@
clocks = <&cru SCLK_UART1>, <&cru PCLK_UART1>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 98 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 2>, <&dmac_peri 3>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -661,6 +665,8 @@
clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 100 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 4>, <&dmac_peri 5>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -674,6 +680,8 @@
clocks = <&cru SCLK_UART3>, <&cru PCLK_UART3>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 101 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 6>, <&dmac_peri 7>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
@@ -1152,6 +1160,8 @@
clocks = <&pmucru SCLK_UART4_PMU>, <&pmucru PCLK_UART4_PMU>;
clock-names = "baudclk", "apb_pclk";
interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH 0>;
+ dmas = <&dmac_peri 8>, <&dmac_peri 9>;
+ dma-names = "tx", "rx";
reg-shift = <2>;
reg-io-width = <4>;
pinctrl-names = "default";
--
2.20.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-25 12:34 ` Heiko Stuebner
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Stuebner @ 2019-03-25 12:34 UTC (permalink / raw)
To: Katsuhiro Suzuki; +Cc: linux-rockchip, linux-arm-kernel, linux-kernel
Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
> Add UART dma channels as specified by the rk3399 TRM.
>
> Refer:
> RK3399 TRM V1.4: Chapter 12 DMA Controller
>
> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
applied for 5.2
Thanks
Heiko
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-25 12:34 ` Heiko Stuebner
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Stuebner @ 2019-03-25 12:34 UTC (permalink / raw)
To: Katsuhiro Suzuki; +Cc: linux-rockchip, linux-kernel, linux-arm-kernel
Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
> Add UART dma channels as specified by the rk3399 TRM.
>
> Refer:
> RK3399 TRM V1.4: Chapter 12 DMA Controller
>
> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
applied for 5.2
Thanks
Heiko
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-25 12:34 ` Heiko Stuebner
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Stuebner @ 2019-03-25 12:34 UTC (permalink / raw)
To: Katsuhiro Suzuki
Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
> Add UART dma channels as specified by the rk3399 TRM.
>
> Refer:
> RK3399 TRM V1.4: Chapter 12 DMA Controller
>
> Signed-off-by: Katsuhiro Suzuki <katsuhiro-WKCMddiH/C4xsqv6Oivclw@public.gmane.org>
applied for 5.2
Thanks
Heiko
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
2019-03-25 12:34 ` Heiko Stuebner
@ 2019-03-26 11:48 ` Robin Murphy
-1 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-03-26 11:48 UTC (permalink / raw)
To: Heiko Stuebner, Katsuhiro Suzuki
Cc: linux-rockchip, linux-kernel, linux-arm-kernel
On 25/03/2019 12:34, Heiko Stuebner wrote:
> Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
>> Add UART dma channels as specified by the rk3399 TRM.
>>
>> Refer:
>> RK3399 TRM V1.4: Chapter 12 DMA Controller
>>
>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
>
> applied for 5.2
As a heads-up, I did manage to try my board with this patch applied over
the weekend, and it makes probing the bluetooth adapter fail with
communication errors, so I'm not sure the 8250 and pl330 drivers are
really cooperating well enough :(
Robin.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-26 11:48 ` Robin Murphy
0 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-03-26 11:48 UTC (permalink / raw)
To: Heiko Stuebner, Katsuhiro Suzuki
Cc: linux-rockchip, linux-kernel, linux-arm-kernel
On 25/03/2019 12:34, Heiko Stuebner wrote:
> Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
>> Add UART dma channels as specified by the rk3399 TRM.
>>
>> Refer:
>> RK3399 TRM V1.4: Chapter 12 DMA Controller
>>
>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
>
> applied for 5.2
As a heads-up, I did manage to try my board with this patch applied over
the weekend, and it makes probing the bluetooth adapter fail with
communication errors, so I'm not sure the 8250 and pl330 drivers are
really cooperating well enough :(
Robin.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
2019-03-26 11:48 ` Robin Murphy
@ 2019-03-26 13:49 ` Katsuhiro Suzuki
-1 siblings, 0 replies; 22+ messages in thread
From: Katsuhiro Suzuki @ 2019-03-26 13:49 UTC (permalink / raw)
To: Robin Murphy, Heiko Stuebner
Cc: linux-rockchip, linux-kernel, linux-arm-kernel
Hello Robin,
Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
please revert my patch if you need.
BTW, there are DMA properties in RK3328 device-tree like as this patch.
RK3328 UART DMA could not work correctly too...??
Best Regards,
Katsuhiro Suzuki
On 2019/03/26 20:48, Robin Murphy wrote:
> On 25/03/2019 12:34, Heiko Stuebner wrote:
>> Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
>>> Add UART dma channels as specified by the rk3399 TRM.
>>>
>>> Refer:
>>> RK3399 TRM V1.4: Chapter 12 DMA Controller
>>>
>>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
>>
>> applied for 5.2
>
> As a heads-up, I did manage to try my board with this patch applied over
> the weekend, and it makes probing the bluetooth adapter fail with
> communication errors, so I'm not sure the 8250 and pl330 drivers are
> really cooperating well enough :(
>
> Robin.
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-26 13:49 ` Katsuhiro Suzuki
0 siblings, 0 replies; 22+ messages in thread
From: Katsuhiro Suzuki @ 2019-03-26 13:49 UTC (permalink / raw)
To: Robin Murphy, Heiko Stuebner
Cc: linux-rockchip, linux-kernel, linux-arm-kernel
Hello Robin,
Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
please revert my patch if you need.
BTW, there are DMA properties in RK3328 device-tree like as this patch.
RK3328 UART DMA could not work correctly too...??
Best Regards,
Katsuhiro Suzuki
On 2019/03/26 20:48, Robin Murphy wrote:
> On 25/03/2019 12:34, Heiko Stuebner wrote:
>> Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
>>> Add UART dma channels as specified by the rk3399 TRM.
>>>
>>> Refer:
>>> RK3399 TRM V1.4: Chapter 12 DMA Controller
>>>
>>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
>>
>> applied for 5.2
>
> As a heads-up, I did manage to try my board with this patch applied over
> the weekend, and it makes probing the bluetooth adapter fail with
> communication errors, so I'm not sure the 8250 and pl330 drivers are
> really cooperating well enough :(
>
> Robin.
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
2019-03-26 13:49 ` Katsuhiro Suzuki
@ 2019-03-27 12:00 ` Heiko Stuebner
-1 siblings, 0 replies; 22+ messages in thread
From: Heiko Stuebner @ 2019-03-27 12:00 UTC (permalink / raw)
To: Katsuhiro Suzuki
Cc: Robin Murphy, linux-rockchip, linux-kernel, linux-arm-kernel
Hi,
Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
> Hello Robin,
>
> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
> please revert my patch if you need.
I've dropped the patch from my queue now.
> BTW, there are DMA properties in RK3328 device-tree like as this patch.
> RK3328 UART DMA could not work correctly too...??
I remember Rockcihip dma-controllers having issues with burst-sizes
and flushing (there is a no-flushp option in pl330), so it's
possible that all share the same error up to rk3399 and rk3328
But so far no-one has shouted regarding the rk3328.
Heiko
> On 2019/03/26 20:48, Robin Murphy wrote:
> > On 25/03/2019 12:34, Heiko Stuebner wrote:
> >> Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
> >>> Add UART dma channels as specified by the rk3399 TRM.
> >>>
> >>> Refer:
> >>> RK3399 TRM V1.4: Chapter 12 DMA Controller
> >>>
> >>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
> >>
> >> applied for 5.2
> >
> > As a heads-up, I did manage to try my board with this patch applied over
> > the weekend, and it makes probing the bluetooth adapter fail with
> > communication errors, so I'm not sure the 8250 and pl330 drivers are
> > really cooperating well enough :(
> >
> > Robin.
> >
>
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-27 12:00 ` Heiko Stuebner
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Stuebner @ 2019-03-27 12:00 UTC (permalink / raw)
To: Katsuhiro Suzuki
Cc: linux-rockchip, Robin Murphy, linux-kernel, linux-arm-kernel
Hi,
Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
> Hello Robin,
>
> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
> please revert my patch if you need.
I've dropped the patch from my queue now.
> BTW, there are DMA properties in RK3328 device-tree like as this patch.
> RK3328 UART DMA could not work correctly too...??
I remember Rockcihip dma-controllers having issues with burst-sizes
and flushing (there is a no-flushp option in pl330), so it's
possible that all share the same error up to rk3399 and rk3328
But so far no-one has shouted regarding the rk3328.
Heiko
> On 2019/03/26 20:48, Robin Murphy wrote:
> > On 25/03/2019 12:34, Heiko Stuebner wrote:
> >> Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
> >>> Add UART dma channels as specified by the rk3399 TRM.
> >>>
> >>> Refer:
> >>> RK3399 TRM V1.4: Chapter 12 DMA Controller
> >>>
> >>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
> >>
> >> applied for 5.2
> >
> > As a heads-up, I did manage to try my board with this patch applied over
> > the weekend, and it makes probing the bluetooth adapter fail with
> > communication errors, so I'm not sure the 8250 and pl330 drivers are
> > really cooperating well enough :(
> >
> > Robin.
> >
>
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
2019-03-27 12:00 ` Heiko Stuebner
@ 2019-03-29 23:24 ` Robin Murphy
-1 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-03-29 23:24 UTC (permalink / raw)
To: Heiko Stuebner, Katsuhiro Suzuki
Cc: linux-rockchip, linux-kernel, linux-arm-kernel
On 2019-03-27 12:00 pm, Heiko Stuebner wrote:
> Hi,
>
> Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
>> Hello Robin,
>>
>> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
>> please revert my patch if you need.
>
> I've dropped the patch from my queue now.
>
>> BTW, there are DMA properties in RK3328 device-tree like as this patch.
>> RK3328 UART DMA could not work correctly too...??
>
> I remember Rockcihip dma-controllers having issues with burst-sizes
> and flushing (there is a no-flushp option in pl330), so it's
> possible that all share the same error up to rk3399 and rk3328
>
> But so far no-one has shouted regarding the rk3328.
Let me be the first, then, I guess :)
I found an easy way to observe the problem on my 3399, and I've just
fired up my 3328 box with a 5.0 distro kernel to find that it behaves
the same. Basically just dump a large pile of text into 'less' on the
serial console, and scroll through line-by-line - certain lines get
dropped except for a few characters at the end.
I'll see if I can narrow it down a bit, starting with trying
broken-flushp...
Robin.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-29 23:24 ` Robin Murphy
0 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-03-29 23:24 UTC (permalink / raw)
To: Heiko Stuebner, Katsuhiro Suzuki
Cc: linux-rockchip, linux-kernel, linux-arm-kernel
On 2019-03-27 12:00 pm, Heiko Stuebner wrote:
> Hi,
>
> Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
>> Hello Robin,
>>
>> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
>> please revert my patch if you need.
>
> I've dropped the patch from my queue now.
>
>> BTW, there are DMA properties in RK3328 device-tree like as this patch.
>> RK3328 UART DMA could not work correctly too...??
>
> I remember Rockcihip dma-controllers having issues with burst-sizes
> and flushing (there is a no-flushp option in pl330), so it's
> possible that all share the same error up to rk3399 and rk3328
>
> But so far no-one has shouted regarding the rk3328.
Let me be the first, then, I guess :)
I found an easy way to observe the problem on my 3399, and I've just
fired up my 3328 box with a 5.0 distro kernel to find that it behaves
the same. Basically just dump a large pile of text into 'less' on the
serial console, and scroll through line-by-line - certain lines get
dropped except for a few characters at the end.
I'll see if I can narrow it down a bit, starting with trying
broken-flushp...
Robin.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
2019-03-29 23:24 ` Robin Murphy
@ 2019-04-11 12:46 ` Heiko Stuebner
-1 siblings, 0 replies; 22+ messages in thread
From: Heiko Stuebner @ 2019-04-11 12:46 UTC (permalink / raw)
To: Robin Murphy
Cc: Katsuhiro Suzuki, linux-rockchip, linux-kernel, linux-arm-kernel
Hi Robin,
Am Samstag, 30. März 2019, 00:24:16 CEST schrieb Robin Murphy:
> On 2019-03-27 12:00 pm, Heiko Stuebner wrote:
> > Hi,
> >
> > Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
> >> Hello Robin,
> >>
> >> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
> >> please revert my patch if you need.
> >
> > I've dropped the patch from my queue now.
> >
> >> BTW, there are DMA properties in RK3328 device-tree like as this patch.
> >> RK3328 UART DMA could not work correctly too...??
> >
> > I remember Rockcihip dma-controllers having issues with burst-sizes
> > and flushing (there is a no-flushp option in pl330), so it's
> > possible that all share the same error up to rk3399 and rk3328
> >
> > But so far no-one has shouted regarding the rk3328.
>
> Let me be the first, then, I guess :)
>
> I found an easy way to observe the problem on my 3399, and I've just
> fired up my 3328 box with a 5.0 distro kernel to find that it behaves
> the same. Basically just dump a large pile of text into 'less' on the
> serial console, and scroll through line-by-line - certain lines get
> dropped except for a few characters at the end.
>
> I'll see if I can narrow it down a bit, starting with trying
> broken-flushp...
did you manage to find time for that test you wanted to do?
Heiko
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-04-11 12:46 ` Heiko Stuebner
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Stuebner @ 2019-04-11 12:46 UTC (permalink / raw)
To: Robin Murphy
Cc: Katsuhiro Suzuki, linux-rockchip, linux-kernel, linux-arm-kernel
Hi Robin,
Am Samstag, 30. März 2019, 00:24:16 CEST schrieb Robin Murphy:
> On 2019-03-27 12:00 pm, Heiko Stuebner wrote:
> > Hi,
> >
> > Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
> >> Hello Robin,
> >>
> >> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
> >> please revert my patch if you need.
> >
> > I've dropped the patch from my queue now.
> >
> >> BTW, there are DMA properties in RK3328 device-tree like as this patch.
> >> RK3328 UART DMA could not work correctly too...??
> >
> > I remember Rockcihip dma-controllers having issues with burst-sizes
> > and flushing (there is a no-flushp option in pl330), so it's
> > possible that all share the same error up to rk3399 and rk3328
> >
> > But so far no-one has shouted regarding the rk3328.
>
> Let me be the first, then, I guess :)
>
> I found an easy way to observe the problem on my 3399, and I've just
> fired up my 3328 box with a 5.0 distro kernel to find that it behaves
> the same. Basically just dump a large pile of text into 'less' on the
> serial console, and scroll through line-by-line - certain lines get
> dropped except for a few characters at the end.
>
> I'll see if I can narrow it down a bit, starting with trying
> broken-flushp...
did you manage to find time for that test you wanted to do?
Heiko
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
2019-04-11 12:46 ` Heiko Stuebner
@ 2019-04-11 13:48 ` Robin Murphy
-1 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-04-11 13:48 UTC (permalink / raw)
To: Heiko Stuebner
Cc: Katsuhiro Suzuki, linux-rockchip, linux-kernel, linux-arm-kernel
On 11/04/2019 13:46, Heiko Stuebner wrote:
> Hi Robin,
>
> Am Samstag, 30. März 2019, 00:24:16 CEST schrieb Robin Murphy:
>> On 2019-03-27 12:00 pm, Heiko Stuebner wrote:
>>> Hi,
>>>
>>> Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
>>>> Hello Robin,
>>>>
>>>> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
>>>> please revert my patch if you need.
>>>
>>> I've dropped the patch from my queue now.
>>>
>>>> BTW, there are DMA properties in RK3328 device-tree like as this patch.
>>>> RK3328 UART DMA could not work correctly too...??
>>>
>>> I remember Rockcihip dma-controllers having issues with burst-sizes
>>> and flushing (there is a no-flushp option in pl330), so it's
>>> possible that all share the same error up to rk3399 and rk3328
>>>
>>> But so far no-one has shouted regarding the rk3328.
>>
>> Let me be the first, then, I guess :)
>>
>> I found an easy way to observe the problem on my 3399, and I've just
>> fired up my 3328 box with a 5.0 distro kernel to find that it behaves
>> the same. Basically just dump a large pile of text into 'less' on the
>> serial console, and scroll through line-by-line - certain lines get
>> dropped except for a few characters at the end.
>>
>> I'll see if I can narrow it down a bit, starting with trying
>> broken-flushp...
>
> did you manage to find time for that test you wanted to do?
Sadly not - I got somewhat distracted by the ethernet thing, and since
then I've had too much else going on to follow up on any of my 'for fun'
projects :(
If it helps, from a quick look based on what I can remember off-hand,
Rockchip UARTs in general have probably been using DMA since 4.19 with
d8095f94e195 ("dmaengine: add support for reporting pause and resume
separately").
Robin.
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-04-11 13:48 ` Robin Murphy
0 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-04-11 13:48 UTC (permalink / raw)
To: Heiko Stuebner
Cc: Katsuhiro Suzuki, linux-rockchip, linux-kernel, linux-arm-kernel
On 11/04/2019 13:46, Heiko Stuebner wrote:
> Hi Robin,
>
> Am Samstag, 30. März 2019, 00:24:16 CEST schrieb Robin Murphy:
>> On 2019-03-27 12:00 pm, Heiko Stuebner wrote:
>>> Hi,
>>>
>>> Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
>>>> Hello Robin,
>>>>
>>>> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
>>>> please revert my patch if you need.
>>>
>>> I've dropped the patch from my queue now.
>>>
>>>> BTW, there are DMA properties in RK3328 device-tree like as this patch.
>>>> RK3328 UART DMA could not work correctly too...??
>>>
>>> I remember Rockcihip dma-controllers having issues with burst-sizes
>>> and flushing (there is a no-flushp option in pl330), so it's
>>> possible that all share the same error up to rk3399 and rk3328
>>>
>>> But so far no-one has shouted regarding the rk3328.
>>
>> Let me be the first, then, I guess :)
>>
>> I found an easy way to observe the problem on my 3399, and I've just
>> fired up my 3328 box with a 5.0 distro kernel to find that it behaves
>> the same. Basically just dump a large pile of text into 'less' on the
>> serial console, and scroll through line-by-line - certain lines get
>> dropped except for a few characters at the end.
>>
>> I'll see if I can narrow it down a bit, starting with trying
>> broken-flushp...
>
> did you manage to find time for that test you wanted to do?
Sadly not - I got somewhat distracted by the ethernet thing, and since
then I've had too much else going on to follow up on any of my 'for fun'
projects :(
If it helps, from a quick look based on what I can remember off-hand,
Rockchip UARTs in general have probably been using DMA since 4.19 with
d8095f94e195 ("dmaengine: add support for reporting pause and resume
separately").
Robin.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
2019-04-11 13:48 ` Robin Murphy
@ 2019-04-11 19:17 ` Robin Murphy
-1 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-04-11 19:17 UTC (permalink / raw)
To: Heiko Stuebner
Cc: Katsuhiro Suzuki, linux-rockchip, linux-kernel, linux-arm-kernel
On 2019-04-11 2:48 pm, Robin Murphy wrote:
> On 11/04/2019 13:46, Heiko Stuebner wrote:
>> Hi Robin,
>>
>> Am Samstag, 30. März 2019, 00:24:16 CEST schrieb Robin Murphy:
>>> On 2019-03-27 12:00 pm, Heiko Stuebner wrote:
>>>> Hi,
>>>>
>>>> Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
>>>>> Hello Robin,
>>>>>
>>>>> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
>>>>> please revert my patch if you need.
>>>>
>>>> I've dropped the patch from my queue now.
>>>>
>>>>> BTW, there are DMA properties in RK3328 device-tree like as this
>>>>> patch.
>>>>> RK3328 UART DMA could not work correctly too...??
>>>>
>>>> I remember Rockcihip dma-controllers having issues with burst-sizes
>>>> and flushing (there is a no-flushp option in pl330), so it's
>>>> possible that all share the same error up to rk3399 and rk3328
>>>>
>>>> But so far no-one has shouted regarding the rk3328.
>>>
>>> Let me be the first, then, I guess :)
>>>
>>> I found an easy way to observe the problem on my 3399, and I've just
>>> fired up my 3328 box with a 5.0 distro kernel to find that it behaves
>>> the same. Basically just dump a large pile of text into 'less' on the
>>> serial console, and scroll through line-by-line - certain lines get
>>> dropped except for a few characters at the end.
>>>
>>> I'll see if I can narrow it down a bit, starting with trying
>>> broken-flushp...
>>
>> did you manage to find time for that test you wanted to do?
>
> Sadly not - I got somewhat distracted by the ethernet thing, and since
> then I've had too much else going on to follow up on any of my 'for fun'
> projects :(
Urgh, and it turns out what I thought were dropped characters seems to
just be my console handling line wrapping weirdly, so now I have no idea
if RK3328 actually has an issue, and coming up with a reliable
reproducer for RK3399 is going to take some thought...
Robin.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-04-11 19:17 ` Robin Murphy
0 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-04-11 19:17 UTC (permalink / raw)
To: Heiko Stuebner
Cc: Katsuhiro Suzuki, linux-rockchip, linux-kernel, linux-arm-kernel
On 2019-04-11 2:48 pm, Robin Murphy wrote:
> On 11/04/2019 13:46, Heiko Stuebner wrote:
>> Hi Robin,
>>
>> Am Samstag, 30. März 2019, 00:24:16 CEST schrieb Robin Murphy:
>>> On 2019-03-27 12:00 pm, Heiko Stuebner wrote:
>>>> Hi,
>>>>
>>>> Am Dienstag, 26. März 2019, 14:49:16 CET schrieb Katsuhiro Suzuki:
>>>>> Hello Robin,
>>>>>
>>>>> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
>>>>> please revert my patch if you need.
>>>>
>>>> I've dropped the patch from my queue now.
>>>>
>>>>> BTW, there are DMA properties in RK3328 device-tree like as this
>>>>> patch.
>>>>> RK3328 UART DMA could not work correctly too...??
>>>>
>>>> I remember Rockcihip dma-controllers having issues with burst-sizes
>>>> and flushing (there is a no-flushp option in pl330), so it's
>>>> possible that all share the same error up to rk3399 and rk3328
>>>>
>>>> But so far no-one has shouted regarding the rk3328.
>>>
>>> Let me be the first, then, I guess :)
>>>
>>> I found an easy way to observe the problem on my 3399, and I've just
>>> fired up my 3328 box with a 5.0 distro kernel to find that it behaves
>>> the same. Basically just dump a large pile of text into 'less' on the
>>> serial console, and scroll through line-by-line - certain lines get
>>> dropped except for a few characters at the end.
>>>
>>> I'll see if I can narrow it down a bit, starting with trying
>>> broken-flushp...
>>
>> did you manage to find time for that test you wanted to do?
>
> Sadly not - I got somewhat distracted by the ethernet thing, and since
> then I've had too much else going on to follow up on any of my 'for fun'
> projects :(
Urgh, and it turns out what I thought were dropped characters seems to
just be my console handling line wrapping weirdly, so now I have no idea
if RK3328 actually has an issue, and coming up with a reliable
reproducer for RK3399 is going to take some thought...
Robin.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
2019-03-26 13:49 ` Katsuhiro Suzuki
@ 2019-03-27 12:10 ` Robin Murphy
-1 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-03-27 12:10 UTC (permalink / raw)
To: Katsuhiro Suzuki, Heiko Stuebner
Cc: linux-rockchip, linux-kernel, linux-arm-kernel
On 26/03/2019 13:49, Katsuhiro Suzuki wrote:
> Hello Robin,
>
> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
> please revert my patch if you need.
I found a little time to fire up the board again this morning, so I gave
this another try to double-check - in fact this time I only saw it fail
3 times in 15 reboots. So something's certainly not quite right, but
it's not quite as terrible as the first try implied.
My suspicion at this point is that the DMA implementation might be
losing characters occasionally, and obviously the bluetooth firmware
transfer is going to be a lot more sensitive to that than a text console is.
> BTW, there are DMA properties in RK3328 device-tree like as this patch.
> RK3328 UART DMA could not work correctly too...??
Quite possibly, although my 3328 box doesn't have any UARTS connected or
exposed other than the standard debug console, so I can't easily
investigate there. I do have a 3288 box with similar serial bluetooth to
my 3399 which might be worth digging out for comparison with an
up-to-date kernel.
Robin.
>
> Best Regards,
> Katsuhiro Suzuki
>
>
> On 2019/03/26 20:48, Robin Murphy wrote:
>> On 25/03/2019 12:34, Heiko Stuebner wrote:
>>> Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
>>>> Add UART dma channels as specified by the rk3399 TRM.
>>>>
>>>> Refer:
>>>> RK3399 TRM V1.4: Chapter 12 DMA Controller
>>>>
>>>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
>>>
>>> applied for 5.2
>>
>> As a heads-up, I did manage to try my board with this patch applied
>> over the weekend, and it makes probing the bluetooth adapter fail with
>> communication errors, so I'm not sure the 8250 and pl330 drivers are
>> really cooperating well enough :(
>>
>> Robin.
>>
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs
@ 2019-03-27 12:10 ` Robin Murphy
0 siblings, 0 replies; 22+ messages in thread
From: Robin Murphy @ 2019-03-27 12:10 UTC (permalink / raw)
To: Katsuhiro Suzuki, Heiko Stuebner
Cc: linux-rockchip, linux-kernel, linux-arm-kernel
On 26/03/2019 13:49, Katsuhiro Suzuki wrote:
> Hello Robin,
>
> Sorry for inconvenience. Since I don't adhere enabling DMA for UARTs,
> please revert my patch if you need.
I found a little time to fire up the board again this morning, so I gave
this another try to double-check - in fact this time I only saw it fail
3 times in 15 reboots. So something's certainly not quite right, but
it's not quite as terrible as the first try implied.
My suspicion at this point is that the DMA implementation might be
losing characters occasionally, and obviously the bluetooth firmware
transfer is going to be a lot more sensitive to that than a text console is.
> BTW, there are DMA properties in RK3328 device-tree like as this patch.
> RK3328 UART DMA could not work correctly too...??
Quite possibly, although my 3328 box doesn't have any UARTS connected or
exposed other than the standard debug console, so I can't easily
investigate there. I do have a 3288 box with similar serial bluetooth to
my 3399 which might be worth digging out for comparison with an
up-to-date kernel.
Robin.
>
> Best Regards,
> Katsuhiro Suzuki
>
>
> On 2019/03/26 20:48, Robin Murphy wrote:
>> On 25/03/2019 12:34, Heiko Stuebner wrote:
>>> Am Donnerstag, 21. März 2019, 17:22:44 CET schrieb Katsuhiro Suzuki:
>>>> Add UART dma channels as specified by the rk3399 TRM.
>>>>
>>>> Refer:
>>>> RK3399 TRM V1.4: Chapter 12 DMA Controller
>>>>
>>>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
>>>
>>> applied for 5.2
>>
>> As a heads-up, I did manage to try my board with this patch applied
>> over the weekend, and it makes probing the bluetooth adapter fail with
>> communication errors, so I'm not sure the 8250 and pl330 drivers are
>> really cooperating well enough :(
>>
>> Robin.
>>
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2019-04-11 19:17 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-21 16:22 [PATCH v2] arm64: dts: rockchip: add rk3399 UART DMAs Katsuhiro Suzuki
2019-03-21 16:22 ` Katsuhiro Suzuki
2019-03-21 16:22 ` Katsuhiro Suzuki
2019-03-25 12:34 ` Heiko Stuebner
2019-03-25 12:34 ` Heiko Stuebner
2019-03-25 12:34 ` Heiko Stuebner
2019-03-26 11:48 ` Robin Murphy
2019-03-26 11:48 ` Robin Murphy
2019-03-26 13:49 ` Katsuhiro Suzuki
2019-03-26 13:49 ` Katsuhiro Suzuki
2019-03-27 12:00 ` Heiko Stuebner
2019-03-27 12:00 ` Heiko Stuebner
2019-03-29 23:24 ` Robin Murphy
2019-03-29 23:24 ` Robin Murphy
2019-04-11 12:46 ` Heiko Stuebner
2019-04-11 12:46 ` Heiko Stuebner
2019-04-11 13:48 ` Robin Murphy
2019-04-11 13:48 ` Robin Murphy
2019-04-11 19:17 ` Robin Murphy
2019-04-11 19:17 ` Robin Murphy
2019-03-27 12:10 ` Robin Murphy
2019-03-27 12:10 ` Robin Murphy
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.