mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] riscv: dts: canaan: k230: Add "b" ISA extension
@ 2026-01-18 14:25 Junhui Liu
  2026-01-19 18:13 ` Conor Dooley
  0 siblings, 1 reply; 2+ messages in thread
From: Junhui Liu @ 2026-01-18 14:25 UTC (permalink / raw)
  To: Conor Dooley, Rob Herring, Krzysztof Kozlowski, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Alexandre Ghiti
  Cc: linux-riscv, devicetree, linux-kernel, Junhui Liu

"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 k230.dtsi to conform to this rule. Also synchronize "riscv,isa"
with "riscv,isa-extensions".

Signed-off-by: Junhui Liu <junhui.liu@pigmoral.tech>
---
Based on Conor's k230-basic branch and rebased to riscv-dt-for-next.

The commit message and implementation are adapted from Guodong's patch
for adding the "b" ISA extension to multiple dtsi files.
---
 arch/riscv/boot/dts/canaan/k230.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/riscv/boot/dts/canaan/k230.dtsi b/arch/riscv/boot/dts/canaan/k230.dtsi
index 5a1bf8a15abe..041c9947a8fa 100644
--- a/arch/riscv/boot/dts/canaan/k230.dtsi
+++ b/arch/riscv/boot/dts/canaan/k230.dtsi
@@ -21,9 +21,9 @@ cpu@0 {
 			compatible = "thead,c908", "riscv";
 			device_type = "cpu";
 			reg = <0>;
-			riscv,isa = "rv64imafdcv_zicbom_zicbop_zicboz_zfh_zba_zbb_zbc_zbs_zvfh_svpbmt";
+			riscv,isa = "rv64imafdcbv_zicbom_zicbop_zicboz_zicntr_zicsr_zifencei_zihpm_zfh_zba_zbb_zbc_zbs_zvfh_svpbmt";
 			riscv,isa-base = "rv64i";
-			riscv,isa-extensions = "i", "m", "a", "f", "d", "c", "v", "zicbom",
+			riscv,isa-extensions = "i", "m", "a", "f", "d", "c", "b", "v", "zicbom",
 					       "zicbop", "zicboz", "zicntr", "zicsr", "zifencei",
 					       "zihpm", "zfh", "zba", "zbb", "zbc", "zbs", "zvfh",
 					       "svpbmt";

---
base-commit: 99a8c0e33dbf0117710b7f0c5cd1cb39638235ab
change-id: 20260118-k230-ext-b-0d7e7c77c38f

Best regards,
-- 
Junhui Liu <junhui.liu@pigmoral.tech>


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] riscv: dts: canaan: k230: Add "b" ISA extension
  2026-01-18 14:25 [PATCH] riscv: dts: canaan: k230: Add "b" ISA extension Junhui Liu
@ 2026-01-19 18:13 ` Conor Dooley
  0 siblings, 0 replies; 2+ messages in thread
From: Conor Dooley @ 2026-01-19 18:13 UTC (permalink / raw)
  To: Junhui Liu
  Cc: Rob Herring, Krzysztof Kozlowski, Paul Walmsley, Palmer Dabbelt,
	Albert Ou, Alexandre Ghiti, linux-riscv, devicetree,
	linux-kernel

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

On Sun, Jan 18, 2026 at 10:25:07PM +0800, Junhui Liu 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 k230.dtsi to conform to this rule. Also synchronize "riscv,isa"
> with "riscv,isa-extensions".
> 
> Signed-off-by: Junhui Liu <junhui.liu@pigmoral.tech>
> ---
> Based on Conor's k230-basic branch and rebased to riscv-dt-for-next.
> 
> The commit message and implementation are adapted from Guodong's patch
> for adding the "b" ISA extension to multiple dtsi files.

Applied to my k230-basic branch.

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-01-19 18:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-01-18 14:25 [PATCH] riscv: dts: canaan: k230: Add "b" ISA extension Junhui Liu
2026-01-19 18:13 ` Conor Dooley

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®