[PATCH v2 5/8] dt-bindings: vendor-prefixes: Add Wuxi i-Core Electronics

Jean-François Lessard posted 8 patches 3 months, 1 week ago
There is a newer version of this series
[PATCH v2 5/8] dt-bindings: vendor-prefixes: Add Wuxi i-Core Electronics
Posted by Jean-François Lessard 3 months, 1 week ago
Assign vendor prefix "icore", based on their domain name.

Signed-off-by: Jean-François Lessard <jefflessard3@gmail.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 a3bf93e5dc..def3530099 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -694,6 +694,8 @@ patternProperties:
     description: International Business Machines (IBM)
   "^ibm,.*":
     description: International Business Machines (IBM)
+  "^icore,.*":
+    description: Wuxi i-Core Electronics Co., Ltd.
   "^icplus,.*":
     description: IC Plus Corp.
   "^idt,.*":
-- 
2.43.0

Re: [PATCH v2 5/8] dt-bindings: vendor-prefixes: Add Wuxi i-Core Electronics
Posted by Geert Uytterhoeven 3 months, 1 week ago
Hi Jean-François,

On Sun, 29 Jun 2025 at 15:00, Jean-François Lessard
<jefflessard3@gmail.com> wrote:
> Assign vendor prefix "icore", based on their domain name.
>
> Signed-off-by: Jean-François Lessard <jefflessard3@gmail.com>

Thanks for your patch!

> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -694,6 +694,8 @@ patternProperties:
>      description: International Business Machines (IBM)
>    "^ibm,.*":
>      description: International Business Machines (IBM)
> +  "^icore,.*":
> +    description: Wuxi i-Core Electronics Co., Ltd.

This sounds a bit too generic to me.  What is the domain name?

>    "^icplus,.*":
>      description: IC Plus Corp.
>    "^idt,.*":
> --

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Re: [PATCH v2 5/8] dt-bindings: vendor-prefixes: Add Wuxi i-Core Electronics
Posted by Krzysztof Kozlowski 3 months, 1 week ago
On 30/06/2025 10:19, Geert Uytterhoeven wrote:
> Hi Jean-François,
> 
> On Sun, 29 Jun 2025 at 15:00, Jean-François Lessard
> <jefflessard3@gmail.com> wrote:
>> Assign vendor prefix "icore", based on their domain name.
>>
>> Signed-off-by: Jean-François Lessard <jefflessard3@gmail.com>
> 
> Thanks for your patch!
> 
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> @@ -694,6 +694,8 @@ patternProperties:
>>      description: International Business Machines (IBM)
>>    "^ibm,.*":
>>      description: International Business Machines (IBM)
>> +  "^icore,.*":
>> +    description: Wuxi i-Core Electronics Co., Ltd.
> 
> This sounds a bit too generic to me.  What is the domain name?

Good point. icore.com should be but obviously it points somewhere else,
so this does not follow ticker / domain name style.

Best regards,
Krzysztof
Re: [PATCH v2 5/8] dt-bindings: vendor-prefixes: Add Wuxi i-Core Electronics
Posted by Christian Hewitt 3 months, 1 week ago
> On 30 Jun 2025, at 4:24 pm, Krzysztof Kozlowski <krzk@kernel.org> wrote:
> 
> On 30/06/2025 10:19, Geert Uytterhoeven wrote:
>> Hi Jean-François,
>> 
>> On Sun, 29 Jun 2025 at 15:00, Jean-François Lessard
>> <jefflessard3@gmail.com> wrote:
>>> Assign vendor prefix "icore", based on their domain name.
>>> 
>>> Signed-off-by: Jean-François Lessard <jefflessard3@gmail.com>
>> 
>> Thanks for your patch!
>> 
>>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>> @@ -694,6 +694,8 @@ patternProperties:
>>>     description: International Business Machines (IBM)
>>>   "^ibm,.*":
>>>     description: International Business Machines (IBM)
>>> +  "^icore,.*":
>>> +    description: Wuxi i-Core Electronics Co., Ltd.
>> 
>> This sounds a bit too generic to me.  What is the domain name?
> 
> Good point. icore.com should be but obviously it points somewhere else,
> so this does not follow ticker / domain name style.

As per the response to Geert earlier, the domain is i-core.cn (not .com)

See: http://www.i-core.cn/en/

CH.
Re: [PATCH v2 5/8] dt-bindings: vendor-prefixes: Add Wuxi i-Core Electronics
Posted by Christian Hewitt 3 months, 1 week ago
> On 30 Jun 2025, at 12:19 pm, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> 
> Hi Jean-François,
> 
> On Sun, 29 Jun 2025 at 15:00, Jean-François Lessard
> <jefflessard3@gmail.com> wrote:
>> Assign vendor prefix "icore", based on their domain name.
>> 
>> Signed-off-by: Jean-François Lessard <jefflessard3@gmail.com>
> 
> Thanks for your patch!
> 
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> @@ -694,6 +694,8 @@ patternProperties:
>>     description: International Business Machines (IBM)
>>   "^ibm,.*":
>>     description: International Business Machines (IBM)
>> +  "^icore,.*":
>> +    description: Wuxi i-Core Electronics Co., Ltd.
> 
> This sounds a bit too generic to me.  What is the domain name?

Their domain/website is http://www.i-core.cn/en/ and i-Core is used
as the watermark in their datasheets [0]. We’ve thought to drop the
hyphen and use plain ‘icore’ to avoid future typos.

[0] https://github.com/jefflessard/tm16xx-display/blob/main/datasheets/AiP1628.pdf

CH.

>>   "^icplus,.*":
>>     description: IC Plus Corp.
>>   "^idt,.*":
>> --
> 
> Gr{oetje,eeting}s,
> 
>                        Geert
> 
> -- 
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                -- Linus Torvalds
Re: [PATCH v2 5/8] dt-bindings: vendor-prefixes: Add Wuxi i-Core Electronics
Posted by Geert Uytterhoeven 3 months, 1 week ago
Hi Christian,

On Mon, 30 Jun 2025 at 10:31, Christian Hewitt
<christianshewitt@gmail.com> wrote:
> > On 30 Jun 2025, at 12:19 pm, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > On Sun, 29 Jun 2025 at 15:00, Jean-François Lessard
> > <jefflessard3@gmail.com> wrote:
> >> Assign vendor prefix "icore", based on their domain name.
> >>
> >> Signed-off-by: Jean-François Lessard <jefflessard3@gmail.com>
> >
> > Thanks for your patch!
> >
> >> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> >> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> >> @@ -694,6 +694,8 @@ patternProperties:
> >>     description: International Business Machines (IBM)
> >>   "^ibm,.*":
> >>     description: International Business Machines (IBM)
> >> +  "^icore,.*":
> >> +    description: Wuxi i-Core Electronics Co., Ltd.
> >
> > This sounds a bit too generic to me.  What is the domain name?
>
> Their domain/website is http://www.i-core.cn/en/ and i-Core is used
> as the watermark in their datasheets [0]. We’ve thought to drop the
> hyphen and use plain ‘icore’ to avoid future typos.

OK (i-core.com seems to be unused, and parked).

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Re: [PATCH v2 5/8] dt-bindings: vendor-prefixes: Add Wuxi i-Core Electronics
Posted by Krzysztof Kozlowski 3 months, 1 week ago
On 29/06/2025 14:59, Jean-François Lessard wrote:
> Assign vendor prefix "icore", based on their domain name.
> 
> Signed-off-by: Jean-François Lessard <jefflessard3@gmail.com>
> ---

These 4 vendor prefix changes are one patch.

Best regards,
Krzysztof