[PATCH 1/1] riscv: dts: Correct the formatting issues about k230 dts

[PATCH 1/1] riscv: dts: Correct the formatting issues about k230 dts

[Jiayu Du]

Correct the errors in the spacing format, and move the aliases from
the SoC's dtsi to the board's dts, since it's more standard.

Signed-off-by: Jiayu Du <jiayu.riscv@isrc.iscas.ac.cn>
---

This patch is based on this initial support for Canaan Kendryte K230
series v6[1], and it is used to fix the [PATCH v6 10/11] of that series.
Although the latest version is v8, this problem still persists.
And this patch hasn't been merged into Conor's branch yet.

[1]: https://lore.kernel.org/all/tencent_DF5D7CD182AFDA188E0FB80E314A21038D08@qq.com/

Thanks for your time and review.

 arch/riscv/boot/dts/canaan/k230-canmv.dts | 4 ++++
 arch/riscv/boot/dts/canaan/k230-evb.dts   | 4 ++++
 arch/riscv/boot/dts/canaan/k230.dtsi      | 6 +-----
 3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/arch/riscv/boot/dts/canaan/k230-canmv.dts b/arch/riscv/boot/dts/canaan/k230-canmv.dts
index 9565915cea..593ca38e68 100644
--- a/arch/riscv/boot/dts/canaan/k230-canmv.dts
+++ b/arch/riscv/boot/dts/canaan/k230-canmv.dts
@@ -9,6 +9,10 @@ / {
 	model = "Canaan CanMV-K230";
 	compatible = "canaan,canmv-k230", "canaan,kendryte-k230";
 
+	aliases {
+		serial0 = &uart0;
+	};
+
 	chosen {
 		stdout-path = "serial0:115200n8";
 	};
diff --git a/arch/riscv/boot/dts/canaan/k230-evb.dts b/arch/riscv/boot/dts/canaan/k230-evb.dts
index f898b8e623..bfa53f2e24 100644
--- a/arch/riscv/boot/dts/canaan/k230-evb.dts
+++ b/arch/riscv/boot/dts/canaan/k230-evb.dts
@@ -9,6 +9,10 @@ / {
 	model = "Kendryte K230 EVB";
 	compatible = "canaan,k230-usip-lp3-evb", "canaan,kendryte-k230";
 
+	aliases {
+		serial0 = &uart0;
+	};
+
 	chosen {
 		stdout-path = "serial0:115200n8";
 	};
diff --git a/arch/riscv/boot/dts/canaan/k230.dtsi b/arch/riscv/boot/dts/canaan/k230.dtsi
index 7da4949894..6bc599f079 100644
--- a/arch/riscv/boot/dts/canaan/k230.dtsi
+++ b/arch/riscv/boot/dts/canaan/k230.dtsi
@@ -11,10 +11,6 @@ / {
 	#size-cells = <2>;
 	compatible = "canaan,kendryte-k230";
 
-	aliases {
-		serial0 = &uart0;
-	};
-
 	cpus {
 		#address-cells = <1>;
 		#size-cells = <0>;
@@ -72,7 +68,7 @@ soc {
 		ranges;
 
 		plic: interrupt-controller@f00000000 {
-			compatible = "canaan,k230-plic" ,"thead,c900-plic";
+			compatible = "canaan,k230-plic", "thead,c900-plic";
 			reg = <0xf 0x00000000 0x0 0x04000000>;
 			interrupts-extended = <&cpu0_intc 11>, <&cpu0_intc 9>;
 			interrupt-controller;
-- 
2.52.0


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

Re: [PATCH 1/1] riscv: dts: Correct the formatting issues about k230 dts

[Conor Dooley]

[-- Attachment #1.1: Type: text/plain, Size: 727 bytes --]

On Sat, Dec 27, 2025 at 05:23:54PM +0800, Jiayu Du wrote:
> Correct the errors in the spacing format, and move the aliases from
> the SoC's dtsi to the board's dts, since it's more standard.
> 
> Signed-off-by: Jiayu Du <jiayu.riscv@isrc.iscas.ac.cn>
> ---
> 
> This patch is based on this initial support for Canaan Kendryte K230
> series v6[1], and it is used to fix the [PATCH v6 10/11] of that series.
> Although the latest version is v8, this problem still persists.
> And this patch hasn't been merged into Conor's branch yet.
> 
> [1]: https://lore.kernel.org/all/tencent_DF5D7CD182AFDA188E0FB80E314A21038D08@qq.com/
> 
> Thanks for your time and review.

I put this on my k230-basic branch, thank you!

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 161 bytes --]

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