[PATCH v2 1/4] dt-bindings: vendor-prefixes: Add prefix for XuanTie

cp0613@linux.alibaba.com posted 4 patches 1 week, 3 days ago
[PATCH v2 1/4] dt-bindings: vendor-prefixes: Add prefix for XuanTie
Posted by cp0613@linux.alibaba.com 1 week, 3 days ago
From: Chen Pei <cp0613@linux.alibaba.com>

Add new vendor strings to dt bindings for RISC-V vendor XuanTie.

Signed-off-by: Chen Pei <cp0613@linux.alibaba.com>
---
 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 f1d1882009ba..5d28b2da94cd 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1829,6 +1829,8 @@ patternProperties:
     description: Xilinx
   "^xnano,.*":
     description: Xnano
+  "^xuantie,.*":
+    description: C-SKY Microsystems Co., Ltd. (XuanTie)
   "^xunlong,.*":
     description: Shenzhen Xunlong Software CO.,Limited
   "^xylon,.*":
-- 
2.50.1
Re: [PATCH v2 1/4] dt-bindings: vendor-prefixes: Add prefix for XuanTie
Posted by Krzysztof Kozlowski 1 week, 3 days ago
On Fri, Nov 21, 2025 at 02:34:36PM +0800, cp0613@linux.alibaba.com wrote:
> From: Chen Pei <cp0613@linux.alibaba.com>
> 
> Add new vendor strings to dt bindings for RISC-V vendor XuanTie.
> 
> Signed-off-by: Chen Pei <cp0613@linux.alibaba.com>
> ---
>  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 f1d1882009ba..5d28b2da94cd 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -1829,6 +1829,8 @@ patternProperties:
>      description: Xilinx
>    "^xnano,.*":
>      description: Xnano
> +  "^xuantie,.*":
> +    description: C-SKY Microsystems Co., Ltd. (XuanTie)

We do not create new prefixes for existing entries. You have already
csky, alibaba and thead and probably many more. Choose some appropriate.

Your commit msg should also explain anything unusual here, like above.

Best regards,
Krzysztof
Re: [PATCH v2 1/4] dt-bindings: vendor-prefixes: Add prefix for XuanTie
Posted by Conor Dooley 1 week, 3 days ago
On Fri, Nov 21, 2025 at 10:01:07AM +0100, Krzysztof Kozlowski wrote:
> On Fri, Nov 21, 2025 at 02:34:36PM +0800, cp0613@linux.alibaba.com wrote:
> > From: Chen Pei <cp0613@linux.alibaba.com>
> > 
> > Add new vendor strings to dt bindings for RISC-V vendor XuanTie.
> > 
> > Signed-off-by: Chen Pei <cp0613@linux.alibaba.com>
> > ---
> >  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 f1d1882009ba..5d28b2da94cd 100644
> > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > @@ -1829,6 +1829,8 @@ patternProperties:
> >      description: Xilinx
> >    "^xnano,.*":
> >      description: Xnano
> > +  "^xuantie,.*":
> > +    description: C-SKY Microsystems Co., Ltd. (XuanTie)
> 
> We do not create new prefixes for existing entries. You have already
> csky, alibaba and thead and probably many more. Choose some appropriate.

When the vendor literally says "C-SKY", it's hard to buy why csky
shouldn't be used here. Xuantie is described as being a brand, where at
least t-head was a company. This seems akin to having
"snapdragon,msm8660" instead of "qcom,msm8660", for example.

> 
> Your commit msg should also explain anything unusual here, like above.
> 
> Best regards,
> Krzysztof
> 
Re: [PATCH v2 1/4] dt-bindings: vendor-prefixes: Add prefix for XuanTie
Posted by Guo Ren 1 week, 3 days ago
On Fri, Nov 21, 2025 at 8:31 PM Conor Dooley <conor@kernel.org> wrote:
>
> On Fri, Nov 21, 2025 at 10:01:07AM +0100, Krzysztof Kozlowski wrote:
> > On Fri, Nov 21, 2025 at 02:34:36PM +0800, cp0613@linux.alibaba.com wrote:
> > > From: Chen Pei <cp0613@linux.alibaba.com>
> > >
> > > Add new vendor strings to dt bindings for RISC-V vendor XuanTie.
> > >
> > > Signed-off-by: Chen Pei <cp0613@linux.alibaba.com>
> > > ---
> > >  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 f1d1882009ba..5d28b2da94cd 100644
> > > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > > @@ -1829,6 +1829,8 @@ patternProperties:
> > >      description: Xilinx
> > >    "^xnano,.*":
> > >      description: Xnano
> > > +  "^xuantie,.*":
> > > +    description: C-SKY Microsystems Co., Ltd. (XuanTie)
> >
> > We do not create new prefixes for existing entries. You have already
> > csky, alibaba and thead and probably many more. Choose some appropriate.
We could leave the one C-SKY vendor entry, how about:

--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -347,8 +347,8 @@ patternProperties:
     description: Creative Technology Ltd
   "^crystalfontz,.*":
     description: Crystalfontz America, Inc.
-  "^csky,.*":
-    description: Hangzhou C-SKY Microsystems Co., Ltd
+  "^xuantie,.*":
+    description: C-SKY Microsystems Co., Ltd. (XuanTie)
   "^csot,.*":
     description: Guangzhou China Star Optoelectronics Technology Co., Ltd
   "^csq,.*":

The arch/csky is the architecture name, which means "xuantie,csky".

The c810 is the CPU product name, which means "xuantie,c810" of the
"xuantie,csky" architecture.

The c930 is the CPU product name, which means "xuantie,c930".

>
> When the vendor literally says "C-SKY", it's hard to buy why csky
> shouldn't be used here. Xuantie is described as being a brand, where at
> least t-head was a company. This seems akin to having
> "snapdragon,msm8660" instead of "qcom,msm8660", for example.
Yes, XuanTie is not only a brand, but also the abbreviation of C-SKY
Microsystems Co., Ltd.

Just like:

  "^100ask,.*":
    description: Baiwen.com (100ask).
  "^linksys,.*":
    description: Belkin International, Inc. (Linksys)

>
> >
> > Your commit msg should also explain anything unusual here, like above.
> >
> > Best regards,
> > Krzysztof
> >

-- 
Best Regards
 Guo Ren
Re: [PATCH v2 1/4] dt-bindings: vendor-prefixes: Add prefix for XuanTie
Posted by Krzysztof Kozlowski 1 week, 3 days ago
On 21/11/2025 14:01, Guo Ren wrote:
> On Fri, Nov 21, 2025 at 8:31 PM Conor Dooley <conor@kernel.org> wrote:
>>
>> On Fri, Nov 21, 2025 at 10:01:07AM +0100, Krzysztof Kozlowski wrote:
>>> On Fri, Nov 21, 2025 at 02:34:36PM +0800, cp0613@linux.alibaba.com wrote:
>>>> From: Chen Pei <cp0613@linux.alibaba.com>
>>>>
>>>> Add new vendor strings to dt bindings for RISC-V vendor XuanTie.
>>>>
>>>> Signed-off-by: Chen Pei <cp0613@linux.alibaba.com>
>>>> ---
>>>>  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 f1d1882009ba..5d28b2da94cd 100644
>>>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>>> @@ -1829,6 +1829,8 @@ patternProperties:
>>>>      description: Xilinx
>>>>    "^xnano,.*":
>>>>      description: Xnano
>>>> +  "^xuantie,.*":
>>>> +    description: C-SKY Microsystems Co., Ltd. (XuanTie)
>>>
>>> We do not create new prefixes for existing entries. You have already
>>> csky, alibaba and thead and probably many more. Choose some appropriate.
> We could leave the one C-SKY vendor entry, how about:
> 
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -347,8 +347,8 @@ patternProperties:
>      description: Creative Technology Ltd
>    "^crystalfontz,.*":
>      description: Crystalfontz America, Inc.
> -  "^csky,.*":
> -    description: Hangzhou C-SKY Microsystems Co., Ltd
> +  "^xuantie,.*":
> +    description: C-SKY Microsystems Co., Ltd. (XuanTie)
>    "^csot,.*":
>      description: Guangzhou China Star Optoelectronics Technology Co., Ltd
>    "^csq,.*":
> 
> The arch/csky is the architecture name, which means "xuantie,csky".
> 
> The c810 is the CPU product name, which means "xuantie,c810" of the
> "xuantie,csky" architecture.
> 
> The c930 is the CPU product name, which means "xuantie,c930".
> 
>>
>> When the vendor literally says "C-SKY", it's hard to buy why csky
>> shouldn't be used here. Xuantie is described as being a brand, where at
>> least t-head was a company. This seems akin to having
>> "snapdragon,msm8660" instead of "qcom,msm8660", for example.
> Yes, XuanTie is not only a brand, but also the abbreviation of C-SKY
> Microsystems Co., Ltd.

"XuanTie is a brand", so not suitable for vendor prefixes.

https://www.xrvm.com/declaration

What's more:

<<A reference to “C-SKY”, “we” or “us” is a reference to C-SKY
Microsystems Co., Ltd. of Room >>

So again c-sky.


Best regards,
Krzysztof
Re: [PATCH v2 1/4] dt-bindings: vendor-prefixes: Add prefix for XuanTie
Posted by Guo Ren 1 week, 3 days ago
On Fri, Nov 21, 2025 at 2:35 PM <cp0613@linux.alibaba.com> wrote:
>
> From: Chen Pei <cp0613@linux.alibaba.com>
>
> Add new vendor strings to dt bindings for RISC-V vendor XuanTie.
>
> Signed-off-by: Chen Pei <cp0613@linux.alibaba.com>
> ---
>  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 f1d1882009ba..5d28b2da94cd 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -1829,6 +1829,8 @@ patternProperties:
>      description: Xilinx
>    "^xnano,.*":
>      description: Xnano
> +  "^xuantie,.*":
> +    description: C-SKY Microsystems Co., Ltd. (XuanTie)
Yes, According to the "Standard Manufacturer's IDENTIFICATION CODE"
[1]. The manufacturer of Code '0x5b7' has changed from thead to
xuantie in 04/30/2024.

105 T-HEAD Semiconductor Co Ltd -> C-SKY Microsystems Co Ltd (XuanTie)

The XuanTie is the abbreviation of C-SKY Microsystems Co., Ltd, which
belongs to Alibaba Damo Academy.

[1] https://www.jedec.org/standards-documents/docs/jep-106ab

Reviewed-by: Guo Ren (Alibaba Damo Academy) <guoren@kernel.org>
Signed-off-by: Guo Ren (Alibaba Damo Academy) <guoren@kernel.org>

>    "^xunlong,.*":
>      description: Shenzhen Xunlong Software CO.,Limited
>    "^xylon,.*":
> --
> 2.50.1
>


--
Best Regards
 Guo Ren