[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260114-evergreen-gluten-b1c8558e7684@spud>
Date: Wed, 14 Jan 2026 20:42:41 +0000
From: Conor Dooley <conor@...nel.org>
To: Guodong Xu <guodong@...cstar.com>
Cc: Yixun Lan <dlan@...too.org>, Jonathan Corbet <corbet@....net>,
Paul Walmsley <pjw@...nel.org>, Palmer Dabbelt <palmer@...belt.com>,
Albert Ou <aou@...s.berkeley.edu>, Alexandre Ghiti <alex@...ti.fr>,
Conor Dooley <conor.dooley@...rochip.com>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Chen Wang <unicorn_wang@...look.com>,
Inochi Amaoto <inochiama@...il.com>,
Junhui Liu <junhui.liu@...moral.tech>, linux-doc@...r.kernel.org,
linux-riscv@...ts.infradead.org, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org, sophgo@...ts.linux.dev,
spacemit@...ts.linux.dev
Subject: Re: [PATCH 2/2] riscv: dts: Add "b" ISA extension to existing
devicetrees
On Wed, Jan 14, 2026 at 10:33:34AM +0800, Guodong Xu wrote:
> Hi, Conor, Yixun
>
> On Tue, Jan 13, 2026 at 3:53 PM Yixun Lan <dlan@...too.org> wrote:
> >
> > Hi Guodong,
> >
> >
> > On 15:45 Tue 13 Jan , Guodong Xu wrote:
> > > "b" is ratified (Apr/2024) much later than its components zba/zbb/zbs
> > > (Jun/2021). With "b" added into riscv/extensions.yaml, a dependency
> > > checking rule is now enforced, which requires that when zba, zbb, and zbs
> > > are all specified, "b" must be added as well. Failing to do this will
> > > cause dtbs_check schema check warnings.
> > >
> > > According to uabi.rst, as a single-letter extension, "b" should be added
> > > after "c" in canonical order.
> > >
> > > Update existing devicetree files to conform to this rule. Line balancing
> > > is performed where needed to improve readability.
> > >
> > > Signed-off-by: Guodong Xu <guodong@...cstar.com>
> > > ---
> > > arch/riscv/boot/dts/anlogic/dr1v90.dtsi | 5 +-
> > > arch/riscv/boot/dts/sophgo/sg2044-cpus.dtsi | 256 ++++++++++++++--------------
> > > arch/riscv/boot/dts/spacemit/k1.dtsi | 32 ++--
> > > 3 files changed, 147 insertions(+), 146 deletions(-)
> > please have separated patch for different SoCs, so they can go via
>
> Understand your concern.
>
> > their own SoC tree.. thanks
>
> Conor, is it possible to take this as one patch instead?
>
> It fixes the same dtbs_check warnings from riscv/extensions.yaml across three
> SoCs. Keeping them together maintains readability and clear tracking, IMHO.
>
> Happy to split if needed, but wanted to check first.
I'd rather you split it, sorry.
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists