linux-rockchip.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/5] Indiedroid Nova Devicetree Updates
@ 2025-11-05 20:57 Chris Morgan
  2025-11-05 20:57 ` [PATCH 1/5] arm64: dts: rockchip: Add device type for Indiedroid Nova Chris Morgan
                   ` (5 more replies)
  0 siblings, 6 replies; 11+ messages in thread
From: Chris Morgan @ 2025-11-05 20:57 UTC (permalink / raw)
  To: linux-rockchip
  Cc: devicetree, sebastian.reichel, damon.ding, jbx6244, heiko,
	conor+dt, krzk+dt, robh, Chris Morgan

From: Chris Morgan <macromorgan@hotmail.com>

This series fixes up a few remaining issues with the devicetree for the
Indiedroid Nova. Namely it adds or updates a few pinctrl definitions,
it removes a few workarounds that no longer appear to be necessary for
eMMC and bluetooth, it adds support for the NPU, and it sets the device
type as embedded.

Chris Morgan (5):
  arm64: dts: rockchip: Add device type for Indiedroid Nova.
  arm64: dts: rockchip: Add Additional pinctrl defs for Indiedroid Nova
  arm64: dts: rockchip: enable NPU on Indiedroid Nova
  arm64: dts: rockchip: Enable HS400 for Indiedroid Nova
  arm64: dts: rockchip: Fix DMA for Indiedroid Nova Bluetooth

 .../dts/rockchip/rk3588s-indiedroid-nova.dts  | 85 ++++++++++++++-----
 1 file changed, 65 insertions(+), 20 deletions(-)

-- 
2.43.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH 1/5] arm64: dts: rockchip: Add device type for Indiedroid Nova.
  2025-11-05 20:57 [PATCH 0/5] Indiedroid Nova Devicetree Updates Chris Morgan
@ 2025-11-05 20:57 ` Chris Morgan
  2025-11-05 20:57 ` [PATCH 2/5] arm64: dts: rockchip: Add Additional pinctrl defs " Chris Morgan
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 11+ messages in thread
From: Chris Morgan @ 2025-11-05 20:57 UTC (permalink / raw)
  To: linux-rockchip
  Cc: devicetree, sebastian.reichel, damon.ding, jbx6244, heiko,
	conor+dt, krzk+dt, robh, Chris Morgan

From: Chris Morgan <macromorgan@hotmail.com>

Update the devicetree node to note the device category of the
Indiedroid Nova as an embedded device.

Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
---
 arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
index 4ec7bc4a9e96..debab7732b71 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
@@ -11,6 +11,7 @@
 
 / {
 	model = "Indiedroid Nova";
+	chassis-type = "embedded";
 	compatible = "indiedroid,nova", "rockchip,rk3588s";
 
 	adc-keys-0 {
-- 
2.43.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 2/5] arm64: dts: rockchip: Add Additional pinctrl defs for Indiedroid Nova
  2025-11-05 20:57 [PATCH 0/5] Indiedroid Nova Devicetree Updates Chris Morgan
  2025-11-05 20:57 ` [PATCH 1/5] arm64: dts: rockchip: Add device type for Indiedroid Nova Chris Morgan
@ 2025-11-05 20:57 ` Chris Morgan
  2025-11-06 19:45   ` Heiko Stuebner
  2025-11-05 20:57 ` [PATCH 3/5] arm64: dts: rockchip: enable NPU on " Chris Morgan
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 11+ messages in thread
From: Chris Morgan @ 2025-11-05 20:57 UTC (permalink / raw)
  To: linux-rockchip
  Cc: devicetree, sebastian.reichel, damon.ding, jbx6244, heiko,
	conor+dt, krzk+dt, robh, Chris Morgan

From: Chris Morgan <macromorgan@hotmail.com>

Define the pinctrl nodes for the WiFi interrupts, correct the pinctrl
for the ethernet according to the schematic, and add the clk32k_in
control for the RTC. Add the correct regulator mapping for the PCIE
too.

Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
---
 .../dts/rockchip/rk3588s-indiedroid-nova.dts  | 43 ++++++++++++-------
 1 file changed, 28 insertions(+), 15 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
index debab7732b71..f40782b6c7db 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
@@ -406,7 +406,7 @@ rtc_hym8563: rtc@51 {
 		clock-output-names = "hym8563";
 		interrupt-parent = <&gpio0>;
 		interrupts = <RK_PB0 IRQ_TYPE_LEVEL_LOW>;
-		pinctrl-0 = <&hym8563_int>;
+		pinctrl-0 = <&hym8563_int>, <&clk32k_in>;
 		pinctrl-names = "default";
 		wakeup-source;
 	};
@@ -459,8 +459,11 @@ &i2s5_8ch {
 };
 
 &pcie2x1l2 {
-	pinctrl-0 = <&rtl8111_perstb>;
+	pinctrl-0 = <&pcie20x1m0_perstn>, <&pcie20x1m0_clkreqn>,
+		    <&pcie20x1m0_waken>;
 	pinctrl-names = "default";
+	reset-gpios = <&gpio3 RK_PD1 GPIO_ACTIVE_HIGH>;
+	vpcie3v3-supply = <&vcc_3v3_s3>;
 	status = "okay";
 };
 
@@ -486,12 +489,6 @@ bt_wake_host: bt-wake-host {
 		};
 	};
 
-	ethernet-pins {
-		rtl8111_perstb: rtl8111-perstb {
-			rockchip,pins = <3 RK_PD1 RK_FUNC_GPIO &pcfg_pull_up>;
-		};
-	};
-
 	hym8563 {
 
 		hym8563_int: hym8563-int {
@@ -500,13 +497,6 @@ hym8563_int: hym8563-int {
 		};
 	};
 
-	sdio-pwrseq {
-		wifi_enable_h: wifi-enable-h {
-			rockchip,pins =
-				<0 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>;
-		};
-	};
-
 	usb-typec {
 		usbc0_int: usbc0-int {
 			rockchip,pins =
@@ -518,6 +508,18 @@ typec5v_pwren: typec5v-pwren {
 				<4 RK_PA5 RK_FUNC_GPIO &pcfg_pull_none>;
 		};
 	};
+
+	wifi {
+		wifi_enable_h: wifi-enable-h {
+			rockchip,pins =
+				<0 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>;
+		};
+
+		wifi_host_wake_irq: wifi-host-wake-irq {
+			rockchip,pins =
+				<0 RK_PA0 RK_FUNC_GPIO &pcfg_pull_down>;
+		};
+	};
 };
 
 &saradc {
@@ -538,6 +540,7 @@ &sdhci {
 };
 
 &sdio {
+	#address-cells = <1>;
 	bus-width = <4>;
 	cap-sd-highspeed;
 	cap-sdio-irq;
@@ -549,9 +552,19 @@ &sdio {
 	no-sd;
 	non-removable;
 	sd-uhs-sdr104;
+	#size-cells = <0>;
 	vmmc-supply = <&vcc_3v3_s3>;
 	vqmmc-supply = <&vcc_1v8_s3>;
 	status = "okay";
+
+	sdio_wifi: wifi@1 {
+		reg = <1>;
+		interrupt-parent = <&gpio0>;
+		interrupts = <RK_PA0 IRQ_TYPE_LEVEL_LOW>;
+		interrupt-names = "host-wake";
+		pinctrl-0 = <&wifi_host_wake_irq>;
+		pinctrl-names = "default";
+	};
 };
 
 &sdmmc {
-- 
2.43.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 3/5] arm64: dts: rockchip: enable NPU on Indiedroid Nova
  2025-11-05 20:57 [PATCH 0/5] Indiedroid Nova Devicetree Updates Chris Morgan
  2025-11-05 20:57 ` [PATCH 1/5] arm64: dts: rockchip: Add device type for Indiedroid Nova Chris Morgan
  2025-11-05 20:57 ` [PATCH 2/5] arm64: dts: rockchip: Add Additional pinctrl defs " Chris Morgan
@ 2025-11-05 20:57 ` Chris Morgan
  2025-11-05 20:57 ` [PATCH 4/5] arm64: dts: rockchip: Enable HS400 for " Chris Morgan
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 11+ messages in thread
From: Chris Morgan @ 2025-11-05 20:57 UTC (permalink / raw)
  To: linux-rockchip
  Cc: devicetree, sebastian.reichel, damon.ding, jbx6244, heiko,
	conor+dt, krzk+dt, robh, Chris Morgan

From: Chris Morgan <macromorgan@hotmail.com>

Enable the NPU on the Indiedroid Nova.

Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
---
 .../dts/rockchip/rk3588s-indiedroid-nova.dts  | 34 +++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
index f40782b6c7db..62c8c8f68713 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
@@ -471,6 +471,10 @@ &pd_gpu {
 	domain-supply = <&vdd_gpu_s0>;
 };
 
+&pd_npu {
+	domain-supply = <&vdd_npu_s0>;
+};
+
 &pinctrl {
 	bluetooth-pins {
 		bt_reset: bt-reset {
@@ -522,6 +526,36 @@ wifi_host_wake_irq: wifi-host-wake-irq {
 	};
 };
 
+&rknn_core_0 {
+	npu-supply = <&vdd_npu_s0>;
+	sram-supply = <&vdd_npu_s0>;
+	status = "okay";
+};
+
+&rknn_core_1 {
+	npu-supply = <&vdd_npu_s0>;
+	sram-supply = <&vdd_npu_s0>;
+	status = "okay";
+};
+
+&rknn_core_2 {
+	npu-supply = <&vdd_npu_s0>;
+	sram-supply = <&vdd_npu_s0>;
+	status = "okay";
+};
+
+&rknn_mmu_0 {
+	status = "okay";
+};
+
+&rknn_mmu_1 {
+	status = "okay";
+};
+
+&rknn_mmu_2 {
+	status = "okay";
+};
+
 &saradc {
 	vref-supply = <&vcca_1v8_s0>;
 	status = "okay";
-- 
2.43.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 4/5] arm64: dts: rockchip: Enable HS400 for Indiedroid Nova
  2025-11-05 20:57 [PATCH 0/5] Indiedroid Nova Devicetree Updates Chris Morgan
                   ` (2 preceding siblings ...)
  2025-11-05 20:57 ` [PATCH 3/5] arm64: dts: rockchip: enable NPU on " Chris Morgan
@ 2025-11-05 20:57 ` Chris Morgan
  2025-11-05 23:18   ` Heiko Stuebner
  2025-11-05 20:57 ` [PATCH 5/5] arm64: dts: rockchip: Fix DMA for Indiedroid Nova Bluetooth Chris Morgan
  2025-11-06 22:58 ` (subset) [PATCH 0/5] Indiedroid Nova Devicetree Updates Heiko Stuebner
  5 siblings, 1 reply; 11+ messages in thread
From: Chris Morgan @ 2025-11-05 20:57 UTC (permalink / raw)
  To: linux-rockchip
  Cc: devicetree, sebastian.reichel, damon.ding, jbx6244, heiko,
	conor+dt, krzk+dt, robh, Chris Morgan

From: Chris Morgan <macromorgan@hotmail.com>

Once the DLL_STRBIN_TAPNUM_DEFAULT is set to the correct value [1], the
workaround to ensure stable eMMC operation is no longer needed. I
have observed testing this that hdparm went from:
"Timing buffered disk reads: 502 MB in  3.00 seconds = 167.14 MB/sec"
to the following:
"Timing buffered disk reads: 832 MB in  3.00 seconds = 277.32 MB/sec"

[1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/drivers/mmc/host/sdhci-of-dwcmshc.c?id=a28352cf2d2f8380e7aca8cb61682396dca7a991

Fixes: 3900160e164b ("arm64: dts: rockchip: Add Indiedroid Nova board")
Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
---
 arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
index 62c8c8f68713..0796ba6d2504 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
@@ -561,10 +561,10 @@ &saradc {
 	status = "okay";
 };
 
-/* HS400 modes seemed to cause io errors. */
 &sdhci {
 	bus-width = <8>;
-	no-mmc-hs400;
+	mmc-hs400-1_8v;
+	mmc-hs400-enhanced-strobe;
 	no-sd;
 	no-sdio;
 	non-removable;
-- 
2.43.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 5/5] arm64: dts: rockchip: Fix DMA for Indiedroid Nova Bluetooth
  2025-11-05 20:57 [PATCH 0/5] Indiedroid Nova Devicetree Updates Chris Morgan
                   ` (3 preceding siblings ...)
  2025-11-05 20:57 ` [PATCH 4/5] arm64: dts: rockchip: Enable HS400 for " Chris Morgan
@ 2025-11-05 20:57 ` Chris Morgan
  2025-11-06 22:58 ` (subset) [PATCH 0/5] Indiedroid Nova Devicetree Updates Heiko Stuebner
  5 siblings, 0 replies; 11+ messages in thread
From: Chris Morgan @ 2025-11-05 20:57 UTC (permalink / raw)
  To: linux-rockchip
  Cc: devicetree, sebastian.reichel, damon.ding, jbx6244, heiko,
	conor+dt, krzk+dt, robh, Chris Morgan

From: Chris Morgan <macromorgan@hotmail.com>

When the device was first added, there was a problem with the bluetooth
controller that manifested when DMA was enabled for the underlying UART
interface. At some point in the intervening time the problem appears to
have been resolved. Add the UART rx and tx channels back to re-enable
UART.

Fixes: 3900160e164b ("arm64: dts: rockchip: Add Indiedroid Nova board")
Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
---
 arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
index 0796ba6d2504..dee053afc6b6 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
@@ -944,12 +944,9 @@ &uart2 {
 	status = "okay";
 };
 
-/* DMA seems to interfere with bluetooth device normal operation. */
 &uart9 {
 	pinctrl-0 = <&uart9m2_xfer>, <&uart9m2_ctsn>, <&uart9m2_rtsn>;
 	pinctrl-names = "default";
-	/delete-property/ dma-names;
-	/delete-property/ dmas;
 	uart-has-rtscts;
 	status = "okay";
 
-- 
2.43.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH 4/5] arm64: dts: rockchip: Enable HS400 for Indiedroid Nova
  2025-11-05 20:57 ` [PATCH 4/5] arm64: dts: rockchip: Enable HS400 for " Chris Morgan
@ 2025-11-05 23:18   ` Heiko Stuebner
  2025-11-06  1:06     ` Chris Morgan
  0 siblings, 1 reply; 11+ messages in thread
From: Heiko Stuebner @ 2025-11-05 23:18 UTC (permalink / raw)
  To: linux-rockchip, Chris Morgan
  Cc: devicetree, sebastian.reichel, damon.ding, jbx6244, conor+dt,
	krzk+dt, robh, Chris Morgan

Am Mittwoch, 5. November 2025, 21:57:07 Mitteleuropäische Normalzeit schrieb Chris Morgan:
> From: Chris Morgan <macromorgan@hotmail.com>
> 
> Once the DLL_STRBIN_TAPNUM_DEFAULT is set to the correct value [1], the
> workaround to ensure stable eMMC operation is no longer needed. I
> have observed testing this that hdparm went from:
> "Timing buffered disk reads: 502 MB in  3.00 seconds = 167.14 MB/sec"
> to the following:
> "Timing buffered disk reads: 832 MB in  3.00 seconds = 277.32 MB/sec"
> 
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/drivers/mmc/host/sdhci-of-dwcmshc.c?id=a28352cf2d2f8380e7aca8cb61682396dca7a991
> 
> Fixes: 3900160e164b ("arm64: dts: rockchip: Add Indiedroid Nova board")

This needs the patch you linked to, to actually work, so I'll drop
the fixes tag when applying, because this should likely not be
backported to older kernels.

Same reasoning for patch 5.


Heiko

> Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
> ---
>  arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> index 62c8c8f68713..0796ba6d2504 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> @@ -561,10 +561,10 @@ &saradc {
>  	status = "okay";
>  };
>  
> -/* HS400 modes seemed to cause io errors. */
>  &sdhci {
>  	bus-width = <8>;
> -	no-mmc-hs400;
> +	mmc-hs400-1_8v;
> +	mmc-hs400-enhanced-strobe;
>  	no-sd;
>  	no-sdio;
>  	non-removable;
> 





_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH 4/5] arm64: dts: rockchip: Enable HS400 for Indiedroid Nova
  2025-11-05 23:18   ` Heiko Stuebner
@ 2025-11-06  1:06     ` Chris Morgan
  0 siblings, 0 replies; 11+ messages in thread
From: Chris Morgan @ 2025-11-06  1:06 UTC (permalink / raw)
  To: Heiko Stuebner
  Cc: linux-rockchip, Chris Morgan, devicetree, sebastian.reichel,
	damon.ding, jbx6244, conor+dt, krzk+dt, robh

On Thu, Nov 06, 2025 at 12:18:51AM +0100, Heiko Stuebner wrote:
> Am Mittwoch, 5. November 2025, 21:57:07 Mitteleuropäische Normalzeit schrieb Chris Morgan:
> > From: Chris Morgan <macromorgan@hotmail.com>
> > 
> > Once the DLL_STRBIN_TAPNUM_DEFAULT is set to the correct value [1], the
> > workaround to ensure stable eMMC operation is no longer needed. I
> > have observed testing this that hdparm went from:
> > "Timing buffered disk reads: 502 MB in  3.00 seconds = 167.14 MB/sec"
> > to the following:
> > "Timing buffered disk reads: 832 MB in  3.00 seconds = 277.32 MB/sec"
> > 
> > [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/drivers/mmc/host/sdhci-of-dwcmshc.c?id=a28352cf2d2f8380e7aca8cb61682396dca7a991
> > 
> > Fixes: 3900160e164b ("arm64: dts: rockchip: Add Indiedroid Nova board")
> 
> This needs the patch you linked to, to actually work, so I'll drop
> the fixes tag when applying, because this should likely not be
> backported to older kernels.
> 
> Same reasoning for patch 5.

Thank you, I wasn't sure about that or not. I can say with certanty
you don't want the eMMC device tree changes prior to that patch going
through or bad things will happen.

Chris

> 
> 
> Heiko
> 
> > Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
> > ---
> >  arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > index 62c8c8f68713..0796ba6d2504 100644
> > --- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > +++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > @@ -561,10 +561,10 @@ &saradc {
> >  	status = "okay";
> >  };
> >  
> > -/* HS400 modes seemed to cause io errors. */
> >  &sdhci {
> >  	bus-width = <8>;
> > -	no-mmc-hs400;
> > +	mmc-hs400-1_8v;
> > +	mmc-hs400-enhanced-strobe;
> >  	no-sd;
> >  	no-sdio;
> >  	non-removable;
> > 
> 
> 
> 
> 

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH 2/5] arm64: dts: rockchip: Add Additional pinctrl defs for Indiedroid Nova
  2025-11-05 20:57 ` [PATCH 2/5] arm64: dts: rockchip: Add Additional pinctrl defs " Chris Morgan
@ 2025-11-06 19:45   ` Heiko Stuebner
  2025-11-06 23:45     ` Chris Morgan
  0 siblings, 1 reply; 11+ messages in thread
From: Heiko Stuebner @ 2025-11-06 19:45 UTC (permalink / raw)
  To: linux-rockchip, Chris Morgan
  Cc: devicetree, sebastian.reichel, damon.ding, jbx6244, conor+dt,
	krzk+dt, robh, Chris Morgan

Hi Chris,

Am Mittwoch, 5. November 2025, 21:57:05 Mitteleuropäische Normalzeit schrieb Chris Morgan:
> From: Chris Morgan <macromorgan@hotmail.com>
> 
> Define the pinctrl nodes for the WiFi interrupts, correct the pinctrl
> for the ethernet according to the schematic, and add the clk32k_in
> control for the RTC. Add the correct regulator mapping for the PCIE
> too.

way too much completely different things to stuff into one patch.

- Adding the sdio wifi is one
- ethernet is next ... you're not fixing, but removing the pinctrl?
- pcie2 fixes is another one
- and the clkin32k as well


Heiko

> 
> Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
> ---
>  .../dts/rockchip/rk3588s-indiedroid-nova.dts  | 43 ++++++++++++-------
>  1 file changed, 28 insertions(+), 15 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> index debab7732b71..f40782b6c7db 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> @@ -406,7 +406,7 @@ rtc_hym8563: rtc@51 {
>  		clock-output-names = "hym8563";
>  		interrupt-parent = <&gpio0>;
>  		interrupts = <RK_PB0 IRQ_TYPE_LEVEL_LOW>;
> -		pinctrl-0 = <&hym8563_int>;
> +		pinctrl-0 = <&hym8563_int>, <&clk32k_in>;
>  		pinctrl-names = "default";
>  		wakeup-source;
>  	};
> @@ -459,8 +459,11 @@ &i2s5_8ch {
>  };
>  
>  &pcie2x1l2 {
> -	pinctrl-0 = <&rtl8111_perstb>;
> +	pinctrl-0 = <&pcie20x1m0_perstn>, <&pcie20x1m0_clkreqn>,
> +		    <&pcie20x1m0_waken>;
>  	pinctrl-names = "default";
> +	reset-gpios = <&gpio3 RK_PD1 GPIO_ACTIVE_HIGH>;
> +	vpcie3v3-supply = <&vcc_3v3_s3>;
>  	status = "okay";
>  };
>  
> @@ -486,12 +489,6 @@ bt_wake_host: bt-wake-host {
>  		};
>  	};
>  
> -	ethernet-pins {
> -		rtl8111_perstb: rtl8111-perstb {
> -			rockchip,pins = <3 RK_PD1 RK_FUNC_GPIO &pcfg_pull_up>;
> -		};
> -	};
> -
>  	hym8563 {
>  
>  		hym8563_int: hym8563-int {
> @@ -500,13 +497,6 @@ hym8563_int: hym8563-int {
>  		};
>  	};
>  
> -	sdio-pwrseq {
> -		wifi_enable_h: wifi-enable-h {
> -			rockchip,pins =
> -				<0 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>;
> -		};
> -	};
> -
>  	usb-typec {
>  		usbc0_int: usbc0-int {
>  			rockchip,pins =
> @@ -518,6 +508,18 @@ typec5v_pwren: typec5v-pwren {
>  				<4 RK_PA5 RK_FUNC_GPIO &pcfg_pull_none>;
>  		};
>  	};
> +
> +	wifi {
> +		wifi_enable_h: wifi-enable-h {
> +			rockchip,pins =
> +				<0 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>;
> +		};
> +
> +		wifi_host_wake_irq: wifi-host-wake-irq {
> +			rockchip,pins =
> +				<0 RK_PA0 RK_FUNC_GPIO &pcfg_pull_down>;
> +		};
> +	};
>  };
>  
>  &saradc {
> @@ -538,6 +540,7 @@ &sdhci {
>  };
>  
>  &sdio {
> +	#address-cells = <1>;
>  	bus-width = <4>;
>  	cap-sd-highspeed;
>  	cap-sdio-irq;
> @@ -549,9 +552,19 @@ &sdio {
>  	no-sd;
>  	non-removable;
>  	sd-uhs-sdr104;
> +	#size-cells = <0>;
>  	vmmc-supply = <&vcc_3v3_s3>;
>  	vqmmc-supply = <&vcc_1v8_s3>;
>  	status = "okay";
> +
> +	sdio_wifi: wifi@1 {
> +		reg = <1>;
> +		interrupt-parent = <&gpio0>;
> +		interrupts = <RK_PA0 IRQ_TYPE_LEVEL_LOW>;
> +		interrupt-names = "host-wake";
> +		pinctrl-0 = <&wifi_host_wake_irq>;
> +		pinctrl-names = "default";
> +	};
>  };
>  
>  &sdmmc {
> 





_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: (subset) [PATCH 0/5] Indiedroid Nova Devicetree Updates
  2025-11-05 20:57 [PATCH 0/5] Indiedroid Nova Devicetree Updates Chris Morgan
                   ` (4 preceding siblings ...)
  2025-11-05 20:57 ` [PATCH 5/5] arm64: dts: rockchip: Fix DMA for Indiedroid Nova Bluetooth Chris Morgan
@ 2025-11-06 22:58 ` Heiko Stuebner
  5 siblings, 0 replies; 11+ messages in thread
From: Heiko Stuebner @ 2025-11-06 22:58 UTC (permalink / raw)
  To: linux-rockchip, Chris Morgan
  Cc: Heiko Stuebner, devicetree, sebastian.reichel, damon.ding,
	jbx6244, conor+dt, krzk+dt, robh, Chris Morgan


On Wed, 05 Nov 2025 14:57:03 -0600, Chris Morgan wrote:
> This series fixes up a few remaining issues with the devicetree for the
> Indiedroid Nova. Namely it adds or updates a few pinctrl definitions,
> it removes a few workarounds that no longer appear to be necessary for
> eMMC and bluetooth, it adds support for the NPU, and it sets the device
> type as embedded.
> 
> Chris Morgan (5):
>   arm64: dts: rockchip: Add device type for Indiedroid Nova.
>   arm64: dts: rockchip: Add Additional pinctrl defs for Indiedroid Nova
>   arm64: dts: rockchip: enable NPU on Indiedroid Nova
>   arm64: dts: rockchip: Enable HS400 for Indiedroid Nova
>   arm64: dts: rockchip: Fix DMA for Indiedroid Nova Bluetooth
> 
> [...]

Applied, thanks!

[1/5] arm64: dts: rockchip: Add device type for Indiedroid Nova.
      commit: 318bc4b5ad229435abbb7b9a7c8b1876fae311e7
[3/5] arm64: dts: rockchip: enable NPU on Indiedroid Nova
      commit: dfa4f7f5cb2c66fac5a17d63e5fcf1bf50ea7344
[4/5] arm64: dts: rockchip: Enable HS400 for Indiedroid Nova
      commit: cce837b7a3599dc83cd9c08afc07b076f127354b
[5/5] arm64: dts: rockchip: Fix DMA for Indiedroid Nova Bluetooth
      commit: c6c76445c30bb73fd4cfba8f4742e642dcfe90f5

Best regards,
-- 
Heiko Stuebner <heiko@sntech.de>

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH 2/5] arm64: dts: rockchip: Add Additional pinctrl defs for Indiedroid Nova
  2025-11-06 19:45   ` Heiko Stuebner
@ 2025-11-06 23:45     ` Chris Morgan
  0 siblings, 0 replies; 11+ messages in thread
From: Chris Morgan @ 2025-11-06 23:45 UTC (permalink / raw)
  To: Heiko Stuebner
  Cc: linux-rockchip, Chris Morgan, devicetree, sebastian.reichel,
	damon.ding, jbx6244, conor+dt, krzk+dt, robh

On Thu, Nov 06, 2025 at 08:45:33PM +0100, Heiko Stuebner wrote:
> Hi Chris,
> 
> Am Mittwoch, 5. November 2025, 21:57:05 Mitteleuropäische Normalzeit schrieb Chris Morgan:
> > From: Chris Morgan <macromorgan@hotmail.com>
> > 
> > Define the pinctrl nodes for the WiFi interrupts, correct the pinctrl
> > for the ethernet according to the schematic, and add the clk32k_in
> > control for the RTC. Add the correct regulator mapping for the PCIE
> > too.
> 
> way too much completely different things to stuff into one patch.

Sorry, to me this seemed like a few tiny fixes that, excluding the pcie
regulator addition, all related to correcting pinctrl definitions. I'll
be happy to split it up if need be. The SDIO wifi is really just adding
the interrupt pin... there was just a bit of stuff to add on top so I
could define it correctly; for ethernet I'm replacing the reset pin with
the PCIE port reset pin and adding other PCIE pins defined in the
schematic, and the clk32k_in is just adding another pin to note that the
RTC output is connected to that pin.

Thank you,
Chris

> 
> - Adding the sdio wifi is one
> - ethernet is next ... you're not fixing, but removing the pinctrl?
> - pcie2 fixes is another one
> - and the clkin32k as well
> 
> 
> Heiko
> 
> > 
> > Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
> > ---
> >  .../dts/rockchip/rk3588s-indiedroid-nova.dts  | 43 ++++++++++++-------
> >  1 file changed, 28 insertions(+), 15 deletions(-)
> > 
> > diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > index debab7732b71..f40782b6c7db 100644
> > --- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > +++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > @@ -406,7 +406,7 @@ rtc_hym8563: rtc@51 {
> >  		clock-output-names = "hym8563";
> >  		interrupt-parent = <&gpio0>;
> >  		interrupts = <RK_PB0 IRQ_TYPE_LEVEL_LOW>;
> > -		pinctrl-0 = <&hym8563_int>;
> > +		pinctrl-0 = <&hym8563_int>, <&clk32k_in>;
> >  		pinctrl-names = "default";
> >  		wakeup-source;
> >  	};
> > @@ -459,8 +459,11 @@ &i2s5_8ch {
> >  };
> >  
> >  &pcie2x1l2 {
> > -	pinctrl-0 = <&rtl8111_perstb>;
> > +	pinctrl-0 = <&pcie20x1m0_perstn>, <&pcie20x1m0_clkreqn>,
> > +		    <&pcie20x1m0_waken>;
> >  	pinctrl-names = "default";
> > +	reset-gpios = <&gpio3 RK_PD1 GPIO_ACTIVE_HIGH>;
> > +	vpcie3v3-supply = <&vcc_3v3_s3>;
> >  	status = "okay";
> >  };
> >  
> > @@ -486,12 +489,6 @@ bt_wake_host: bt-wake-host {
> >  		};
> >  	};
> >  
> > -	ethernet-pins {
> > -		rtl8111_perstb: rtl8111-perstb {
> > -			rockchip,pins = <3 RK_PD1 RK_FUNC_GPIO &pcfg_pull_up>;
> > -		};
> > -	};
> > -
> >  	hym8563 {
> >  
> >  		hym8563_int: hym8563-int {
> > @@ -500,13 +497,6 @@ hym8563_int: hym8563-int {
> >  		};
> >  	};
> >  
> > -	sdio-pwrseq {
> > -		wifi_enable_h: wifi-enable-h {
> > -			rockchip,pins =
> > -				<0 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>;
> > -		};
> > -	};
> > -
> >  	usb-typec {
> >  		usbc0_int: usbc0-int {
> >  			rockchip,pins =
> > @@ -518,6 +508,18 @@ typec5v_pwren: typec5v-pwren {
> >  				<4 RK_PA5 RK_FUNC_GPIO &pcfg_pull_none>;
> >  		};
> >  	};
> > +
> > +	wifi {
> > +		wifi_enable_h: wifi-enable-h {
> > +			rockchip,pins =
> > +				<0 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>;
> > +		};
> > +
> > +		wifi_host_wake_irq: wifi-host-wake-irq {
> > +			rockchip,pins =
> > +				<0 RK_PA0 RK_FUNC_GPIO &pcfg_pull_down>;
> > +		};
> > +	};
> >  };
> >  
> >  &saradc {
> > @@ -538,6 +540,7 @@ &sdhci {
> >  };
> >  
> >  &sdio {
> > +	#address-cells = <1>;
> >  	bus-width = <4>;
> >  	cap-sd-highspeed;
> >  	cap-sdio-irq;
> > @@ -549,9 +552,19 @@ &sdio {
> >  	no-sd;
> >  	non-removable;
> >  	sd-uhs-sdr104;
> > +	#size-cells = <0>;
> >  	vmmc-supply = <&vcc_3v3_s3>;
> >  	vqmmc-supply = <&vcc_1v8_s3>;
> >  	status = "okay";
> > +
> > +	sdio_wifi: wifi@1 {
> > +		reg = <1>;
> > +		interrupt-parent = <&gpio0>;
> > +		interrupts = <RK_PA0 IRQ_TYPE_LEVEL_LOW>;
> > +		interrupt-names = "host-wake";
> > +		pinctrl-0 = <&wifi_host_wake_irq>;
> > +		pinctrl-names = "default";
> > +	};
> >  };
> >  
> >  &sdmmc {
> > 
> 
> 
> 
> 

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2025-11-06 23:45 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-05 20:57 [PATCH 0/5] Indiedroid Nova Devicetree Updates Chris Morgan
2025-11-05 20:57 ` [PATCH 1/5] arm64: dts: rockchip: Add device type for Indiedroid Nova Chris Morgan
2025-11-05 20:57 ` [PATCH 2/5] arm64: dts: rockchip: Add Additional pinctrl defs " Chris Morgan
2025-11-06 19:45   ` Heiko Stuebner
2025-11-06 23:45     ` Chris Morgan
2025-11-05 20:57 ` [PATCH 3/5] arm64: dts: rockchip: enable NPU on " Chris Morgan
2025-11-05 20:57 ` [PATCH 4/5] arm64: dts: rockchip: Enable HS400 for " Chris Morgan
2025-11-05 23:18   ` Heiko Stuebner
2025-11-06  1:06     ` Chris Morgan
2025-11-05 20:57 ` [PATCH 5/5] arm64: dts: rockchip: Fix DMA for Indiedroid Nova Bluetooth Chris Morgan
2025-11-06 22:58 ` (subset) [PATCH 0/5] Indiedroid Nova Devicetree Updates Heiko Stuebner

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).