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

["Jean-François Lessard" <jefflessard3@gmail.com>]

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