[PATCH] dt-bindings: vendor-prefixes: add Densitron

[Marek Vasut]

On 7/21/22 07:41, Laurent Pinchart wrote:
> Hi Marek,
> 
> Thank you for the patch.
> 
> On Thu, Jul 21, 2022 at 05:03:27AM +0200, Marek Vasut wrote:
>> Densitron is a manufacturer of LCD panels.
>> https://www.densitron.com
>>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> Cc: Guido Günther <agx at sigxcpu.org>
>> Cc: Jagan Teki <jagan at amarulasolutions.com>
>> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>> Cc: Linus Walleij <linus.walleij at linaro.org>
>> Cc: Rob Herring <robh+dt at kernel.org>
>> Cc: Sam Ravnborg <sam at ravnborg.org>
>> Cc: Thierry Reding <thierry.reding at 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 88859dd4040ee..6277240536b44 100644
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> @@ -312,6 +312,8 @@ patternProperties:
>>       description: Dell Inc.
>>     "^delta,.*":
>>       description: Delta Electronics, Inc.
>> +  "^densitron,.*":
> 
> How about "dsn", to follow the practice of using stock names as vendor
> prefixes ?

Is there any benefit to that ? All I can see is that it's making DTS 
less clear and more difficult to read. It is easy to map "densitron" to 
"densitron" when it is spelled out like so in the DT, but it sure isn't 
immediately obvious that "dsn" means "densitron" without extra look up. 
And even that extra look up of "dsn" does not return densitron, but some 
woodworking company and other totally unrelated results.