1. Patchew
  2. linux
  3. [PATCH v2 0/8] auxdisplay: Add TM16xx and compatible LED display controllers driver
  4. View patch

[PATCH v2 4/8] dt-bindings: vendor-prefixes: Add Winrise Technology

Jean-François Lessard posted 8 patches 3 months, 1 week ago
There is a newer version of this series
[PATCH v2 4/8] dt-bindings: vendor-prefixes: Add Winrise Technology
Posted by Jean-François Lessard 3 months, 1 week ago 
Assign vendor prefix "winrise", matching 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 f03ab02afe..a3bf93e5dc 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1711,6 +1711,8 @@ patternProperties:
     description: Wingtech Technology Co., Ltd.
   "^winlink,.*":
     description: WinLink Co., Ltd
+  "^winrise,.*":
+    description: Shenzhen Winrise Technology Co., Ltd.
   "^winsen,.*":
     description: Winsen Corp.
   "^winstar,.*":
-- 
2.43.0
Re: [PATCH v2 4/8] dt-bindings: vendor-prefixes: Add Winrise Technology
Posted by Krzysztof Kozlowski 3 months, 1 week ago 
On 29/06/2025 14:59, Jean-François Lessard wrote:
> Assign vendor prefix "winrise", matching 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 f03ab02afe..a3bf93e5dc 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -1711,6 +1711,8 @@ patternProperties:
>      description: Wingtech Technology Co., Ltd.
>    "^winlink,.*":
>      description: WinLink Co., Ltd
> +  "^winrise,.*":
> +    description: Shenzhen Winrise Technology Co., Ltd.
Hm? https://winrise.com/ redirects to https://amdaluminum.com/ for
fences and other alu products.

Best regards,
Krzysztof
Re: [PATCH v2 4/8] dt-bindings: vendor-prefixes: Add Winrise Technology
Posted by Christian Hewitt 3 months, 1 week ago 
> On 30 Jun 2025, at 4:25 pm, Krzysztof Kozlowski <krzk@kernel.org> wrote:
> 
> On 29/06/2025 14:59, Jean-François Lessard wrote:
>> Assign vendor prefix "winrise", matching 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 f03ab02afe..a3bf93e5dc 100644
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> @@ -1711,6 +1711,8 @@ patternProperties:
>>     description: Wingtech Technology Co., Ltd.
>>   "^winlink,.*":
>>     description: WinLink Co., Ltd
>> +  "^winrise,.*":
>> +    description: Shenzhen Winrise Technology Co., Ltd.
> Hm? https://winrise.com/ redirects to https://amdaluminum.com/ for
> fences and other alu products.

According to multiple Chinese chip-buying/trade websites [0],[1] and
the internet archive [2] their domain is winrise.cn (not .com). There
is currently no active website despite whois entries showing that the
domain registration is still active/alive.

[0] http://www.84878.tradebig.com/
[1] https://www.tradeeasy.com/supplier/714703/products
[2] https://web.archive.org/web/20160312143416/http://winrise.cn/

If you’d prefer “Assign vendor prefix based on their name” as the
patch description? we can change that for the next iteration.

CH.
Re: [PATCH v2 4/8] dt-bindings: vendor-prefixes: Add Winrise Technology
Posted by Krzysztof Kozlowski 3 months, 1 week ago 
On 30/06/2025 15:51, Christian Hewitt wrote:
>> On 30 Jun 2025, at 4:25 pm, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>
>> On 29/06/2025 14:59, Jean-François Lessard wrote:
>>> Assign vendor prefix "winrise", matching 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 f03ab02afe..a3bf93e5dc 100644
>>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>> @@ -1711,6 +1711,8 @@ patternProperties:
>>>     description: Wingtech Technology Co., Ltd.
>>>   "^winlink,.*":
>>>     description: WinLink Co., Ltd
>>> +  "^winrise,.*":
>>> +    description: Shenzhen Winrise Technology Co., Ltd.
>> Hm? https://winrise.com/ redirects to https://amdaluminum.com/ for
>> fences and other alu products.
> 
> According to multiple Chinese chip-buying/trade websites [0],[1] and
> the internet archive [2] their domain is winrise.cn (not .com). There
> is currently no active website despite whois entries showing that the
> domain registration is still active/alive.
> 
> [0] http://www.84878.tradebig.com/
> [1] https://www.tradeeasy.com/supplier/714703/products
> [2] https://web.archive.org/web/20160312143416/http://winrise.cn/
> 
> If you’d prefer “Assign vendor prefix based on their name” as the
> patch description? we can change that for the next iteration.
If commit msg says "domain name" as an argument and it turns out it is
not matching domain name, then that other domain name least needs to be
in commit msg. The rule of domain name comes from US tickers, so only
.com should be considered. If there is no conflict and no .com
manufacturer it is fine to use whatever other name, but the commit msg
is not then correct.

Best regards,
Krzysztof
Re: [PATCH v2 4/8] dt-bindings: vendor-prefixes: Add Winrise Technology
Posted by Jean-François Lessard 3 months, 1 week ago 
Le 2 juillet 2025 16 h 14 min 34 s HAE, Krzysztof Kozlowski <krzk@kernel.org> a écrit :
>On 30/06/2025 15:51, Christian Hewitt wrote:
>>> On 30 Jun 2025, at 4:25 pm, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>>
>>> On 29/06/2025 14:59, Jean-François Lessard wrote:
>>>> Assign vendor prefix "winrise", matching 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 f03ab02afe..a3bf93e5dc 100644
>>>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>>> @@ -1711,6 +1711,8 @@ patternProperties:
>>>>     description: Wingtech Technology Co., Ltd.
>>>>   "^winlink,.*":
>>>>     description: WinLink Co., Ltd
>>>> +  "^winrise,.*":
>>>> +    description: Shenzhen Winrise Technology Co., Ltd.
>>> Hm? https://winrise.com/ redirects to https://amdaluminum.com/ for
>>> fences and other alu products.
>> 
>> According to multiple Chinese chip-buying/trade websites [0],[1] and
>> the internet archive [2] their domain is winrise.cn (not .com). There
>> is currently no active website despite whois entries showing that the
>> domain registration is still active/alive.
>> 
>> [0] http://www.84878.tradebig.com/
>> [1] https://www.tradeeasy.com/supplier/714703/products
>> [2] https://web.archive.org/web/20160312143416/http://winrise.cn/
>> 
>> If you’d prefer “Assign vendor prefix based on their name” as the
>> patch description? we can change that for the next iteration.
>If commit msg says "domain name" as an argument and it turns out it is
>not matching domain name, then that other domain name least needs to be
>in commit msg. The rule of domain name comes from US tickers, so only
>.com should be considered. If there is no conflict and no .com
>manufacturer it is fine to use whatever other name, but the commit msg
>is not then correct.
>
>Best regards,
>Krzysztof

Well noted, for v3:
- Will update the commit message (non-ticker/.com)
- Will avoid generic vendor prefixes (icore)
- Will submit the 4 vendor prefix changes in one patch

Best regards,
Jean-François Lessard

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.