* [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288
@ 2020-03-19 15:13 Katsuhiro Suzuki
2020-03-19 15:25 ` Heiko Stübner
0 siblings, 1 reply; 7+ messages in thread
From: Katsuhiro Suzuki @ 2020-03-19 15:13 UTC (permalink / raw)
To: Heiko Stuebner, linux-rockchip
Cc: Katsuhiro Suzuki, linux-kernel, linux-arm-kernel
This patch removes conflicted pinctrl settings uart2 and pwm 2/3
from common rk3288.dtsi and moves exist uart2 pinctrl settings
into each rk3288*.dts files.
- pwm2_pin : use GPIO7_C6
- pwm3_pin : use GPIO7_C7
- uart2_xfer: use GPIO7_C6, GPIO7_C7
Currently uart2 rk3288 user is the following:
- rk3288-evb.dtsi:&uart2 {
- rk3288-firefly-reload.dts:&uart2 {
- rk3288-firefly.dtsi:&uart2 {
- rk3288-miqi.dts:&uart2 {
- rk3288-phycore-rdk.dts:&uart2 {
- rk3288-popmetal.dts:&uart2 {
- rk3288-r89.dts:&uart2 {
- rk3288-rock2-square.dts:&uart2 {
- rk3288-tinker.dtsi:&uart2 {
- rk3288-veyron.dtsi:&uart2 {
- rk3288-vyasa.dts:&uart2 {
And no one is using pwm2 nor pwm3.
Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net>
---
arch/arm/boot/dts/rk3288-evb.dtsi | 2 ++
arch/arm/boot/dts/rk3288-firefly-reload.dts | 2 ++
arch/arm/boot/dts/rk3288-firefly.dtsi | 2 ++
arch/arm/boot/dts/rk3288-miqi.dts | 2 ++
arch/arm/boot/dts/rk3288-phycore-rdk.dts | 2 ++
arch/arm/boot/dts/rk3288-popmetal.dts | 2 ++
arch/arm/boot/dts/rk3288-r89.dts | 2 ++
arch/arm/boot/dts/rk3288-rock2-square.dts | 2 ++
arch/arm/boot/dts/rk3288-tinker.dtsi | 2 ++
arch/arm/boot/dts/rk3288-veyron.dtsi | 2 ++
arch/arm/boot/dts/rk3288-vyasa.dts | 2 ++
arch/arm/boot/dts/rk3288.dtsi | 6 ------
12 files changed, 22 insertions(+), 6 deletions(-)
diff --git a/arch/arm/boot/dts/rk3288-evb.dtsi b/arch/arm/boot/dts/rk3288-evb.dtsi
index 018802df4c0e..74091f831ecf 100644
--- a/arch/arm/boot/dts/rk3288-evb.dtsi
+++ b/arch/arm/boot/dts/rk3288-evb.dtsi
@@ -285,6 +285,8 @@ &uart1 {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-firefly-reload.dts b/arch/arm/boot/dts/rk3288-firefly-reload.dts
index 8c38bda21a7c..b0c976c8e35b 100644
--- a/arch/arm/boot/dts/rk3288-firefly-reload.dts
+++ b/arch/arm/boot/dts/rk3288-firefly-reload.dts
@@ -283,6 +283,8 @@ &uart1 {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-firefly.dtsi b/arch/arm/boot/dts/rk3288-firefly.dtsi
index 5e0a19004e46..1632cc083c12 100644
--- a/arch/arm/boot/dts/rk3288-firefly.dtsi
+++ b/arch/arm/boot/dts/rk3288-firefly.dtsi
@@ -532,6 +532,8 @@ &uart1 {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-miqi.dts b/arch/arm/boot/dts/rk3288-miqi.dts
index c41d012c8850..2c0ed37fde80 100644
--- a/arch/arm/boot/dts/rk3288-miqi.dts
+++ b/arch/arm/boot/dts/rk3288-miqi.dts
@@ -379,6 +379,8 @@ &tsadc {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-phycore-rdk.dts b/arch/arm/boot/dts/rk3288-phycore-rdk.dts
index 1e33859de484..6532c1ac43cd 100644
--- a/arch/arm/boot/dts/rk3288-phycore-rdk.dts
+++ b/arch/arm/boot/dts/rk3288-phycore-rdk.dts
@@ -244,6 +244,8 @@ &uart0 {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-popmetal.dts b/arch/arm/boot/dts/rk3288-popmetal.dts
index 6a51940398b5..f18306bd9e6e 100644
--- a/arch/arm/boot/dts/rk3288-popmetal.dts
+++ b/arch/arm/boot/dts/rk3288-popmetal.dts
@@ -481,6 +481,8 @@ &uart1 {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-r89.dts b/arch/arm/boot/dts/rk3288-r89.dts
index a258c7ae5329..02d2f5cfe201 100644
--- a/arch/arm/boot/dts/rk3288-r89.dts
+++ b/arch/arm/boot/dts/rk3288-r89.dts
@@ -340,6 +340,8 @@ &uart1 {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-rock2-square.dts b/arch/arm/boot/dts/rk3288-rock2-square.dts
index cdcdc921ee09..a44290e882be 100644
--- a/arch/arm/boot/dts/rk3288-rock2-square.dts
+++ b/arch/arm/boot/dts/rk3288-rock2-square.dts
@@ -264,6 +264,8 @@ &spdif {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-tinker.dtsi b/arch/arm/boot/dts/rk3288-tinker.dtsi
index acfaa12ec239..0327119f71b4 100644
--- a/arch/arm/boot/dts/rk3288-tinker.dtsi
+++ b/arch/arm/boot/dts/rk3288-tinker.dtsi
@@ -500,6 +500,8 @@ &uart1 {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-veyron.dtsi b/arch/arm/boot/dts/rk3288-veyron.dtsi
index 54a6838d73f5..baa44d00e49a 100644
--- a/arch/arm/boot/dts/rk3288-veyron.dtsi
+++ b/arch/arm/boot/dts/rk3288-veyron.dtsi
@@ -412,6 +412,8 @@ &uart1 {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288-vyasa.dts b/arch/arm/boot/dts/rk3288-vyasa.dts
index 385dd59393e1..aa50cdc7f839 100644
--- a/arch/arm/boot/dts/rk3288-vyasa.dts
+++ b/arch/arm/boot/dts/rk3288-vyasa.dts
@@ -398,6 +398,8 @@ &tsadc {
};
&uart2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart2_xfer>;
status = "okay";
};
diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi
index 0cd88774db95..4c1f8cabb5eb 100644
--- a/arch/arm/boot/dts/rk3288.dtsi
+++ b/arch/arm/boot/dts/rk3288.dtsi
@@ -450,8 +450,6 @@ uart2: serial@ff690000 {
reg-io-width = <4>;
clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>;
clock-names = "baudclk", "apb_pclk";
- pinctrl-names = "default";
- pinctrl-0 = <&uart2_xfer>;
status = "disabled";
};
@@ -706,8 +704,6 @@ pwm2: pwm@ff680020 {
compatible = "rockchip,rk3288-pwm";
reg = <0x0 0xff680020 0x0 0x10>;
#pwm-cells = <3>;
- pinctrl-names = "default";
- pinctrl-0 = <&pwm2_pin>;
clocks = <&cru PCLK_RKPWM>;
clock-names = "pwm";
status = "disabled";
@@ -717,8 +713,6 @@ pwm3: pwm@ff680030 {
compatible = "rockchip,rk3288-pwm";
reg = <0x0 0xff680030 0x0 0x10>;
#pwm-cells = <3>;
- pinctrl-names = "default";
- pinctrl-0 = <&pwm3_pin>;
clocks = <&cru PCLK_RKPWM>;
clock-names = "pwm";
status = "disabled";
--
2.25.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] 7+ messages in thread* Re: [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288 2020-03-19 15:13 [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288 Katsuhiro Suzuki @ 2020-03-19 15:25 ` Heiko Stübner 2020-03-19 15:58 ` Katsuhiro Suzuki 0 siblings, 1 reply; 7+ messages in thread From: Heiko Stübner @ 2020-03-19 15:25 UTC (permalink / raw) To: Katsuhiro Suzuki; +Cc: linux-rockchip, linux-kernel, linux-arm-kernel Hi, Am Donnerstag, 19. März 2020, 16:13:39 CET schrieb Katsuhiro Suzuki: > This patch removes conflicted pinctrl settings uart2 and pwm 2/3 > from common rk3288.dtsi and moves exist uart2 pinctrl settings > into each rk3288*.dts files. > > - pwm2_pin : use GPIO7_C6 > - pwm3_pin : use GPIO7_C7 > - uart2_xfer: use GPIO7_C6, GPIO7_C7 Board files only ever get to enable either pwm2/3 or uart2, not both at once - so I'm not sure where you see conflicts. And of course there are alternative pins to use, if you need uart2 you need both pins in uart-pinmux and if you need either as pwm, then the board by design just can't use them as uart2. So pin setting should stay where they are, as there really is no conflict. Heiko > Currently uart2 rk3288 user is the following: > > - rk3288-evb.dtsi:&uart2 { > - rk3288-firefly-reload.dts:&uart2 { > - rk3288-firefly.dtsi:&uart2 { > - rk3288-miqi.dts:&uart2 { > - rk3288-phycore-rdk.dts:&uart2 { > - rk3288-popmetal.dts:&uart2 { > - rk3288-r89.dts:&uart2 { > - rk3288-rock2-square.dts:&uart2 { > - rk3288-tinker.dtsi:&uart2 { > - rk3288-veyron.dtsi:&uart2 { > - rk3288-vyasa.dts:&uart2 { > > And no one is using pwm2 nor pwm3. > > Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net> > --- > arch/arm/boot/dts/rk3288-evb.dtsi | 2 ++ > arch/arm/boot/dts/rk3288-firefly-reload.dts | 2 ++ > arch/arm/boot/dts/rk3288-firefly.dtsi | 2 ++ > arch/arm/boot/dts/rk3288-miqi.dts | 2 ++ > arch/arm/boot/dts/rk3288-phycore-rdk.dts | 2 ++ > arch/arm/boot/dts/rk3288-popmetal.dts | 2 ++ > arch/arm/boot/dts/rk3288-r89.dts | 2 ++ > arch/arm/boot/dts/rk3288-rock2-square.dts | 2 ++ > arch/arm/boot/dts/rk3288-tinker.dtsi | 2 ++ > arch/arm/boot/dts/rk3288-veyron.dtsi | 2 ++ > arch/arm/boot/dts/rk3288-vyasa.dts | 2 ++ > arch/arm/boot/dts/rk3288.dtsi | 6 ------ > 12 files changed, 22 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/boot/dts/rk3288-evb.dtsi b/arch/arm/boot/dts/rk3288-evb.dtsi > index 018802df4c0e..74091f831ecf 100644 > --- a/arch/arm/boot/dts/rk3288-evb.dtsi > +++ b/arch/arm/boot/dts/rk3288-evb.dtsi > @@ -285,6 +285,8 @@ &uart1 { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-firefly-reload.dts b/arch/arm/boot/dts/rk3288-firefly-reload.dts > index 8c38bda21a7c..b0c976c8e35b 100644 > --- a/arch/arm/boot/dts/rk3288-firefly-reload.dts > +++ b/arch/arm/boot/dts/rk3288-firefly-reload.dts > @@ -283,6 +283,8 @@ &uart1 { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-firefly.dtsi b/arch/arm/boot/dts/rk3288-firefly.dtsi > index 5e0a19004e46..1632cc083c12 100644 > --- a/arch/arm/boot/dts/rk3288-firefly.dtsi > +++ b/arch/arm/boot/dts/rk3288-firefly.dtsi > @@ -532,6 +532,8 @@ &uart1 { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-miqi.dts b/arch/arm/boot/dts/rk3288-miqi.dts > index c41d012c8850..2c0ed37fde80 100644 > --- a/arch/arm/boot/dts/rk3288-miqi.dts > +++ b/arch/arm/boot/dts/rk3288-miqi.dts > @@ -379,6 +379,8 @@ &tsadc { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-phycore-rdk.dts b/arch/arm/boot/dts/rk3288-phycore-rdk.dts > index 1e33859de484..6532c1ac43cd 100644 > --- a/arch/arm/boot/dts/rk3288-phycore-rdk.dts > +++ b/arch/arm/boot/dts/rk3288-phycore-rdk.dts > @@ -244,6 +244,8 @@ &uart0 { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-popmetal.dts b/arch/arm/boot/dts/rk3288-popmetal.dts > index 6a51940398b5..f18306bd9e6e 100644 > --- a/arch/arm/boot/dts/rk3288-popmetal.dts > +++ b/arch/arm/boot/dts/rk3288-popmetal.dts > @@ -481,6 +481,8 @@ &uart1 { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-r89.dts b/arch/arm/boot/dts/rk3288-r89.dts > index a258c7ae5329..02d2f5cfe201 100644 > --- a/arch/arm/boot/dts/rk3288-r89.dts > +++ b/arch/arm/boot/dts/rk3288-r89.dts > @@ -340,6 +340,8 @@ &uart1 { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-rock2-square.dts b/arch/arm/boot/dts/rk3288-rock2-square.dts > index cdcdc921ee09..a44290e882be 100644 > --- a/arch/arm/boot/dts/rk3288-rock2-square.dts > +++ b/arch/arm/boot/dts/rk3288-rock2-square.dts > @@ -264,6 +264,8 @@ &spdif { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-tinker.dtsi b/arch/arm/boot/dts/rk3288-tinker.dtsi > index acfaa12ec239..0327119f71b4 100644 > --- a/arch/arm/boot/dts/rk3288-tinker.dtsi > +++ b/arch/arm/boot/dts/rk3288-tinker.dtsi > @@ -500,6 +500,8 @@ &uart1 { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-veyron.dtsi b/arch/arm/boot/dts/rk3288-veyron.dtsi > index 54a6838d73f5..baa44d00e49a 100644 > --- a/arch/arm/boot/dts/rk3288-veyron.dtsi > +++ b/arch/arm/boot/dts/rk3288-veyron.dtsi > @@ -412,6 +412,8 @@ &uart1 { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288-vyasa.dts b/arch/arm/boot/dts/rk3288-vyasa.dts > index 385dd59393e1..aa50cdc7f839 100644 > --- a/arch/arm/boot/dts/rk3288-vyasa.dts > +++ b/arch/arm/boot/dts/rk3288-vyasa.dts > @@ -398,6 +398,8 @@ &tsadc { > }; > > &uart2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart2_xfer>; > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi > index 0cd88774db95..4c1f8cabb5eb 100644 > --- a/arch/arm/boot/dts/rk3288.dtsi > +++ b/arch/arm/boot/dts/rk3288.dtsi > @@ -450,8 +450,6 @@ uart2: serial@ff690000 { > reg-io-width = <4>; > clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>; > clock-names = "baudclk", "apb_pclk"; > - pinctrl-names = "default"; > - pinctrl-0 = <&uart2_xfer>; > status = "disabled"; > }; > > @@ -706,8 +704,6 @@ pwm2: pwm@ff680020 { > compatible = "rockchip,rk3288-pwm"; > reg = <0x0 0xff680020 0x0 0x10>; > #pwm-cells = <3>; > - pinctrl-names = "default"; > - pinctrl-0 = <&pwm2_pin>; > clocks = <&cru PCLK_RKPWM>; > clock-names = "pwm"; > status = "disabled"; > @@ -717,8 +713,6 @@ pwm3: pwm@ff680030 { > compatible = "rockchip,rk3288-pwm"; > reg = <0x0 0xff680030 0x0 0x10>; > #pwm-cells = <3>; > - pinctrl-names = "default"; > - pinctrl-0 = <&pwm3_pin>; > clocks = <&cru PCLK_RKPWM>; > clock-names = "pwm"; > status = "disabled"; > _______________________________________________ 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] 7+ messages in thread
* Re: [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288 2020-03-19 15:25 ` Heiko Stübner @ 2020-03-19 15:58 ` Katsuhiro Suzuki 2020-03-19 16:17 ` Heiko Stübner 2020-03-19 16:29 ` Robin Murphy 0 siblings, 2 replies; 7+ messages in thread From: Katsuhiro Suzuki @ 2020-03-19 15:58 UTC (permalink / raw) To: Heiko Stübner; +Cc: linux-rockchip, linux-kernel, linux-arm-kernel Hello Heiko, On 2020/03/20 0:25, Heiko Stübner wrote: > Hi, > > Am Donnerstag, 19. März 2020, 16:13:39 CET schrieb Katsuhiro Suzuki: >> This patch removes conflicted pinctrl settings uart2 and pwm 2/3 >> from common rk3288.dtsi and moves exist uart2 pinctrl settings >> into each rk3288*.dts files. >> >> - pwm2_pin : use GPIO7_C6 >> - pwm3_pin : use GPIO7_C7 >> - uart2_xfer: use GPIO7_C6, GPIO7_C7 > > Board files only ever get to enable either pwm2/3 or uart2, > not both at once - so I'm not sure where you see conflicts. > At first, I think so too. But I've saw this message when booting. ---------- [ 2.435504] rockchip-pinctrl pinctrl: pin gpio7-22 already requested by ff680020.pwm; cannot claim for ff690000.serial [ 2.447506] rockchip-pinctrl pinctrl: pin-238 (ff690000.serial) status -22 [ 2.455198] rockchip-pinctrl pinctrl: could not request pin 238 (gpio7-22) from group uart2-xfer on device rockchip-pinctrl ---------- And it seems that uart2(ttyS2) is not working correctly. ---------- # cat /dev/ttyS2 cat: /dev/ttyS2: Input/output error ---------- I'm using newest linux-next, make defconfig and CONFIG_MODULES = n setting with TinkerBoard. Can I resolve this issue in other ways? Best Regards, Katsuhiro Suzuki > And of course there are alternative pins to use, if you need uart2 > you need both pins in uart-pinmux and if you need either as pwm, then > the board by design just can't use them as uart2. > > So pin setting should stay where they are, as there really is no conflict. > > > Heiko > > > >> Currently uart2 rk3288 user is the following: >> >> - rk3288-evb.dtsi:&uart2 { >> - rk3288-firefly-reload.dts:&uart2 { >> - rk3288-firefly.dtsi:&uart2 { >> - rk3288-miqi.dts:&uart2 { >> - rk3288-phycore-rdk.dts:&uart2 { >> - rk3288-popmetal.dts:&uart2 { >> - rk3288-r89.dts:&uart2 { >> - rk3288-rock2-square.dts:&uart2 { >> - rk3288-tinker.dtsi:&uart2 { >> - rk3288-veyron.dtsi:&uart2 { >> - rk3288-vyasa.dts:&uart2 { >> >> And no one is using pwm2 nor pwm3. >> >> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net> >> --- >> arch/arm/boot/dts/rk3288-evb.dtsi | 2 ++ >> arch/arm/boot/dts/rk3288-firefly-reload.dts | 2 ++ >> arch/arm/boot/dts/rk3288-firefly.dtsi | 2 ++ >> arch/arm/boot/dts/rk3288-miqi.dts | 2 ++ >> arch/arm/boot/dts/rk3288-phycore-rdk.dts | 2 ++ >> arch/arm/boot/dts/rk3288-popmetal.dts | 2 ++ >> arch/arm/boot/dts/rk3288-r89.dts | 2 ++ >> arch/arm/boot/dts/rk3288-rock2-square.dts | 2 ++ >> arch/arm/boot/dts/rk3288-tinker.dtsi | 2 ++ >> arch/arm/boot/dts/rk3288-veyron.dtsi | 2 ++ >> arch/arm/boot/dts/rk3288-vyasa.dts | 2 ++ >> arch/arm/boot/dts/rk3288.dtsi | 6 ------ >> 12 files changed, 22 insertions(+), 6 deletions(-) >> >> diff --git a/arch/arm/boot/dts/rk3288-evb.dtsi b/arch/arm/boot/dts/rk3288-evb.dtsi >> index 018802df4c0e..74091f831ecf 100644 >> --- a/arch/arm/boot/dts/rk3288-evb.dtsi >> +++ b/arch/arm/boot/dts/rk3288-evb.dtsi >> @@ -285,6 +285,8 @@ &uart1 { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-firefly-reload.dts b/arch/arm/boot/dts/rk3288-firefly-reload.dts >> index 8c38bda21a7c..b0c976c8e35b 100644 >> --- a/arch/arm/boot/dts/rk3288-firefly-reload.dts >> +++ b/arch/arm/boot/dts/rk3288-firefly-reload.dts >> @@ -283,6 +283,8 @@ &uart1 { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-firefly.dtsi b/arch/arm/boot/dts/rk3288-firefly.dtsi >> index 5e0a19004e46..1632cc083c12 100644 >> --- a/arch/arm/boot/dts/rk3288-firefly.dtsi >> +++ b/arch/arm/boot/dts/rk3288-firefly.dtsi >> @@ -532,6 +532,8 @@ &uart1 { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-miqi.dts b/arch/arm/boot/dts/rk3288-miqi.dts >> index c41d012c8850..2c0ed37fde80 100644 >> --- a/arch/arm/boot/dts/rk3288-miqi.dts >> +++ b/arch/arm/boot/dts/rk3288-miqi.dts >> @@ -379,6 +379,8 @@ &tsadc { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-phycore-rdk.dts b/arch/arm/boot/dts/rk3288-phycore-rdk.dts >> index 1e33859de484..6532c1ac43cd 100644 >> --- a/arch/arm/boot/dts/rk3288-phycore-rdk.dts >> +++ b/arch/arm/boot/dts/rk3288-phycore-rdk.dts >> @@ -244,6 +244,8 @@ &uart0 { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-popmetal.dts b/arch/arm/boot/dts/rk3288-popmetal.dts >> index 6a51940398b5..f18306bd9e6e 100644 >> --- a/arch/arm/boot/dts/rk3288-popmetal.dts >> +++ b/arch/arm/boot/dts/rk3288-popmetal.dts >> @@ -481,6 +481,8 @@ &uart1 { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-r89.dts b/arch/arm/boot/dts/rk3288-r89.dts >> index a258c7ae5329..02d2f5cfe201 100644 >> --- a/arch/arm/boot/dts/rk3288-r89.dts >> +++ b/arch/arm/boot/dts/rk3288-r89.dts >> @@ -340,6 +340,8 @@ &uart1 { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-rock2-square.dts b/arch/arm/boot/dts/rk3288-rock2-square.dts >> index cdcdc921ee09..a44290e882be 100644 >> --- a/arch/arm/boot/dts/rk3288-rock2-square.dts >> +++ b/arch/arm/boot/dts/rk3288-rock2-square.dts >> @@ -264,6 +264,8 @@ &spdif { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-tinker.dtsi b/arch/arm/boot/dts/rk3288-tinker.dtsi >> index acfaa12ec239..0327119f71b4 100644 >> --- a/arch/arm/boot/dts/rk3288-tinker.dtsi >> +++ b/arch/arm/boot/dts/rk3288-tinker.dtsi >> @@ -500,6 +500,8 @@ &uart1 { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-veyron.dtsi b/arch/arm/boot/dts/rk3288-veyron.dtsi >> index 54a6838d73f5..baa44d00e49a 100644 >> --- a/arch/arm/boot/dts/rk3288-veyron.dtsi >> +++ b/arch/arm/boot/dts/rk3288-veyron.dtsi >> @@ -412,6 +412,8 @@ &uart1 { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288-vyasa.dts b/arch/arm/boot/dts/rk3288-vyasa.dts >> index 385dd59393e1..aa50cdc7f839 100644 >> --- a/arch/arm/boot/dts/rk3288-vyasa.dts >> +++ b/arch/arm/boot/dts/rk3288-vyasa.dts >> @@ -398,6 +398,8 @@ &tsadc { >> }; >> >> &uart2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart2_xfer>; >> status = "okay"; >> }; >> >> diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi >> index 0cd88774db95..4c1f8cabb5eb 100644 >> --- a/arch/arm/boot/dts/rk3288.dtsi >> +++ b/arch/arm/boot/dts/rk3288.dtsi >> @@ -450,8 +450,6 @@ uart2: serial@ff690000 { >> reg-io-width = <4>; >> clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>; >> clock-names = "baudclk", "apb_pclk"; >> - pinctrl-names = "default"; >> - pinctrl-0 = <&uart2_xfer>; >> status = "disabled"; >> }; >> >> @@ -706,8 +704,6 @@ pwm2: pwm@ff680020 { >> compatible = "rockchip,rk3288-pwm"; >> reg = <0x0 0xff680020 0x0 0x10>; >> #pwm-cells = <3>; >> - pinctrl-names = "default"; >> - pinctrl-0 = <&pwm2_pin>; >> clocks = <&cru PCLK_RKPWM>; >> clock-names = "pwm"; >> status = "disabled"; >> @@ -717,8 +713,6 @@ pwm3: pwm@ff680030 { >> compatible = "rockchip,rk3288-pwm"; >> reg = <0x0 0xff680030 0x0 0x10>; >> #pwm-cells = <3>; >> - pinctrl-names = "default"; >> - pinctrl-0 = <&pwm3_pin>; >> clocks = <&cru PCLK_RKPWM>; >> clock-names = "pwm"; >> status = "disabled"; >> > > > > > _______________________________________________ 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] 7+ messages in thread
* Re: [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288 2020-03-19 15:58 ` Katsuhiro Suzuki @ 2020-03-19 16:17 ` Heiko Stübner 2020-03-19 18:31 ` Katsuhiro Suzuki 2020-03-19 16:29 ` Robin Murphy 1 sibling, 1 reply; 7+ messages in thread From: Heiko Stübner @ 2020-03-19 16:17 UTC (permalink / raw) To: Katsuhiro Suzuki; +Cc: linux-rockchip, linux-kernel, linux-arm-kernel Hi, Am Donnerstag, 19. März 2020, 16:58:00 CET schrieb Katsuhiro Suzuki: > On 2020/03/20 0:25, Heiko Stübner wrote: > > Am Donnerstag, 19. März 2020, 16:13:39 CET schrieb Katsuhiro Suzuki: > >> This patch removes conflicted pinctrl settings uart2 and pwm 2/3 > >> from common rk3288.dtsi and moves exist uart2 pinctrl settings > >> into each rk3288*.dts files. > >> > >> - pwm2_pin : use GPIO7_C6 > >> - pwm3_pin : use GPIO7_C7 > >> - uart2_xfer: use GPIO7_C6, GPIO7_C7 > > > > Board files only ever get to enable either pwm2/3 or uart2, > > not both at once - so I'm not sure where you see conflicts. > > > > At first, I think so too. But I've saw this message when booting. > > ---------- > [ 2.435504] rockchip-pinctrl pinctrl: pin gpio7-22 already requested by ff680020.pwm; cannot claim for ff690000.serial > [ 2.447506] rockchip-pinctrl pinctrl: pin-238 (ff690000.serial) status -22 > [ 2.455198] rockchip-pinctrl pinctrl: could not request pin 238 (gpio7-22) from group uart2-xfer on device rockchip-pinctrl > ---------- > > And it seems that uart2(ttyS2) is not working correctly. > > ---------- > # cat /dev/ttyS2 > cat: /dev/ttyS2: Input/output error > ---------- > > I'm using newest linux-next, make defconfig and CONFIG_MODULES = n setting > with TinkerBoard. Can I resolve this issue in other ways? On the tinker-board pwm2 is unused and has a status=disabled in the devicetree [at least when checking my current branches], so the kernel shouldn't even create a plaform-device for it. I.e. see [0] - the of_device_is_available() call should already drop the disabled devices, so there should not even exist a ff680020.pwm . So I guess the solution would be to find out why it creates ff680020.pwm in the first place and fix that. Heiko [0] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/of/platform.c#n174 > > And of course there are alternative pins to use, if you need uart2 > > you need both pins in uart-pinmux and if you need either as pwm, then > > the board by design just can't use them as uart2. > > > > So pin setting should stay where they are, as there really is no conflict. > > > > > > Heiko > > > > > > > >> Currently uart2 rk3288 user is the following: > >> > >> - rk3288-evb.dtsi:&uart2 { > >> - rk3288-firefly-reload.dts:&uart2 { > >> - rk3288-firefly.dtsi:&uart2 { > >> - rk3288-miqi.dts:&uart2 { > >> - rk3288-phycore-rdk.dts:&uart2 { > >> - rk3288-popmetal.dts:&uart2 { > >> - rk3288-r89.dts:&uart2 { > >> - rk3288-rock2-square.dts:&uart2 { > >> - rk3288-tinker.dtsi:&uart2 { > >> - rk3288-veyron.dtsi:&uart2 { > >> - rk3288-vyasa.dts:&uart2 { > >> > >> And no one is using pwm2 nor pwm3. > >> > >> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net> > >> --- > >> arch/arm/boot/dts/rk3288-evb.dtsi | 2 ++ > >> arch/arm/boot/dts/rk3288-firefly-reload.dts | 2 ++ > >> arch/arm/boot/dts/rk3288-firefly.dtsi | 2 ++ > >> arch/arm/boot/dts/rk3288-miqi.dts | 2 ++ > >> arch/arm/boot/dts/rk3288-phycore-rdk.dts | 2 ++ > >> arch/arm/boot/dts/rk3288-popmetal.dts | 2 ++ > >> arch/arm/boot/dts/rk3288-r89.dts | 2 ++ > >> arch/arm/boot/dts/rk3288-rock2-square.dts | 2 ++ > >> arch/arm/boot/dts/rk3288-tinker.dtsi | 2 ++ > >> arch/arm/boot/dts/rk3288-veyron.dtsi | 2 ++ > >> arch/arm/boot/dts/rk3288-vyasa.dts | 2 ++ > >> arch/arm/boot/dts/rk3288.dtsi | 6 ------ > >> 12 files changed, 22 insertions(+), 6 deletions(-) > >> > >> diff --git a/arch/arm/boot/dts/rk3288-evb.dtsi b/arch/arm/boot/dts/rk3288-evb.dtsi > >> index 018802df4c0e..74091f831ecf 100644 > >> --- a/arch/arm/boot/dts/rk3288-evb.dtsi > >> +++ b/arch/arm/boot/dts/rk3288-evb.dtsi > >> @@ -285,6 +285,8 @@ &uart1 { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-firefly-reload.dts b/arch/arm/boot/dts/rk3288-firefly-reload.dts > >> index 8c38bda21a7c..b0c976c8e35b 100644 > >> --- a/arch/arm/boot/dts/rk3288-firefly-reload.dts > >> +++ b/arch/arm/boot/dts/rk3288-firefly-reload.dts > >> @@ -283,6 +283,8 @@ &uart1 { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-firefly.dtsi b/arch/arm/boot/dts/rk3288-firefly.dtsi > >> index 5e0a19004e46..1632cc083c12 100644 > >> --- a/arch/arm/boot/dts/rk3288-firefly.dtsi > >> +++ b/arch/arm/boot/dts/rk3288-firefly.dtsi > >> @@ -532,6 +532,8 @@ &uart1 { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-miqi.dts b/arch/arm/boot/dts/rk3288-miqi.dts > >> index c41d012c8850..2c0ed37fde80 100644 > >> --- a/arch/arm/boot/dts/rk3288-miqi.dts > >> +++ b/arch/arm/boot/dts/rk3288-miqi.dts > >> @@ -379,6 +379,8 @@ &tsadc { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-phycore-rdk.dts b/arch/arm/boot/dts/rk3288-phycore-rdk.dts > >> index 1e33859de484..6532c1ac43cd 100644 > >> --- a/arch/arm/boot/dts/rk3288-phycore-rdk.dts > >> +++ b/arch/arm/boot/dts/rk3288-phycore-rdk.dts > >> @@ -244,6 +244,8 @@ &uart0 { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-popmetal.dts b/arch/arm/boot/dts/rk3288-popmetal.dts > >> index 6a51940398b5..f18306bd9e6e 100644 > >> --- a/arch/arm/boot/dts/rk3288-popmetal.dts > >> +++ b/arch/arm/boot/dts/rk3288-popmetal.dts > >> @@ -481,6 +481,8 @@ &uart1 { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-r89.dts b/arch/arm/boot/dts/rk3288-r89.dts > >> index a258c7ae5329..02d2f5cfe201 100644 > >> --- a/arch/arm/boot/dts/rk3288-r89.dts > >> +++ b/arch/arm/boot/dts/rk3288-r89.dts > >> @@ -340,6 +340,8 @@ &uart1 { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-rock2-square.dts b/arch/arm/boot/dts/rk3288-rock2-square.dts > >> index cdcdc921ee09..a44290e882be 100644 > >> --- a/arch/arm/boot/dts/rk3288-rock2-square.dts > >> +++ b/arch/arm/boot/dts/rk3288-rock2-square.dts > >> @@ -264,6 +264,8 @@ &spdif { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-tinker.dtsi b/arch/arm/boot/dts/rk3288-tinker.dtsi > >> index acfaa12ec239..0327119f71b4 100644 > >> --- a/arch/arm/boot/dts/rk3288-tinker.dtsi > >> +++ b/arch/arm/boot/dts/rk3288-tinker.dtsi > >> @@ -500,6 +500,8 @@ &uart1 { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-veyron.dtsi b/arch/arm/boot/dts/rk3288-veyron.dtsi > >> index 54a6838d73f5..baa44d00e49a 100644 > >> --- a/arch/arm/boot/dts/rk3288-veyron.dtsi > >> +++ b/arch/arm/boot/dts/rk3288-veyron.dtsi > >> @@ -412,6 +412,8 @@ &uart1 { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288-vyasa.dts b/arch/arm/boot/dts/rk3288-vyasa.dts > >> index 385dd59393e1..aa50cdc7f839 100644 > >> --- a/arch/arm/boot/dts/rk3288-vyasa.dts > >> +++ b/arch/arm/boot/dts/rk3288-vyasa.dts > >> @@ -398,6 +398,8 @@ &tsadc { > >> }; > >> > >> &uart2 { > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&uart2_xfer>; > >> status = "okay"; > >> }; > >> > >> diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi > >> index 0cd88774db95..4c1f8cabb5eb 100644 > >> --- a/arch/arm/boot/dts/rk3288.dtsi > >> +++ b/arch/arm/boot/dts/rk3288.dtsi > >> @@ -450,8 +450,6 @@ uart2: serial@ff690000 { > >> reg-io-width = <4>; > >> clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>; > >> clock-names = "baudclk", "apb_pclk"; > >> - pinctrl-names = "default"; > >> - pinctrl-0 = <&uart2_xfer>; > >> status = "disabled"; > >> }; > >> > >> @@ -706,8 +704,6 @@ pwm2: pwm@ff680020 { > >> compatible = "rockchip,rk3288-pwm"; > >> reg = <0x0 0xff680020 0x0 0x10>; > >> #pwm-cells = <3>; > >> - pinctrl-names = "default"; > >> - pinctrl-0 = <&pwm2_pin>; > >> clocks = <&cru PCLK_RKPWM>; > >> clock-names = "pwm"; > >> status = "disabled"; > >> @@ -717,8 +713,6 @@ pwm3: pwm@ff680030 { > >> compatible = "rockchip,rk3288-pwm"; > >> reg = <0x0 0xff680030 0x0 0x10>; > >> #pwm-cells = <3>; > >> - pinctrl-names = "default"; > >> - pinctrl-0 = <&pwm3_pin>; > >> clocks = <&cru PCLK_RKPWM>; > >> clock-names = "pwm"; > >> status = "disabled"; > >> > > > > > > > > > > > > _______________________________________________ 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] 7+ messages in thread
* Re: [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288 2020-03-19 16:17 ` Heiko Stübner @ 2020-03-19 18:31 ` Katsuhiro Suzuki 0 siblings, 0 replies; 7+ messages in thread From: Katsuhiro Suzuki @ 2020-03-19 18:31 UTC (permalink / raw) To: Heiko Stübner; +Cc: linux-rockchip, linux-kernel, linux-arm-kernel Hello Heiko, On 2020/03/20 1:17, Heiko Stübner wrote: > Hi, > > Am Donnerstag, 19. März 2020, 16:58:00 CET schrieb Katsuhiro Suzuki: >> On 2020/03/20 0:25, Heiko Stübner wrote: >>> Am Donnerstag, 19. März 2020, 16:13:39 CET schrieb Katsuhiro Suzuki: >>>> This patch removes conflicted pinctrl settings uart2 and pwm 2/3 >>>> from common rk3288.dtsi and moves exist uart2 pinctrl settings >>>> into each rk3288*.dts files. >>>> >>>> - pwm2_pin : use GPIO7_C6 >>>> - pwm3_pin : use GPIO7_C7 >>>> - uart2_xfer: use GPIO7_C6, GPIO7_C7 >>> >>> Board files only ever get to enable either pwm2/3 or uart2, >>> not both at once - so I'm not sure where you see conflicts. >>> >> >> At first, I think so too. But I've saw this message when booting. >> >> ---------- >> [ 2.435504] rockchip-pinctrl pinctrl: pin gpio7-22 already requested by ff680020.pwm; cannot claim for ff690000.serial >> [ 2.447506] rockchip-pinctrl pinctrl: pin-238 (ff690000.serial) status -22 >> [ 2.455198] rockchip-pinctrl pinctrl: could not request pin 238 (gpio7-22) from group uart2-xfer on device rockchip-pinctrl >> ---------- >> >> And it seems that uart2(ttyS2) is not working correctly. >> >> ---------- >> # cat /dev/ttyS2 >> cat: /dev/ttyS2: Input/output error >> ---------- >> >> I'm using newest linux-next, make defconfig and CONFIG_MODULES = n setting >> with TinkerBoard. Can I resolve this issue in other ways? > > On the tinker-board pwm2 is unused and has a status=disabled in the > devicetree [at least when checking my current branches], so the kernel > shouldn't even create a plaform-device for it. > > I.e. see [0] - the of_device_is_available() call should already drop the > disabled devices, so there should not even exist a ff680020.pwm . > > So I guess the solution would be to find out why it creates ff680020.pwm > in the first place and fix that. Thank you for comment. Yes, I agree. Root cause is a wrong setting in my TinkerBoard (maybe bootloader) that is suggested by Robin in another mail. Sorry for confusing, please drop my patch. Best Regards, Katsuhiro Suzuki > > Heiko > > [0] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/of/platform.c#n174 > > > >>> And of course there are alternative pins to use, if you need uart2 >>> you need both pins in uart-pinmux and if you need either as pwm, then >>> the board by design just can't use them as uart2. >>> >>> So pin setting should stay where they are, as there really is no conflict. >>> >>> >>> Heiko >>> >>> >>> >>>> Currently uart2 rk3288 user is the following: >>>> >>>> - rk3288-evb.dtsi:&uart2 { >>>> - rk3288-firefly-reload.dts:&uart2 { >>>> - rk3288-firefly.dtsi:&uart2 { >>>> - rk3288-miqi.dts:&uart2 { >>>> - rk3288-phycore-rdk.dts:&uart2 { >>>> - rk3288-popmetal.dts:&uart2 { >>>> - rk3288-r89.dts:&uart2 { >>>> - rk3288-rock2-square.dts:&uart2 { >>>> - rk3288-tinker.dtsi:&uart2 { >>>> - rk3288-veyron.dtsi:&uart2 { >>>> - rk3288-vyasa.dts:&uart2 { >>>> >>>> And no one is using pwm2 nor pwm3. >>>> >>>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net> >>>> --- >>>> arch/arm/boot/dts/rk3288-evb.dtsi | 2 ++ >>>> arch/arm/boot/dts/rk3288-firefly-reload.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-firefly.dtsi | 2 ++ >>>> arch/arm/boot/dts/rk3288-miqi.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-phycore-rdk.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-popmetal.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-r89.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-rock2-square.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-tinker.dtsi | 2 ++ >>>> arch/arm/boot/dts/rk3288-veyron.dtsi | 2 ++ >>>> arch/arm/boot/dts/rk3288-vyasa.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288.dtsi | 6 ------ >>>> 12 files changed, 22 insertions(+), 6 deletions(-) >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-evb.dtsi b/arch/arm/boot/dts/rk3288-evb.dtsi >>>> index 018802df4c0e..74091f831ecf 100644 >>>> --- a/arch/arm/boot/dts/rk3288-evb.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288-evb.dtsi >>>> @@ -285,6 +285,8 @@ &uart1 { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-firefly-reload.dts b/arch/arm/boot/dts/rk3288-firefly-reload.dts >>>> index 8c38bda21a7c..b0c976c8e35b 100644 >>>> --- a/arch/arm/boot/dts/rk3288-firefly-reload.dts >>>> +++ b/arch/arm/boot/dts/rk3288-firefly-reload.dts >>>> @@ -283,6 +283,8 @@ &uart1 { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-firefly.dtsi b/arch/arm/boot/dts/rk3288-firefly.dtsi >>>> index 5e0a19004e46..1632cc083c12 100644 >>>> --- a/arch/arm/boot/dts/rk3288-firefly.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288-firefly.dtsi >>>> @@ -532,6 +532,8 @@ &uart1 { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-miqi.dts b/arch/arm/boot/dts/rk3288-miqi.dts >>>> index c41d012c8850..2c0ed37fde80 100644 >>>> --- a/arch/arm/boot/dts/rk3288-miqi.dts >>>> +++ b/arch/arm/boot/dts/rk3288-miqi.dts >>>> @@ -379,6 +379,8 @@ &tsadc { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-phycore-rdk.dts b/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>>> index 1e33859de484..6532c1ac43cd 100644 >>>> --- a/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>>> +++ b/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>>> @@ -244,6 +244,8 @@ &uart0 { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-popmetal.dts b/arch/arm/boot/dts/rk3288-popmetal.dts >>>> index 6a51940398b5..f18306bd9e6e 100644 >>>> --- a/arch/arm/boot/dts/rk3288-popmetal.dts >>>> +++ b/arch/arm/boot/dts/rk3288-popmetal.dts >>>> @@ -481,6 +481,8 @@ &uart1 { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-r89.dts b/arch/arm/boot/dts/rk3288-r89.dts >>>> index a258c7ae5329..02d2f5cfe201 100644 >>>> --- a/arch/arm/boot/dts/rk3288-r89.dts >>>> +++ b/arch/arm/boot/dts/rk3288-r89.dts >>>> @@ -340,6 +340,8 @@ &uart1 { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-rock2-square.dts b/arch/arm/boot/dts/rk3288-rock2-square.dts >>>> index cdcdc921ee09..a44290e882be 100644 >>>> --- a/arch/arm/boot/dts/rk3288-rock2-square.dts >>>> +++ b/arch/arm/boot/dts/rk3288-rock2-square.dts >>>> @@ -264,6 +264,8 @@ &spdif { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-tinker.dtsi b/arch/arm/boot/dts/rk3288-tinker.dtsi >>>> index acfaa12ec239..0327119f71b4 100644 >>>> --- a/arch/arm/boot/dts/rk3288-tinker.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288-tinker.dtsi >>>> @@ -500,6 +500,8 @@ &uart1 { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-veyron.dtsi b/arch/arm/boot/dts/rk3288-veyron.dtsi >>>> index 54a6838d73f5..baa44d00e49a 100644 >>>> --- a/arch/arm/boot/dts/rk3288-veyron.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288-veyron.dtsi >>>> @@ -412,6 +412,8 @@ &uart1 { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-vyasa.dts b/arch/arm/boot/dts/rk3288-vyasa.dts >>>> index 385dd59393e1..aa50cdc7f839 100644 >>>> --- a/arch/arm/boot/dts/rk3288-vyasa.dts >>>> +++ b/arch/arm/boot/dts/rk3288-vyasa.dts >>>> @@ -398,6 +398,8 @@ &tsadc { >>>> }; >>>> >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi >>>> index 0cd88774db95..4c1f8cabb5eb 100644 >>>> --- a/arch/arm/boot/dts/rk3288.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288.dtsi >>>> @@ -450,8 +450,6 @@ uart2: serial@ff690000 { >>>> reg-io-width = <4>; >>>> clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>; >>>> clock-names = "baudclk", "apb_pclk"; >>>> - pinctrl-names = "default"; >>>> - pinctrl-0 = <&uart2_xfer>; >>>> status = "disabled"; >>>> }; >>>> >>>> @@ -706,8 +704,6 @@ pwm2: pwm@ff680020 { >>>> compatible = "rockchip,rk3288-pwm"; >>>> reg = <0x0 0xff680020 0x0 0x10>; >>>> #pwm-cells = <3>; >>>> - pinctrl-names = "default"; >>>> - pinctrl-0 = <&pwm2_pin>; >>>> clocks = <&cru PCLK_RKPWM>; >>>> clock-names = "pwm"; >>>> status = "disabled"; >>>> @@ -717,8 +713,6 @@ pwm3: pwm@ff680030 { >>>> compatible = "rockchip,rk3288-pwm"; >>>> reg = <0x0 0xff680030 0x0 0x10>; >>>> #pwm-cells = <3>; >>>> - pinctrl-names = "default"; >>>> - pinctrl-0 = <&pwm3_pin>; >>>> clocks = <&cru PCLK_RKPWM>; >>>> clock-names = "pwm"; >>>> status = "disabled"; >>>> >>> >>> >>> >>> >>> >> >> > > > > > _______________________________________________ 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] 7+ messages in thread
* Re: [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288 2020-03-19 15:58 ` Katsuhiro Suzuki 2020-03-19 16:17 ` Heiko Stübner @ 2020-03-19 16:29 ` Robin Murphy 2020-03-19 18:31 ` Katsuhiro Suzuki 1 sibling, 1 reply; 7+ messages in thread From: Robin Murphy @ 2020-03-19 16:29 UTC (permalink / raw) To: Katsuhiro Suzuki, Heiko Stübner Cc: linux-rockchip, linux-kernel, linux-arm-kernel On 2020-03-19 3:58 pm, Katsuhiro Suzuki wrote: > Hello Heiko, > > On 2020/03/20 0:25, Heiko Stübner wrote: >> Hi, >> >> Am Donnerstag, 19. März 2020, 16:13:39 CET schrieb Katsuhiro Suzuki: >>> This patch removes conflicted pinctrl settings uart2 and pwm 2/3 >>> from common rk3288.dtsi and moves exist uart2 pinctrl settings >>> into each rk3288*.dts files. >>> >>> - pwm2_pin : use GPIO7_C6 >>> - pwm3_pin : use GPIO7_C7 >>> - uart2_xfer: use GPIO7_C6, GPIO7_C7 >> >> Board files only ever get to enable either pwm2/3 or uart2, >> not both at once - so I'm not sure where you see conflicts. >> > > At first, I think so too. But I've saw this message when booting. > > ---------- > [ 2.435504] rockchip-pinctrl pinctrl: pin gpio7-22 already requested > by ff680020.pwm; cannot claim for ff690000.serial > [ 2.447506] rockchip-pinctrl pinctrl: pin-238 (ff690000.serial) > status -22 > [ 2.455198] rockchip-pinctrl pinctrl: could not request pin 238 > (gpio7-22) from group uart2-xfer on device rockchip-pinctrl > ---------- > > And it seems that uart2(ttyS2) is not working correctly. > > ---------- > # cat /dev/ttyS2 > cat: /dev/ttyS2: Input/output error > ---------- > > I'm using newest linux-next, make defconfig and CONFIG_MODULES = n setting > with TinkerBoard. Can I resolve this issue in other ways? Do you perhaps have a DT overlay or bootloader script enabling pwm2? (try `cat /sys/firmware/devicetree/base/pwm@ff680020/status` for a sanity check). FWIW I don't recall ever seeing this on my RK3288 box. (or of course maybe there's just some pinctrl bug in -next that's claiming configs for disabled devices) Robin. > > Best Regards, > Katsuhiro Suzuki > > >> And of course there are alternative pins to use, if you need uart2 >> you need both pins in uart-pinmux and if you need either as pwm, then >> the board by design just can't use them as uart2. >> >> So pin setting should stay where they are, as there really is no >> conflict. >> >> >> Heiko >> >> >> >>> Currently uart2 rk3288 user is the following: >>> >>> - rk3288-evb.dtsi:&uart2 { >>> - rk3288-firefly-reload.dts:&uart2 { >>> - rk3288-firefly.dtsi:&uart2 { >>> - rk3288-miqi.dts:&uart2 { >>> - rk3288-phycore-rdk.dts:&uart2 { >>> - rk3288-popmetal.dts:&uart2 { >>> - rk3288-r89.dts:&uart2 { >>> - rk3288-rock2-square.dts:&uart2 { >>> - rk3288-tinker.dtsi:&uart2 { >>> - rk3288-veyron.dtsi:&uart2 { >>> - rk3288-vyasa.dts:&uart2 { >>> >>> And no one is using pwm2 nor pwm3. >>> >>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net> >>> --- >>> arch/arm/boot/dts/rk3288-evb.dtsi | 2 ++ >>> arch/arm/boot/dts/rk3288-firefly-reload.dts | 2 ++ >>> arch/arm/boot/dts/rk3288-firefly.dtsi | 2 ++ >>> arch/arm/boot/dts/rk3288-miqi.dts | 2 ++ >>> arch/arm/boot/dts/rk3288-phycore-rdk.dts | 2 ++ >>> arch/arm/boot/dts/rk3288-popmetal.dts | 2 ++ >>> arch/arm/boot/dts/rk3288-r89.dts | 2 ++ >>> arch/arm/boot/dts/rk3288-rock2-square.dts | 2 ++ >>> arch/arm/boot/dts/rk3288-tinker.dtsi | 2 ++ >>> arch/arm/boot/dts/rk3288-veyron.dtsi | 2 ++ >>> arch/arm/boot/dts/rk3288-vyasa.dts | 2 ++ >>> arch/arm/boot/dts/rk3288.dtsi | 6 ------ >>> 12 files changed, 22 insertions(+), 6 deletions(-) >>> >>> diff --git a/arch/arm/boot/dts/rk3288-evb.dtsi >>> b/arch/arm/boot/dts/rk3288-evb.dtsi >>> index 018802df4c0e..74091f831ecf 100644 >>> --- a/arch/arm/boot/dts/rk3288-evb.dtsi >>> +++ b/arch/arm/boot/dts/rk3288-evb.dtsi >>> @@ -285,6 +285,8 @@ &uart1 { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-firefly-reload.dts >>> b/arch/arm/boot/dts/rk3288-firefly-reload.dts >>> index 8c38bda21a7c..b0c976c8e35b 100644 >>> --- a/arch/arm/boot/dts/rk3288-firefly-reload.dts >>> +++ b/arch/arm/boot/dts/rk3288-firefly-reload.dts >>> @@ -283,6 +283,8 @@ &uart1 { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-firefly.dtsi >>> b/arch/arm/boot/dts/rk3288-firefly.dtsi >>> index 5e0a19004e46..1632cc083c12 100644 >>> --- a/arch/arm/boot/dts/rk3288-firefly.dtsi >>> +++ b/arch/arm/boot/dts/rk3288-firefly.dtsi >>> @@ -532,6 +532,8 @@ &uart1 { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-miqi.dts >>> b/arch/arm/boot/dts/rk3288-miqi.dts >>> index c41d012c8850..2c0ed37fde80 100644 >>> --- a/arch/arm/boot/dts/rk3288-miqi.dts >>> +++ b/arch/arm/boot/dts/rk3288-miqi.dts >>> @@ -379,6 +379,8 @@ &tsadc { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>> b/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>> index 1e33859de484..6532c1ac43cd 100644 >>> --- a/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>> +++ b/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>> @@ -244,6 +244,8 @@ &uart0 { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-popmetal.dts >>> b/arch/arm/boot/dts/rk3288-popmetal.dts >>> index 6a51940398b5..f18306bd9e6e 100644 >>> --- a/arch/arm/boot/dts/rk3288-popmetal.dts >>> +++ b/arch/arm/boot/dts/rk3288-popmetal.dts >>> @@ -481,6 +481,8 @@ &uart1 { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-r89.dts >>> b/arch/arm/boot/dts/rk3288-r89.dts >>> index a258c7ae5329..02d2f5cfe201 100644 >>> --- a/arch/arm/boot/dts/rk3288-r89.dts >>> +++ b/arch/arm/boot/dts/rk3288-r89.dts >>> @@ -340,6 +340,8 @@ &uart1 { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-rock2-square.dts >>> b/arch/arm/boot/dts/rk3288-rock2-square.dts >>> index cdcdc921ee09..a44290e882be 100644 >>> --- a/arch/arm/boot/dts/rk3288-rock2-square.dts >>> +++ b/arch/arm/boot/dts/rk3288-rock2-square.dts >>> @@ -264,6 +264,8 @@ &spdif { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-tinker.dtsi >>> b/arch/arm/boot/dts/rk3288-tinker.dtsi >>> index acfaa12ec239..0327119f71b4 100644 >>> --- a/arch/arm/boot/dts/rk3288-tinker.dtsi >>> +++ b/arch/arm/boot/dts/rk3288-tinker.dtsi >>> @@ -500,6 +500,8 @@ &uart1 { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-veyron.dtsi >>> b/arch/arm/boot/dts/rk3288-veyron.dtsi >>> index 54a6838d73f5..baa44d00e49a 100644 >>> --- a/arch/arm/boot/dts/rk3288-veyron.dtsi >>> +++ b/arch/arm/boot/dts/rk3288-veyron.dtsi >>> @@ -412,6 +412,8 @@ &uart1 { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288-vyasa.dts >>> b/arch/arm/boot/dts/rk3288-vyasa.dts >>> index 385dd59393e1..aa50cdc7f839 100644 >>> --- a/arch/arm/boot/dts/rk3288-vyasa.dts >>> +++ b/arch/arm/boot/dts/rk3288-vyasa.dts >>> @@ -398,6 +398,8 @@ &tsadc { >>> }; >>> &uart2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart2_xfer>; >>> status = "okay"; >>> }; >>> diff --git a/arch/arm/boot/dts/rk3288.dtsi >>> b/arch/arm/boot/dts/rk3288.dtsi >>> index 0cd88774db95..4c1f8cabb5eb 100644 >>> --- a/arch/arm/boot/dts/rk3288.dtsi >>> +++ b/arch/arm/boot/dts/rk3288.dtsi >>> @@ -450,8 +450,6 @@ uart2: serial@ff690000 { >>> reg-io-width = <4>; >>> clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>; >>> clock-names = "baudclk", "apb_pclk"; >>> - pinctrl-names = "default"; >>> - pinctrl-0 = <&uart2_xfer>; >>> status = "disabled"; >>> }; >>> @@ -706,8 +704,6 @@ pwm2: pwm@ff680020 { >>> compatible = "rockchip,rk3288-pwm"; >>> reg = <0x0 0xff680020 0x0 0x10>; >>> #pwm-cells = <3>; >>> - pinctrl-names = "default"; >>> - pinctrl-0 = <&pwm2_pin>; >>> clocks = <&cru PCLK_RKPWM>; >>> clock-names = "pwm"; >>> status = "disabled"; >>> @@ -717,8 +713,6 @@ pwm3: pwm@ff680030 { >>> compatible = "rockchip,rk3288-pwm"; >>> reg = <0x0 0xff680030 0x0 0x10>; >>> #pwm-cells = <3>; >>> - pinctrl-names = "default"; >>> - pinctrl-0 = <&pwm3_pin>; >>> clocks = <&cru PCLK_RKPWM>; >>> clock-names = "pwm"; >>> status = "disabled"; >>> >> >> >> >> >> > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel _______________________________________________ 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] 7+ messages in thread
* Re: [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288 2020-03-19 16:29 ` Robin Murphy @ 2020-03-19 18:31 ` Katsuhiro Suzuki 0 siblings, 0 replies; 7+ messages in thread From: Katsuhiro Suzuki @ 2020-03-19 18:31 UTC (permalink / raw) To: Robin Murphy, Heiko Stübner Cc: linux-rockchip, linux-kernel, linux-arm-kernel Hello Robin, Thank you for reply. On 2020/03/20 1:29, Robin Murphy wrote: > On 2020-03-19 3:58 pm, Katsuhiro Suzuki wrote: >> Hello Heiko, >> >> On 2020/03/20 0:25, Heiko Stübner wrote: >>> Hi, >>> >>> Am Donnerstag, 19. März 2020, 16:13:39 CET schrieb Katsuhiro Suzuki: >>>> This patch removes conflicted pinctrl settings uart2 and pwm 2/3 >>>> from common rk3288.dtsi and moves exist uart2 pinctrl settings >>>> into each rk3288*.dts files. >>>> >>>> - pwm2_pin : use GPIO7_C6 >>>> - pwm3_pin : use GPIO7_C7 >>>> - uart2_xfer: use GPIO7_C6, GPIO7_C7 >>> >>> Board files only ever get to enable either pwm2/3 or uart2, >>> not both at once - so I'm not sure where you see conflicts. >>> >> >> At first, I think so too. But I've saw this message when booting. >> >> ---------- >> [ 2.435504] rockchip-pinctrl pinctrl: pin gpio7-22 already requested by ff680020.pwm; cannot claim for ff690000.serial >> [ 2.447506] rockchip-pinctrl pinctrl: pin-238 (ff690000.serial) status -22 >> [ 2.455198] rockchip-pinctrl pinctrl: could not request pin 238 (gpio7-22) from group uart2-xfer on device rockchip-pinctrl >> ---------- >> >> And it seems that uart2(ttyS2) is not working correctly. >> >> ---------- >> # cat /dev/ttyS2 >> cat: /dev/ttyS2: Input/output error >> ---------- >> >> I'm using newest linux-next, make defconfig and CONFIG_MODULES = n setting >> with TinkerBoard. Can I resolve this issue in other ways? > > Do you perhaps have a DT overlay or bootloader script enabling pwm2? (try `cat /sys/firmware/devicetree/base/pwm@ff680020/status` for a sanity check). FWIW I don't recall ever seeing this on my RK3288 box. > > (or of course maybe there's just some pinctrl bug in -next that's claiming configs for disabled devices) Ah, cat /sys/firmware/devicetree/base/pwm@ff680020/status is 'okay'. So bootloader or someone enables pwm2 on booting. I'll check settings of bootloader of my board. Thanks a lot! Best Regards, Katsuhiro Suzuki > > Robin. > >> >> Best Regards, >> Katsuhiro Suzuki >> >> >>> And of course there are alternative pins to use, if you need uart2 >>> you need both pins in uart-pinmux and if you need either as pwm, then >>> the board by design just can't use them as uart2. >>> >>> So pin setting should stay where they are, as there really is no conflict. >>> >>> >>> Heiko >>> >>> >>> >>>> Currently uart2 rk3288 user is the following: >>>> >>>> - rk3288-evb.dtsi:&uart2 { >>>> - rk3288-firefly-reload.dts:&uart2 { >>>> - rk3288-firefly.dtsi:&uart2 { >>>> - rk3288-miqi.dts:&uart2 { >>>> - rk3288-phycore-rdk.dts:&uart2 { >>>> - rk3288-popmetal.dts:&uart2 { >>>> - rk3288-r89.dts:&uart2 { >>>> - rk3288-rock2-square.dts:&uart2 { >>>> - rk3288-tinker.dtsi:&uart2 { >>>> - rk3288-veyron.dtsi:&uart2 { >>>> - rk3288-vyasa.dts:&uart2 { >>>> >>>> And no one is using pwm2 nor pwm3. >>>> >>>> Signed-off-by: Katsuhiro Suzuki <katsuhiro@katsuster.net> >>>> --- >>>> arch/arm/boot/dts/rk3288-evb.dtsi | 2 ++ >>>> arch/arm/boot/dts/rk3288-firefly-reload.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-firefly.dtsi | 2 ++ >>>> arch/arm/boot/dts/rk3288-miqi.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-phycore-rdk.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-popmetal.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-r89.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-rock2-square.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288-tinker.dtsi | 2 ++ >>>> arch/arm/boot/dts/rk3288-veyron.dtsi | 2 ++ >>>> arch/arm/boot/dts/rk3288-vyasa.dts | 2 ++ >>>> arch/arm/boot/dts/rk3288.dtsi | 6 ------ >>>> 12 files changed, 22 insertions(+), 6 deletions(-) >>>> >>>> diff --git a/arch/arm/boot/dts/rk3288-evb.dtsi b/arch/arm/boot/dts/rk3288-evb.dtsi >>>> index 018802df4c0e..74091f831ecf 100644 >>>> --- a/arch/arm/boot/dts/rk3288-evb.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288-evb.dtsi >>>> @@ -285,6 +285,8 @@ &uart1 { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-firefly-reload.dts b/arch/arm/boot/dts/rk3288-firefly-reload.dts >>>> index 8c38bda21a7c..b0c976c8e35b 100644 >>>> --- a/arch/arm/boot/dts/rk3288-firefly-reload.dts >>>> +++ b/arch/arm/boot/dts/rk3288-firefly-reload.dts >>>> @@ -283,6 +283,8 @@ &uart1 { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-firefly.dtsi b/arch/arm/boot/dts/rk3288-firefly.dtsi >>>> index 5e0a19004e46..1632cc083c12 100644 >>>> --- a/arch/arm/boot/dts/rk3288-firefly.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288-firefly.dtsi >>>> @@ -532,6 +532,8 @@ &uart1 { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-miqi.dts b/arch/arm/boot/dts/rk3288-miqi.dts >>>> index c41d012c8850..2c0ed37fde80 100644 >>>> --- a/arch/arm/boot/dts/rk3288-miqi.dts >>>> +++ b/arch/arm/boot/dts/rk3288-miqi.dts >>>> @@ -379,6 +379,8 @@ &tsadc { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-phycore-rdk.dts b/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>>> index 1e33859de484..6532c1ac43cd 100644 >>>> --- a/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>>> +++ b/arch/arm/boot/dts/rk3288-phycore-rdk.dts >>>> @@ -244,6 +244,8 @@ &uart0 { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-popmetal.dts b/arch/arm/boot/dts/rk3288-popmetal.dts >>>> index 6a51940398b5..f18306bd9e6e 100644 >>>> --- a/arch/arm/boot/dts/rk3288-popmetal.dts >>>> +++ b/arch/arm/boot/dts/rk3288-popmetal.dts >>>> @@ -481,6 +481,8 @@ &uart1 { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-r89.dts b/arch/arm/boot/dts/rk3288-r89.dts >>>> index a258c7ae5329..02d2f5cfe201 100644 >>>> --- a/arch/arm/boot/dts/rk3288-r89.dts >>>> +++ b/arch/arm/boot/dts/rk3288-r89.dts >>>> @@ -340,6 +340,8 @@ &uart1 { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-rock2-square.dts b/arch/arm/boot/dts/rk3288-rock2-square.dts >>>> index cdcdc921ee09..a44290e882be 100644 >>>> --- a/arch/arm/boot/dts/rk3288-rock2-square.dts >>>> +++ b/arch/arm/boot/dts/rk3288-rock2-square.dts >>>> @@ -264,6 +264,8 @@ &spdif { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-tinker.dtsi b/arch/arm/boot/dts/rk3288-tinker.dtsi >>>> index acfaa12ec239..0327119f71b4 100644 >>>> --- a/arch/arm/boot/dts/rk3288-tinker.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288-tinker.dtsi >>>> @@ -500,6 +500,8 @@ &uart1 { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-veyron.dtsi b/arch/arm/boot/dts/rk3288-veyron.dtsi >>>> index 54a6838d73f5..baa44d00e49a 100644 >>>> --- a/arch/arm/boot/dts/rk3288-veyron.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288-veyron.dtsi >>>> @@ -412,6 +412,8 @@ &uart1 { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288-vyasa.dts b/arch/arm/boot/dts/rk3288-vyasa.dts >>>> index 385dd59393e1..aa50cdc7f839 100644 >>>> --- a/arch/arm/boot/dts/rk3288-vyasa.dts >>>> +++ b/arch/arm/boot/dts/rk3288-vyasa.dts >>>> @@ -398,6 +398,8 @@ &tsadc { >>>> }; >>>> &uart2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart2_xfer>; >>>> status = "okay"; >>>> }; >>>> diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi >>>> index 0cd88774db95..4c1f8cabb5eb 100644 >>>> --- a/arch/arm/boot/dts/rk3288.dtsi >>>> +++ b/arch/arm/boot/dts/rk3288.dtsi >>>> @@ -450,8 +450,6 @@ uart2: serial@ff690000 { >>>> reg-io-width = <4>; >>>> clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>; >>>> clock-names = "baudclk", "apb_pclk"; >>>> - pinctrl-names = "default"; >>>> - pinctrl-0 = <&uart2_xfer>; >>>> status = "disabled"; >>>> }; >>>> @@ -706,8 +704,6 @@ pwm2: pwm@ff680020 { >>>> compatible = "rockchip,rk3288-pwm"; >>>> reg = <0x0 0xff680020 0x0 0x10>; >>>> #pwm-cells = <3>; >>>> - pinctrl-names = "default"; >>>> - pinctrl-0 = <&pwm2_pin>; >>>> clocks = <&cru PCLK_RKPWM>; >>>> clock-names = "pwm"; >>>> status = "disabled"; >>>> @@ -717,8 +713,6 @@ pwm3: pwm@ff680030 { >>>> compatible = "rockchip,rk3288-pwm"; >>>> reg = <0x0 0xff680030 0x0 0x10>; >>>> #pwm-cells = <3>; >>>> - pinctrl-names = "default"; >>>> - pinctrl-0 = <&pwm3_pin>; >>>> clocks = <&cru PCLK_RKPWM>; >>>> clock-names = "pwm"; >>>> status = "disabled"; >>>> >>> >>> >>> >>> >>> >> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel _______________________________________________ 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] 7+ messages in thread
end of thread, other threads:[~2020-03-19 18:31 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-03-19 15:13 [PATCH] ARM: dts: rockchip: move uart2 pinctrl settings to each dts for rk3288 Katsuhiro Suzuki 2020-03-19 15:25 ` Heiko Stübner 2020-03-19 15:58 ` Katsuhiro Suzuki 2020-03-19 16:17 ` Heiko Stübner 2020-03-19 18:31 ` Katsuhiro Suzuki 2020-03-19 16:29 ` Robin Murphy 2020-03-19 18:31 ` Katsuhiro Suzuki
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).