linux-kernel - Re: [PATCH 1/3] riscv: dts: spacemit: Add DMA translation buses for K1

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAH1PCMYbP1mV_bO8f94qTbjqwairV3LQ1r1=B1Rmuphs-6VjxA@mail.gmail.com>
Date: Thu, 26 Jun 2025 15:54:43 +0800
From: Guodong Xu <guodong@...cstar.com>
To: Vivian Wang <wangruikang@...as.ac.cn>
Cc: Yixun Lan <dlan@...too.org>, Ze Huang <huangze@...t.edu.cn>, 
	Alex Elder <elder@...cstar.com>, spacemit@...ts.linux.dev, 
	Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Paul Walmsley <paul.walmsley@...ive.com>, Palmer Dabbelt <palmer@...belt.com>, 
	Albert Ou <aou@...s.berkeley.edu>, Alexandre Ghiti <alex@...ti.fr>, Vivian Wang <uwu@...m.page>, 
	devicetree@...r.kernel.org, linux-riscv@...ts.infradead.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] riscv: dts: spacemit: Add DMA translation buses for K1

On Mon, Jun 23, 2025 at 6:04 PM Vivian Wang <wangruikang@...as.ac.cn> wrote:
>
> The SpacemiT K1 has various static translations of DMA accesses. Add
> these as simple-bus nodes with dma-ranges. Devices actually using these
> translations will be added to or moved inside the bus nodes in later
> patches.
>
> The bus names are assigned according to consensus with SpacemiT [1].
>
> Link: https://lore.kernel.org/r/CAH1PCMaC+imcMZCFYtRdmH6ge=dPgnANn_GqVfsGRS=+YhyJCw@mail.gmail.com [1]
> Signed-off-by: Vivian Wang <wangruikang@...as.ac.cn>
> ---
>  arch/riscv/boot/dts/spacemit/k1.dtsi | 53 ++++++++++++++++++++++++++++++++++++
>  1 file changed, 53 insertions(+)
>
> diff --git a/arch/riscv/boot/dts/spacemit/k1.dtsi b/arch/riscv/boot/dts/spacemit/k1.dtsi
> index 14097f1f6f447bd33ff3aaa07382d27ca8e59a48..99c76997b367f733c9dda2c30dd85817294ef9b7 100644
> --- a/arch/riscv/boot/dts/spacemit/k1.dtsi
> +++ b/arch/riscv/boot/dts/spacemit/k1.dtsi
> @@ -571,5 +571,58 @@ sec_uart1: serial@...12000 {
>                         reg-io-width = <4>;
>                         status = "reserved"; /* for TEE usage */
>                 };
> +
> +               camera-bus {
> +                       compatible = "simple-bus";
> +                       ranges;
> +                       #address-cells = <2>;
> +                       #size-cells = <2>;
> +                       dma-ranges = <0x0 0x00000000 0x0 0x00000000 0x0 0x80000000>,
> +                                    <0x0 0x80000000 0x1 0x00000000 0x1 0x80000000>;
> +               };
> +
> +               dma-bus {
> +                       compatible = "simple-bus";
> +                       ranges;
> +                       #address-cells = <2>;
> +                       #size-cells = <2>;
> +                       dma-ranges = <0x0 0x00000000 0x0 0x00000000 0x0 0x80000000>,
> +                                    <0x1 0x00000000 0x1 0x80000000 0x3 0x00000000>;
> +               };
> +
> +               multimedia-bus {
> +                       compatible = "simple-bus";
> +                       ranges;
> +                       #address-cells = <2>;
> +                       #size-cells = <2>;
> +                       dma-ranges = <0x0 0x00000000 0x0 0x00000000 0x0 0x80000000>,
> +                                    <0x0 0x80000000 0x1 0x00000000 0x3 0x80000000>;
> +               };
> +
> +               network-bus {
> +                       compatible = "simple-bus";
> +                       ranges;
> +                       #address-cells = <2>;
> +                       #size-cells = <2>;
> +                       dma-ranges = <0x0 0x00000000 0x0 0x00000000 0x0 0x80000000>,
> +                                    <0x0 0x80000000 0x1 0x00000000 0x0 0x80000000>;
> +               };
> +
> +               pcie-bus {
> +                       compatible = "simple-bus";
> +                       ranges;
> +                       #address-cells = <2>;
> +                       #size-cells = <2>;
> +                       dma-ranges = <0x0 0x00000000 0x0 0x00000000 0x0 0x80000000>,
> +                                    <0x0 0xb8000000 0x1 0x38000000 0x3 0x48000000>;
> +               };
> +
> +               storage-bus {
> +                       compatible = "simple-bus";
> +                       ranges;
> +                       #address-cells = <2>;
> +                       #size-cells = <2>;
> +                       dma-ranges = <0x0 0x00000000 0x0 0x00000000 0x0 0x80000000>;
> +               };

This looks good to me.

Reviewed-by: Guodong Xu <guodong@...cstar.com>

>         };
>  };
>
> --
> 2.49.0
>