public inbox for linux-doc@vger.kernel.org
 help / color / mirror / Atom feed
From: Conor Dooley <conor@kernel.org>
To: Guodong Xu <guodong@riscstar.com>
Cc: Jonathan Corbet <corbet@lwn.net>, Paul Walmsley <pjw@kernel.org>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Albert Ou <aou@eecs.berkeley.edu>,
	Alexandre Ghiti <alex@ghiti.fr>,
	Conor Dooley <conor.dooley@microchip.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Chen Wang <unicorn_wang@outlook.com>,
	Inochi Amaoto <inochiama@gmail.com>, Yixun Lan <dlan@gentoo.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Junhui Liu <junhui.liu@pigmoral.tech>,
	linux-doc@vger.kernel.org, linux-riscv@lists.infradead.org,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	sophgo@lists.linux.dev, spacemit@lists.linux.dev
Subject: Re: [PATCH v2 0/4] riscv: dts: Add "b" ISA extension to existing devicetrees
Date: Thu, 15 Jan 2026 19:13:17 +0000	[thread overview]
Message-ID: <20260115-phonics-reversal-80541ca1f2df@spud> (raw)
In-Reply-To: <20260115-adding-b-dtsi-v2-0-254dd61cf947@riscstar.com>

[-- Attachment #1: Type: text/plain, Size: 3163 bytes --]

On Thu, Jan 15, 2026 at 07:18:56AM +0800, Guodong Xu wrote:
> The RISC-V "b" (Bit-manipulation) extension was ratified in April 2024,
> much later than its component extensions zba/zbb/zbs (June 2021). Recent
> updates to the device tree bindings [2] enforce that when all three
> component extensions are present, "b" must also be specified. Related
> discussion can also be found in [1].
> 
> Patch 1 clarifies the ISA spec version for canonical ordering in uabi.rst.
> It is a trivial update, but can help readers reference the correct
> document version. Acked-by Paul Walmsley in v1.
> 
> Patch 2, 3 and 4 adds "b" after "c" in 3 different device tree files
> respectivly, anlogic, sophgo and spacemit, fixing the related dtbs_check
> warnings.
> 
> This patchset is based on top of linux-next, tag: next-20260109, and
> depends on [2].
> 
> Link: https://lore.kernel.org/all/20251230-imprison-sleet-6b5a1e26d34b@spud/ [1]
> Link: https://lore.kernel.org/all/20260110-k3-basic-dt-v4-0-d492f3a30ffa@riscstar.com/ [2]
> 
> Changes in v2:
> - Patch 1: Add Acked-by from Paul Walmsley.
> - Patch 2/3/4: These are splits from the v1 Patch 2. Split into three
>     different patches for each SoC.
> - Link to v1: https://lore.kernel.org/r/20260113-adding-b-dtsi-v1-0-22d6e55d19df@riscstar.com
> 
> Signed-off-by: Guodong Xu <guodong@riscstar.com>
> ---
> Guodong Xu (4):
>       Documentation: riscv: uabi: Clarify ISA spec version for canonical order
>       riscv: dts: anlogic: dr1v90: Add "b" ISA extension
>       riscv: dts: sophgo: sg2044: Add "b" ISA extension
>       riscv: dts: spacemit: k1: Add "b" ISA extension
> 
>  Documentation/arch/riscv/uabi.rst           |   4 +-
>  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 ++--
>  4 files changed, 150 insertions(+), 147 deletions(-)
> ---
> base-commit: 31d167f54de93f14fa8e4bc6cbc4adaf7019fd94
> change-id: 20260113-adding-b-dtsi-148714533f07
> prerequisite-message-id: <20260110-k3-basic-dt-v4-0-d492f3a30ffa@riscstar.com>
> prerequisite-patch-id: 0c859b4d131b3360875c795c6148c6176b55fb91
> prerequisite-patch-id: 2ed98dc1ab0f5ed923cc252415c345dc8caf6f17
> prerequisite-patch-id: 1be1a031763fac029076a768f012af31e455be66
> prerequisite-patch-id: 21bb8387c946e050910440e7a7622305d46d946d
> prerequisite-patch-id: f3bdc2c74b230663710086bd770a755d56cb8b9c
> prerequisite-patch-id: 1f162c02f8bdb5bbc8ce52ead4fcb76258f5c2b9
> prerequisite-patch-id: 76e1ff26c2f1fe4019cfa574942b568000e6ca1f
> prerequisite-patch-id: 77ddc9e5dc85495adc803cdc605bdda2ddc7fa47
> prerequisite-patch-id: a75c798383b46a14d40436357c769c3671184768
> prerequisite-patch-id: 781fc10dcae2c38c84c25bee887ef7474786dd36
> prerequisite-patch-id: 5be5d3e62aa73024bf9e1de6aad155be6d618f40

FWIW, this kind of prereq-patch-id stuff isn't really helpful, because I
don't think there's actually any dependencies on that work. It actually
trips up some parts of b4 that think they're needed.

I applied the Anlogic patch.

Thanks,
Conor.

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

  parent reply	other threads:[~2026-01-15 19:13 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-14 23:18 [PATCH v2 0/4] riscv: dts: Add "b" ISA extension to existing devicetrees Guodong Xu
2026-01-14 23:18 ` [PATCH v2 1/4] Documentation: riscv: uabi: Clarify ISA spec version for canonical order Guodong Xu
2026-01-15  0:10   ` Paul Walmsley
2026-01-14 23:18 ` [PATCH v2 2/4] riscv: dts: anlogic: dr1v90: Add "b" ISA extension Guodong Xu
2026-01-15  2:25   ` Junhui Liu
2026-01-14 23:18 ` [PATCH v2 3/4] riscv: dts: sophgo: sg2044: " Guodong Xu
2026-01-15  0:28   ` Inochi Amaoto
2026-01-14 23:19 ` [PATCH v2 4/4] riscv: dts: spacemit: k1: " Guodong Xu
2026-01-15  0:39   ` Yixun Lan
2026-01-15 11:24 ` (subset) [PATCH v2 0/4] riscv: dts: Add "b" ISA extension to existing devicetrees Yixun Lan
2026-01-15 19:13 ` Conor Dooley [this message]
2026-01-15 23:04   ` Guodong Xu
2026-01-20  1:16 ` (subset) " Inochi Amaoto
2026-01-20  2:48   ` Inochi Amaoto

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260115-phonics-reversal-80541ca1f2df@spud \
    --to=conor@kernel.org \
    --cc=alex@ghiti.fr \
    --cc=aou@eecs.berkeley.edu \
    --cc=conor+dt@kernel.org \
    --cc=conor.dooley@microchip.com \
    --cc=corbet@lwn.net \
    --cc=devicetree@vger.kernel.org \
    --cc=dlan@gentoo.org \
    --cc=guodong@riscstar.com \
    --cc=inochiama@gmail.com \
    --cc=junhui.liu@pigmoral.tech \
    --cc=krzk+dt@kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=palmer@dabbelt.com \
    --cc=pjw@kernel.org \
    --cc=robh@kernel.org \
    --cc=sophgo@lists.linux.dev \
    --cc=spacemit@lists.linux.dev \
    --cc=unicorn_wang@outlook.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox