* [PATCH 0/2] dt-bindings: Add Xuantie Semiconductor Co., Ltd. @ 2026-08-15 1:57 Guo Ren 2026-08-15 1:57 ` [PATCH 1/2] dt-bindings: vendor-prefixes: Add Xuantie Semiconductor Guo Ren 2026-08-15 1:57 ` [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles Guo Ren 0 siblings, 2 replies; 9+ messages in thread From: Guo Ren @ 2026-08-15 1:57 UTC (permalink / raw) To: robh Cc: devicetree, guoren, linux-kernel, linux-riscv, robh+dt, conor.dooley, krzk 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 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/2] dt-bindings: vendor-prefixes: Add Xuantie Semiconductor 2026-08-15 1:57 [PATCH 0/2] dt-bindings: Add Xuantie Semiconductor Co., Ltd Guo Ren @ 2026-08-15 1:57 ` Guo Ren 2026-08-17 6:08 ` Krzysztof Kozlowski 2026-08-15 1:57 ` [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles Guo Ren 1 sibling, 1 reply; 9+ messages in thread From: Guo Ren @ 2026-08-15 1:57 UTC (permalink / raw) To: robh Cc: devicetree, guoren, linux-kernel, linux-riscv, robh+dt, conor.dooley, krzk 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 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] dt-bindings: vendor-prefixes: Add Xuantie Semiconductor 2026-08-15 1:57 ` [PATCH 1/2] dt-bindings: vendor-prefixes: Add Xuantie Semiconductor Guo Ren @ 2026-08-17 6:08 ` Krzysztof Kozlowski 2026-08-17 9:42 ` Guo Ren 0 siblings, 1 reply; 9+ messages in thread From: Krzysztof Kozlowski @ 2026-08-17 6:08 UTC (permalink / raw) To: Guo Ren, robh Cc: devicetree, linux-kernel, linux-riscv, robh+dt, conor.dooley 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 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] dt-bindings: vendor-prefixes: Add Xuantie Semiconductor 2026-08-17 6:08 ` Krzysztof Kozlowski @ 2026-08-17 9:42 ` Guo Ren 0 siblings, 0 replies; 9+ messages in thread From: Guo Ren @ 2026-08-17 9:42 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: robh, devicetree, linux-kernel, linux-riscv, robh+dt, conor.dooley 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 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles 2026-08-15 1:57 [PATCH 0/2] dt-bindings: Add Xuantie Semiconductor Co., Ltd Guo Ren 2026-08-15 1:57 ` [PATCH 1/2] dt-bindings: vendor-prefixes: Add Xuantie Semiconductor Guo Ren @ 2026-08-15 1:57 ` Guo Ren 2026-08-15 2:02 ` sashiko-bot ` (2 more replies) 1 sibling, 3 replies; 9+ messages in thread From: Guo Ren @ 2026-08-15 1:57 UTC (permalink / raw) To: robh Cc: devicetree, guoren, linux-kernel, linux-riscv, robh+dt, conor.dooley, krzk 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 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles 2026-08-15 1:57 ` [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles Guo Ren @ 2026-08-15 2:02 ` sashiko-bot 2026-08-15 19:44 ` Shengyu Qu 2026-08-17 6:10 ` Krzysztof Kozlowski 2 siblings, 0 replies; 9+ messages in thread From: sashiko-bot @ 2026-08-15 2:02 UTC (permalink / raw) To: Guo Ren; +Cc: robh, conor+dt, devicetree 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 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles 2026-08-15 1:57 ` [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles Guo Ren 2026-08-15 2:02 ` sashiko-bot @ 2026-08-15 19:44 ` Shengyu Qu 2026-08-17 1:40 ` Guo Ren 2026-08-17 6:10 ` Krzysztof Kozlowski 2 siblings, 1 reply; 9+ messages in thread From: Shengyu Qu @ 2026-08-15 19:44 UTC (permalink / raw) To: Guo Ren, robh Cc: devicetree, linux-kernel, linux-riscv, robh+dt, conor.dooley, krzk 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: ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles 2026-08-15 19:44 ` Shengyu Qu @ 2026-08-17 1:40 ` Guo Ren 0 siblings, 0 replies; 9+ messages in thread From: Guo Ren @ 2026-08-17 1:40 UTC (permalink / raw) To: Shengyu Qu Cc: robh, devicetree, linux-kernel, linux-riscv, robh+dt, conor.dooley, krzk 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 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles 2026-08-15 1:57 ` [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles Guo Ren 2026-08-15 2:02 ` sashiko-bot 2026-08-15 19:44 ` Shengyu Qu @ 2026-08-17 6:10 ` Krzysztof Kozlowski 2 siblings, 0 replies; 9+ messages in thread From: Krzysztof Kozlowski @ 2026-08-17 6:10 UTC (permalink / raw) To: Guo Ren, robh Cc: devicetree, linux-kernel, linux-riscv, robh+dt, conor.dooley 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 ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2026-08-17 9:42 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-08-15 1:57 [PATCH 0/2] dt-bindings: Add Xuantie Semiconductor Co., Ltd Guo Ren 2026-08-15 1:57 ` [PATCH 1/2] dt-bindings: vendor-prefixes: Add Xuantie Semiconductor Guo Ren 2026-08-17 6:08 ` Krzysztof Kozlowski 2026-08-17 9:42 ` Guo Ren 2026-08-15 1:57 ` [PATCH 2/2] dt-bindings: riscv: Add Xuantie C906/C908/C910/C920 compatibles Guo Ren 2026-08-15 2:02 ` sashiko-bot 2026-08-15 19:44 ` Shengyu Qu 2026-08-17 1:40 ` Guo Ren 2026-08-17 6:10 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox