Re: [PATCH 21/32] dt-bindings: Add Kendryte and Canaan vendor prefix

Damien Le Moal
2020-11-09T22:04:41Z

On 2020/11/10 7:01, Rob Herring wrote:
> On Sat, Nov 07, 2020 at 05:14:09PM +0900, Damien Le Moal wrote:
>> Update Documentation/devicetree/bindings/vendor-prefixes.yaml to include
>> "Kendryte" and "canaan" as a vendor prefix for "Canaan Inc."
>>
>> Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
>> ---
>> Documentation/devicetree/bindings/vendor-prefixes.yaml | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> index 2735be1a8470..f53d4d8e7f2a 100644
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> @@ -179,6 +179,8 @@ patternProperties:
>> description: CALAO Systems SAS
>> "^calxeda,.*":
>> description: Calxeda
>> + "^canaan,.*":
>> + description: Canaan, Inc.
>> "^caninos,.*":
>> description: Caninos Loucos Program
>> "^capella,.*":
>> @@ -537,6 +539,8 @@ patternProperties:
>> description: Ka-Ro electronics GmbH
>> "^keithkoep,.*":
>> description: Keith & Koep GmbH
>> + "^kendryte,.*":
>
> AFAICT, this is a marketing name for a series of SoCs? That's not a
> vendor.

Correct. Sean pointed this out to me already. The vendor is "Canaan".
I will change in V2 all compatible strings to have the proper format starting
with the vendor, e.g. "canaan,kendryte-k210-..." or may be more simply "canaan,
k210-...".

>
>> + description: Canaan, Inc.
>> "^keymile,.*":
>> description: Keymile GmbH
>> "^khadas,.*":
>> --
>> 2.28.0
>>
>

--
Damien Le Moal
Western Digital Research