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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxx>
Cc: Guido Günther <agx@xxxxxxxxxxx>
Cc: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx>
Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
Cc: Rob Herring <robh+dt@xxxxxxxxxx>
Cc: Sam Ravnborg <sam@xxxxxxxxxxxx>
Cc: Thierry Reding <thierry.reding@xxxxxxxxx>
---
  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.



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux