[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

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

[Shengyu Qu]

Can't we just replace all existing dts' thead to xuantie? So we can 
remove thead here.

> 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:

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

[Guo Ren]

On Sun, Aug 16, 2026 at 3:45 AM Shengyu Qu <wiagn233@outlook.com> wrote:
>
> Can't we just replace all existing dts' thead to xuantie? So we can
> remove thead here.

Good advice, I'll try that in the next version. Thx.

>
> > 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:
>


-- 
Best Regards
 Guo Ren

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

[Krzysztof Kozlowski]

On 15/08/2026 03:57, Guo Ren wrote:
> 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>

Please drop the autogenerated scripts/get_maintainer.pl CC-entries from
commit msg. There is no single need to store automated output of
get_maintainers.pl in the git log. It can be easily re-created at any
given time, thus its presence in the git history is redundant and
obfuscates the log.

If you need it for your own patch management purposes, keep it under the
--- separator.


> Signed-off-by: GUO Ren (XuanTie) <guoren@kernel.org>
> ---
>  Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++

Anyway odd cc-list.

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC (and consider --no-git-fallback argument, so you will
not CC people just because they made one commit years ago). It might
happen, that command when run on an older kernel, gives you outdated
entries. Therefore please be sure you base your patches on recent Linux
kernel.

Tools like b4 or scripts/get_maintainer.pl provide you proper list of
people, so fix your workflow. Tools might also fail if you work on some
ancient tree (don't, instead use mainline) or work on fork of kernel
(don't, instead use mainline). Just use b4 and everything should be
fine, although remember about `b4 prep --auto-to-cc` if you added new
patches to the patchset.


Best regards,
Krzysztof

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

[Krzysztof Kozlowski]

On 15/08/2026 03:57, Guo Ren wrote:
> 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.

No, you already received feedback - you were on Cc, you replied, you got
answers.

https://lore.kernel.org/all/20251121-able-terrestrial-spaniel-90c3cf@kuoka/

NAK

Best regards,
Krzysztof

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

[Guo Ren]

On Mon, Aug 17, 2026 at 2:08 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On 15/08/2026 03:57, Guo Ren wrote:
> > 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>
>
> Please drop the autogenerated scripts/get_maintainer.pl CC-entries from
> commit msg. There is no single need to store automated output of
> get_maintainers.pl in the git log. It can be easily re-created at any
> given time, thus its presence in the git history is redundant and
> obfuscates the log.

Thanks for pointing that out; I would drop the autogenerated
scripts/get_maintainer.pl CC entries.

>
> If you need it for your own patch management purposes, keep it under the
> --- separator.
>
>
> > Signed-off-by: GUO Ren (XuanTie) <guoren@kernel.org>
> > ---
> >  Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
>
> Anyway odd cc-list.
>
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC (and consider --no-git-fallback argument, so you will
> not CC people just because they made one commit years ago). It might
> happen, that command when run on an older kernel, gives you outdated
> entries. Therefore please be sure you base your patches on recent Linux
> kernel.
>
> Tools like b4 or scripts/get_maintainer.pl provide you proper list of
> people, so fix your workflow. Tools might also fail if you work on some
> ancient tree (don't, instead use mainline) or work on fork of kernel
> (don't, instead use mainline). Just use b4 and everything should be
> fine, although remember about `b4 prep --auto-to-cc` if you added new
> patches to the patchset.
Okay, I'll try that.

--
Best Regards
 Guo Ren