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: <74e3c488-4457-4026-9597-806b98fd4e11@kernel.org>
Date: Tue, 17 Jun 2025 08:21:51 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Vivian Wang <wangruikang@...as.ac.cn>, Yixun Lan <dlan@...too.org>,
 Guodong Xu <guodong@...cstar.com>, Ze Huang <huangze@...t.edu.cn>,
 spacemit@...ts.linux.dev
Cc: Vivian Wang <uwu@...m.page>, 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>, devicetree@...r.kernel.org,
 linux-riscv@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH RFC] riscv: dts: spacemit: Add DMA translation buses for
 K1

On 17/06/2025 07:21, Vivian Wang wrote:
> The SpacemiT K1 has various static translations of DMA accesses. Add
> these as simple-bus nodes. Devices actually using these translation will
> be added in later patches.
> 
> The bus names are assigned according to consensus with SpacemiT [1].


Read the feedback there:

"So, as you are submitting the first node(s) under network_bus: bus@5, you
should have this added into your patchset, instead of sending out with
none."

Plus simple bus within MMIO node needs unit address. IOW, don't mix MMIO
with non-MMIO. I also suspect this does not pass checks, so the tools
can do our review...

Best regards,
Krzysztof

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ