[PATCH 0/2] dt-bindings: Add Xuantie Semiconductor Co., Ltd.

[PATCH 0/2] dt-bindings: Add Xuantie Semiconductor Co., Ltd.

[Guo Ren]

From: "GUO Ren (XuanTie)" <guoren@kernel.org>

The RISC-V CPU bindings currently describe the XuanTie C906, C908,
C910 and C920 processors using the historical thead vendor prefix.

Add xuantie as the vendor prefix for Shanghai Xuantie Semiconductor
Technology Co., Ltd., and add corresponding xuantie,* compatible
strings for these processors.

The existing thead,* compatible strings are retained to preserve the
Device Tree ABI and compatibility with existing device trees.

The vendor information is available from the Xuantie declaration
page [1].

[1] https://www.xrvm.com/declaration

GUO Ren (XuanTie) (2):
  dt-bindings: vendor-prefixes: Add Xuantie Semiconductor
  dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles

 Documentation/devicetree/bindings/riscv/cpus.yaml      | 4 ++++
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 2 files changed, 6 insertions(+)

-- 
2.43.0

[PATCH 1/2] dt-bindings: vendor-prefixes: Add Xuantie Semiconductor

[Guo Ren]

From: "GUO Ren (XuanTie)" <guoren@kernel.org>

Add the xuantie vendor prefix for Shanghai Xuantie Semiconductor
Technology Co., Ltd., which provides the XuanTie processor IP.

The vendor information is available from the Xuantie declaration
pagei [1].

[1]: https://www.xrvm.com/declaration

Cc: Rob Herring <robh+dt@kernel.org>
Cc: devicetree <devicetree@vger.kernel.org>
Signed-off-by: GUO Ren (XuanTie) <guoren@kernel.org>
---
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
index 396044f368e7..89391cb467d5 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1905,6 +1905,8 @@ patternProperties:
     description: Xilinx
   "^xnano,.*":
     description: Xnano
+  "^xuantie,.*":
+    description: Shanghai Xuantie Semiconductor Technology Co., Ltd.
   "^xunlong,.*":
     description: Shenzhen Xunlong Software CO.,Limited
   "^xylon,.*":
-- 
2.43.0

[PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles

[Guo Ren]

From: "GUO Ren (XuanTie)" <guoren@kernel.org>

The RISC-V CPU bindings currently use the historical thead vendor
prefix for the XuanTie C906, C908, C910 and C920 processors.

Add equivalent compatibles using the newly defined xuantie vendor
prefix. Keep the existing thead compatibles to preserve compatibility
with existing device trees.

Cc: Conor Dooley <conor.dooley@microchip.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: devicetree <devicetree@vger.kernel.org>
Signed-off-by: GUO Ren (XuanTie) <guoren@kernel.org>
---
 Documentation/devicetree/bindings/riscv/cpus.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/riscv/cpus.yaml b/Documentation/devicetree/bindings/riscv/cpus.yaml
index 5feeb2203050..5aba1e8ed195 100644
--- a/Documentation/devicetree/bindings/riscv/cpus.yaml
+++ b/Documentation/devicetree/bindings/riscv/cpus.yaml
@@ -67,6 +67,10 @@ properties:
               - thead,c908
               - thead,c910
               - thead,c920
+              - xuantie,c906
+              - xuantie,c908
+              - xuantie,c910
+              - xuantie,c920
           - const: riscv
       - items:
           - enum:
-- 
2.43.0

Re: [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The newly added `xuantie` equivalent compatible strings were omitted from the `allOf` conditional block that restricts the `thead,vlenb` property, causing valid device trees to fail schema validation.
--

--- Patch [2]: [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles ---
commit 8fd8dd2156e30474207bea785ee6877565dc1eff
Author: GUO Ren (XuanTie) <guoren@kernel.org>

dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles

The RISC-V CPU bindings currently use the historical thead vendor
prefix for the XuanTie C906, C908, C910 and C920 processors. This patch
adds equivalent compatibles using the newly defined xuantie vendor
prefix. It keeps the existing thead compatibles to preserve compatibility
with existing device trees.

> diff --git a/Documentation/devicetree/bindings/riscv/cpus.yaml b/Documentation/devicetree/bindings/riscv/cpus.yaml
> index 5feeb2203050a..5aba1e8ed1952 100644
> --- a/Documentation/devicetree/bindings/riscv/cpus.yaml
> +++ b/Documentation/devicetree/bindings/riscv/cpus.yaml
> @@ -67,6 +67,10 @@ properties:
>                - thead,c908
>                - thead,c910
>                - thead,c920
> +              - xuantie,c906
> +              - xuantie,c908
> +              - xuantie,c910
> +              - xuantie,c920

[Severity: Medium]
Since these new compatible strings are equivalents to the existing "thead,*"
compatibles, shouldn't they also be added to the allOf conditional block that
governs the "thead,vlenb" property?

Earlier in this file, the schema restricts the property like this:

allOf:
  - $ref: /schemas/cpu.yaml#
  - $ref: extensions.yaml
  - if:
      not:
        properties:
          compatible:
            contains:
              enum:
                - thead,c906
                - thead,c910
                - thead,c920
    then:
      properties:
        thead,vlenb: false

If a device tree node is defined with compatible = "xuantie,c906" and
includes the "thead,vlenb" property, won't the allOf block evaluate true
and incorrectly enforce "thead,vlenb: false", causing schema validation to
fail?

>            - const: riscv
>        - items:
>            - enum:

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260815015722.1736831-1-guoren@kernel.org?part=2