Thread (3 messages) 3 messages, 3 authors, 2026-01-04

Re: [PATCH v4 7/7] ARM: dts: rockchip: rk3506: Add pinctrl and rmio dtsi for rk3506

From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2025-12-28 10:37:40
Also in: linux-devicetree, linux-gpio, linux-rockchip, lkml

On 27/12/2025 12:49, Ye Zhang wrote:
+		/omit-if-no-ref/
+		rm_io28_touch_key_in6: rm-io28-touch-key-in6 {
+			rockchip,pins =
+				<1 RK_PC3 7 &pcfg_pull_none>;
+			rockchip,rmio-pins =
+				<0 28 49>;
+		};
+
+		/omit-if-no-ref/
+		rm_io28_touch_key_in7: rm-io28-touch-key-in7 {
+			rockchip,pins =
+				<1 RK_PC3 7 &pcfg_pull_none>;
+			rockchip,rmio-pins =
+				<0 28 50>;
+		};
+
+		/omit-if-no-ref/
+		rm_io28_sai0_mclk: rm-io28-sai0-mclk {
+			rockchip,pins =
+				<1 RK_PC3 7 &pcfg_pull_none>;
+			rockchip,rmio-pins =
+				<0 28 51>;
+		};
+
+		/omit-if-no-ref/
+		rm_io28_sai0_sclk: rm-io28-sai0-sclk {
+			rockchip,pins =
+				<1 RK_PC3 7 &pcfg_pull_none>;
+			rockchip,rmio-pins =
+				<0 28 52>;
+		};
+
+		/omit-if-no-ref/
+		rm_io28_sai0_lrck: rm-io28-sai0-lrck {
+			rockchip,pins =
+				<1 RK_PC3 7 &pcfg_pull_none>;
+			rockchip,rmio-pins =
+				<0 28 53>;
+		};
+
+		/omit-if-no-ref/
+		rm_io28_sai0_sdi0: rm-io28-sai0-sdi0 {
+			rockchip,pins =
+				<1 RK_PC3 7 &pcfg_pull_none>;
+			rockchip,rmio-pins =
+				<0 28 54>;
+		};
Why are you defining all pins? This is wrong, your driver has to do it,
not DTS. All these definitions when not used are just pointless.


Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help