dt-bindings: display: Add dsi-controller.yaml in DSI controller schemas
Subsystems (10)
Rob Herring
maintainerPATCH 1/1 dt-bindings: display: Add dsi-controller.yaml in DSI controller schemas
Some DSI controllers are missing a reference to the recently added dsi-controller.yaml schema. Add it and we can drop the duplicate parts.
Cc Maxime Ripard <[email protected]>
Cc Chen-Yu Tsai <[email protected]>
Cc Eric Anholt <[email protected]>
Cc Nicolas Saenz Julienne <[email protected]>
Cc Florian Fainelli <[email protected]>
Cc Ray Jui <[email protected]>
Cc Scott Branden <[email protected]>
Cc Maxime Coquelin <[email protected]>
Cc Alexandre Torgue <[email protected]>
Cc "Guido Gúnther" <[email protected]>
Cc Robert Chiras <[email protected]>
Cc Philippe Cornu <[email protected]>
Cc Yannick Fertre <[email protected]>
Signed-off-by Rob Herring <[email protected]>
--- .../display/allwinner,sun6i-a31-mipi-dsi.yaml | 11 ++------- .../bindings/display/brcm,bcm2835-dsi0.yaml | 3 +++ .../bindings/display/bridge/nwl-dsi.yaml | 11 ++++----- .../bindings/display/st,stm32-dsi.yaml | 23 ++++--------------- 4 files changed, 14 insertions(+), 34 deletions(-)
Documentation/devicetree/bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml +2 -9
@@ -11,9 +11,6 @@ maintainers: - Maxime Ripard <[email protected]> properties: - "#address-cells": true - "#size-cells": true - compatible: enum: - allwinner,sun6i-a31-mipi-dsi
@@ -57,12 +54,7 @@ properties: port should be the input endpoint, usually coming from the associated TCON. -patternProperties: - "^panel@[0-9]+$": true - required: - - "#address-cells" - - "#size-cells" - compatible - reg - interrupts
@@ -74,6 +66,7 @@ required: - port allOf: + - $ref: dsi-controller.yaml# - if: properties: compatible:
@@ -99,7 +92,7 @@ allOf: clocks: minItems: 1 -additionalProperties: false +unevaluatedProperties: false examples: - |
Documentation/devicetree/bindings/display/brcm,bcm2835-dsi0.yaml +3 -0
@@ -9,6 +9,9 @@ title: Broadcom VC4 (VideoCore4) DSI Controller maintainers: - Eric Anholt <[email protected]> +allOf: + - $ref: dsi-controller.yaml# + properties: "#clock-cells": const: 1
Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml +5 -6
@@ -14,6 +14,9 @@ description: | NWL MIPI-DSI host controller found on i.MX8 platforms. This is a dsi bridge for the SOCs NWL MIPI-DSI host controller. +allOf: + - $ref: ../dsi-controller.yaml# + properties: compatible: const: fsl,imx8mq-nwl-dsi
@@ -144,10 +147,6 @@ properties: additionalProperties: false -patternProperties: - "^panel@[0-9]+$": - type: object - required: - '#address-cells' - '#size-cells'
@@ -163,7 +162,7 @@ required: - reset-names - resets -additionalProperties: false +unevaluatedProperties: false examples: - |
@@ -172,7 +171,7 @@ examples: #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/reset/imx8mq-reset.h> - mipi_dsi: mipi_dsi@30a00000 { + dsi@30a00000 { #address-cells = <1>; #size-cells = <0>; compatible = "fsl,imx8mq-nwl-dsi";
Documentation/devicetree/bindings/display/st,stm32-dsi.yaml +4 -19
@@ -13,6 +13,9 @@ maintainers: description: The STMicroelectronics STM32 DSI controller uses the Synopsys DesignWare MIPI-DSI host controller. +allOf: + - $ref: dsi-controller.yaml# + properties: compatible: const: st,stm32-dsi
@@ -65,24 +68,6 @@ properties: description: DSI output port node, connected to a panel or a bridge input port" -patternProperties: - "^(panel|panel-dsi)@[0-9]$": - type: object - description: - A node containing the panel or bridge description as documented in - Documentation/devicetree/bindings/display/mipi-dsi-bus.txt - properties: - port: - type: object - description: - Panel or bridge port node, connected to the DSI output port (port@1) - - "#address-cells": - const: 1 - - "#size-cells": - const: 0 - required: - "#address-cells" - "#size-cells"
@@ -92,7 +77,7 @@ required: - clock-names - ports -additionalProperties: false +unevaluatedProperties: false examples: - |
Signature · 1 line
2.25.1
Hi,
Quoted 3 lines · Rob Herring
Some DSI controllers are missing a reference to the recently added
dsi-controller.yamlschema. Add it and we can drop the duplicate parts.
For the NWL part:
Reviewed-by Guido Günther <[email protected]>
Quoted 175 lines
Cc Maxime Ripard <[email protected]>Cc Chen-Yu Tsai <[email protected]>Cc Eric Anholt <[email protected]>Cc Nicolas Saenz Julienne <[email protected]>Cc Florian Fainelli <[email protected]>Cc Ray Jui <[email protected]>Cc Scott Branden <[email protected]>Cc Maxime Coquelin <[email protected]>Cc Alexandre Torgue <[email protected]>Cc "Guido Gúnther" <[email protected]>Cc Robert Chiras <[email protected]>Cc Philippe Cornu <[email protected]>Cc Yannick Fertre <[email protected]>Signed-off-by Rob Herring <[email protected]>--- .../display/allwinner,sun6i-a31-mipi-dsi.yaml | 11 ++------- .../bindings/display/brcm,bcm2835-dsi0.yaml | 3 +++ .../bindings/display/bridge/nwl-dsi.yaml | 11 ++++----- .../bindings/display/st,stm32-dsi.yaml | 23 ++++--------------- 4 files changed, 14 insertions(+), 34 deletions(-)Documentation/devicetree/bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml +2
-9@@ -11,9 +11,6 @@ maintainers: - Maxime Ripard <[email protected]> properties: - "#address-cells": true - "#size-cells": true - compatible: enum: - allwinner,sun6i-a31-mipi-dsi@@ -57,12 +54,7 @@ properties: port should be the input endpoint, usually coming from the associated TCON. -patternProperties: - "^panel@[0-9]+$": true - required: - - "#address-cells" - - "#size-cells" - compatible - reg - interrupts@@ -74,6 +66,7 @@ required: - port allOf: + - $ref: dsi-controller.yaml# - if: properties: compatible:@@ -99,7 +92,7 @@ allOf: clocks: minItems: 1 -additionalProperties: false +unevaluatedProperties: false examples: - |Documentation/devicetree/bindings/display/brcm,bcm2835-dsi0.yaml +3
-0@@ -9,6 +9,9 @@ title: Broadcom VC4 (VideoCore4) DSI Controller maintainers: - Eric Anholt <[email protected]> +allOf: + - $ref: dsi-controller.yaml# + properties: "#clock-cells": const: 1Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml +5
-6@@ -14,6 +14,9 @@ description: | NWL MIPI-DSI host controller found on i.MX8 platforms. This is a dsi bridge for the SOCs NWL MIPI-DSI host controller. +allOf: + - $ref: ../dsi-controller.yaml# + properties: compatible: const: fsl,imx8mq-nwl-dsi@@ -144,10 +147,6 @@ properties: additionalProperties: false -patternProperties: - "^panel@[0-9]+$": - type: object - required: - '#address-cells' - '#size-cells'@@ -163,7 +162,7 @@ required: - reset-names - resets -additionalProperties: false +unevaluatedProperties: false examples: - |@@ -172,7 +171,7 @@ examples: #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/reset/imx8mq-reset.h> - mipi_dsi: mipi_dsi@30a00000 { + dsi@30a00000 { #address-cells = <1>; #size-cells = <0>; compatible = "fsl,imx8mq-nwl-dsi";Documentation/devicetree/bindings/display/st,stm32-dsi.yaml +4
-19@@ -13,6 +13,9 @@ maintainers: description: The STMicroelectronics STM32 DSI controller uses the Synopsys DesignWare MIPI-DSI host controller. +allOf: + - $ref: dsi-controller.yaml# + properties: compatible: const: st,stm32-dsi@@ -65,24 +68,6 @@ properties: description: DSI output port node, connected to a panel or a bridge input port" -patternProperties: - "^(panel|panel-dsi)@[0-9]$": - type: object - description: - A node containing the panel or bridge description as documented in - Documentation/devicetree/bindings/display/mipi-dsi-bus.txt - properties: - port: - type: object - description: - Panel or bridge port node, connected to the DSI output port (port@1) - - "#address-cells": - const: 1 - - "#size-cells": - const: 0 - required: - "#address-cells" - "#size-cells"@@ -92,7 +77,7 @@ required: - clock-names - ports -additionalProperties: false +unevaluatedProperties: false examples: - |Signature · 1 line
2.25.1
Quoted 4 lines · Rob Herring
Some DSI controllers are missing a reference to the recently added
dsi-controller.yamlschema. Add it and we can drop the duplicate parts.Cc Maxime Ripard <[email protected]>
Acked-by Maxime Ripard <[email protected]>
Thanks!
Maxime
Quoted 16 lines · Rob Herring
Some DSI controllers are missing a reference to the recently added
dsi-controller.yamlschema. Add it and we can drop the duplicate parts.Cc Maxime Ripard <[email protected]>Cc Chen-Yu Tsai <[email protected]>Cc Eric Anholt <[email protected]>Cc Nicolas Saenz Julienne <[email protected]>Cc Florian Fainelli <[email protected]>Cc Ray Jui <[email protected]>Cc Scott Branden <[email protected]>Cc Maxime Coquelin <[email protected]>Cc Alexandre Torgue <[email protected]>Cc "Guido Gúnther" <[email protected]>Cc Robert Chiras <[email protected]>Cc Philippe Cornu <[email protected]>
Hi Rob,
and many thanks for the patch.
For the stm32 part,
Reviewed-by Philippe Cornu <[email protected]>
Philippe :-)