Re: [PATCH v5 1/4] dt-bindings: display: bridge: ldb: Adjust imx6sx entries

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

 



On 5/22/23 14:51, Fabio Estevam wrote:
From: Fabio Estevam <festevam@xxxxxxx>

On the i.MX6SX there is a single entry for 'reg' and 'reg-names', so add
some logic to reflect that.

Signed-off-by: Fabio Estevam <festevam@xxxxxxx>
---
Changes since v4:
- Replaced items: with const: for the single item (Conor)

  .../bindings/display/bridge/fsl,ldb.yaml      | 26 ++++++++++++++++---
  1 file changed, 23 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
index 07388bf2b90d..d97064140547 100644
--- a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
@@ -28,12 +28,12 @@ properties:
      const: ldb
reg:
+    minItems: 1
      maxItems: 2
reg-names:
-    items:
-      - const: ldb
-      - const: lvds
+    minItems: 1
+    maxItems: 2
ports:
      $ref: /schemas/graph.yaml#/properties/ports
@@ -74,6 +74,26 @@ allOf:
            properties:
              port@2: false
+ - if:
+      properties:
+        compatible:
+          contains:
+            const: fsl,imx6sx-ldb
+    then:
+      properties:
+        reg:
+          maxItems: 1
+        reg-names:
+           const: ldb

Can we drop the reg-names on MX6SX if we have reg maxItems=1 here ?

I think we can, but we need to retain that on the newer iMXes which have reg maxItems=2



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux