Re: [PATCH v5 2/2] loongarch: boot: dts: Add Loongson-2K0300 support

[Krzysztof Kozlowski]

On 22/03/2026 08:13, wjjsn wrote:
> From: wjjsn <2858482031@xxxxxx>
> 
> Signed-off-by: wjjsn <2858482031@xxxxxx>

I don't understand what is happening here. You sent bunch of unthreaded
patches, empty commit msgs, some are being duplicated (seems at least?)...

> ---
>  arch/loongarch/boot/dts/Makefile              |  2 +-
>  .../boot/dts/loongson-2k0300-ref.dts          | 34 +++++++
>  arch/loongarch/boot/dts/loongson-2k0300.dtsi  | 94 +++++++++++++++++++
>  3 files changed, 129 insertions(+), 1 deletion(-)
>  create mode 100644 arch/loongarch/boot/dts/loongson-2k0300-ref.dts
>  create mode 100644 arch/loongarch/boot/dts/loongson-2k0300.dtsi
> 
> diff --git a/arch/loongarch/boot/dts/Makefile b/arch/loongarch/boot/dts/Makefile
> index 15d5e14fe418..4a096608cfad 100644
> --- a/arch/loongarch/boot/dts/Makefile
> +++ b/arch/loongarch/boot/dts/Makefile
> @@ -1,3 +1,3 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  
> -dtb-y = loongson-2k0500-ref.dtb loongson-2k1000-ref.dtb loongson-2k2000-ref.dtb
> +dtb-y = loongson-2k0300-ref.dtb loongson-2k0500-ref.dtb loongson-2k1000-ref.dtb loongson-2k2000-ref.dtb
> diff --git a/arch/loongarch/boot/dts/loongson-2k0300-ref.dts b/arch/loongarch/boot/dts/loongson-2k0300-ref.dts
> new file mode 100644
> index 000000000000..f85d2caa94b0
> --- /dev/null
> +++ b/arch/loongarch/boot/dts/loongson-2k0300-ref.dts
> @@ -0,0 +1,34 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Copyright (C) 2023 Loongson Technology Corporation Limited
> + */
> +
> +/dts-v1/;
> +
> +#include "loongson-2k0300.dtsi"
> +
> +/ {
> +	compatible = "loongson,ls2k0300-ref", "loongson,ls2k0300";
> +	model = "Loongson-2K0300 Reference Board";
> +
> +	aliases {
> +		serial0 = &uart0;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	memory@200000 {
> +		device_type = "memory";
> +		reg = <0x0 0x00200000 0x0 0x0ee00000>,
> +			  <0x0 0x90000000 0x0 0x10000000>;
> +	};
> +
> +};
> +
> +

This code has many white space issues. Please clean it up everywhere.

> +&uart0 {
> +	status = "okay";
> +};
> +

And here



Best regards,
Krzysztof