Hi Benjamin, and many thanks for this serie. Regarding this patch:

Reviewed-by: Philippe Cornu philippe.cornu@st.com Philippe :-)

On 2/6/20 2:33 PM, Benjamin Gaignard wrote:

From: Sam Ravnborg sam@ravnborg.org

To complement panel-simple.yaml, create panel-simple-dsi.yaml. panel-simple-dsi-yaml are for all simple DSP panels with a single power-supply and optional backlight / enable GPIO.

Migrate panasonic,vvx10f034n00 over to the new file.

The objectives with one file for all the simple DSI panels are: - Make it simpler to add bindings for simple DSI panels - Keep the number of bindings file lower - Keep the binding documentation for simple DSI panels more consistent

Signed-off-by: Sam Ravnborg sam@ravnborg.org Signed-off-by: Benjamin Gaignard benjamin.gaignard@st.com Cc: Thierry Reding thierry.reding@gmail.com Cc: Rob Herring robh@kernel.org Cc: Maxime Ripard mripard@kernel.org Cc: Yannick Fertre yannick.fertre@st.com Cc: Mark Rutland mark.rutland@arm.com Cc: Daniel Vetter daniel@ffwll.ch Cc: dri-devel@lists.freedesktop.org Cc: devicetree@vger.kernel.org

---

version 4:

• remove orisetech,otm8009a and raydium,rm68200 compatibles
• remove reset-gpios optional property

version 3:

• add orisetech,otm8009a and raydium,rm68200 compatibles

• add reset-gpios optional property

• fix indentation on compatible enumeration

  .../display/panel/panasonic,vvx10f034n00.txt | 20 ------- .../bindings/display/panel/panel-simple-dsi.yaml | 67 ++++++++++++++++++++++ 2 files changed, 67 insertions(+), 20 deletions(-) delete mode 100644 Documentation/devicetree/bindings/display/panel/panasonic,vvx10f034n00.txt create mode 100644 Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml

diff --git a/Documentation/devicetree/bindings/display/panel/panasonic,vvx10f034n00.txt b/Documentation/devicetree/bindings/display/panel/panasonic,vvx10f034n00.txt deleted file mode 100644 index 37dedf6a6702..000000000000 --- a/Documentation/devicetree/bindings/display/panel/panasonic,vvx10f034n00.txt +++ /dev/null @@ -1,20 +0,0 @@ -Panasonic 10" WUXGA TFT LCD panel

-Required properties: -- compatible: should be "panasonic,vvx10f034n00" -- reg: DSI virtual channel of the peripheral -- power-supply: phandle of the regulator that provides the supply voltage

-Optional properties: -- backlight: phandle of the backlight device attached to the panel

-Example:

• mdss_dsi@fd922800 {

• panel@0 {
  

• 	compatible = "panasonic,vvx10f034n00";
  

• 	reg = <0>;
  

• 	power-supply = <&vreg_vsp>;
  

• 	backlight = <&lp8566_wled>;
  

• };
  

• };

diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml new file mode 100644 index 000000000000..8b60368a2425 --- /dev/null +++ b/Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml @@ -0,0 +1,67 @@ +# SPDX-License-Identifier: (GPL-2.0-only or BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/panel/panel-simple-dsi.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#

+title: Simple DSI panels with a single power-supply

+maintainers:

• • Thierry Reding thierry.reding@gmail.com
• • Sam Ravnborg sam@ravnborg.org

+description: |

• This binding file is a collection of the DSI panels that
• requires only a single power-supply.
• There are optionally a backlight and an enable GPIO.
• The panel may use an OF graph binding for the association to the display,
• or it may be a direct child node of the display.
• If the panel is more advanced a dedicated binding file is required.

+allOf:

• • $ref: panel-common.yaml#

+properties:

• compatible:
• enum:

•  # compatible must be listed in alphabetical order, ordered by compatible.
  

•  # The description in the comment is mandatory for each compatible.
  

•    # Panasonic 10" WUXGA TFT LCD panel
  

•  - panasonic,vvx10f034n00
  

• reg:
• maxItems: 1
• description: DSI virtual channel
• backlight: true
• enable-gpios: true
• port: true
• power-supply: true

+additionalProperties: false

+required:

• • compatible
• • power-supply
• • reg

+examples:

• • |
• mdss_dsi@fd922800 {

•  #address-cells = <1>;
  

•  #size-cells = <0>;
  

•  panel@0 {
  

•    compatible = "panasonic,vvx10f034n00";
  

•    reg = <0>;
  

•    power-supply = <&vcc_lcd_reg>;
  

•    port {
  

•      panel: endpoint {
  

•        remote-endpoint = <&ltdc_out>;
  

•      };
  

•    };
  

•  };
  

• };

Hi Benjamin,

On 2/6/20 2:33 PM, Benjamin Gaignard wrote:

Convert orisetech,otm8009a to json-schema.

Signed-off-by: Benjamin Gaignard benjamin.gaignard@st.com

.../bindings/display/panel/orisetech,otm8009a.txt | 23 ---------- .../bindings/display/panel/orisetech,otm8009a.yaml | 53 ++++++++++++++++++++++ 2 files changed, 53 insertions(+), 23 deletions(-) delete mode 100644 Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.txt create mode 100644 Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.yaml

diff --git a/Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.txt b/Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.txt deleted file mode 100644 index 203b03eefb68..000000000000 --- a/Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.txt +++ /dev/null @@ -1,23 +0,0 @@ -Orise Tech OTM8009A 3.97" 480x800 TFT LCD panel (MIPI-DSI video mode)

-The Orise Tech OTM8009A is a 3.97" 480x800 TFT LCD panel connected using -a MIPI-DSI video interface. Its backlight is managed through the DSI link.

-Required properties:

• • compatible: "orisetech,otm8009a"
• • reg: the virtual channel number of a DSI peripheral

-Optional properties:

• • reset-gpios: a GPIO spec for the reset pin (active low).
• • power-supply: phandle of the regulator that provides the supply voltage.

-Example: -&dsi {

• ...
• panel@0 {

• compatible = "orisetech,otm8009a";
  

• reg = <0>;
  

• reset-gpios = <&gpioh 7 GPIO_ACTIVE_LOW>;
  

• power-supply = <&v1v8>;
  

• };

-}; diff --git a/Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.yaml b/Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.yaml new file mode 100644 index 000000000000..6e6ac995c27b --- /dev/null +++ b/Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.yaml @@ -0,0 +1,53 @@ +# SPDX-License-Identifier: (GPL-2.0-only or BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/panel/orisetech,otm8009a.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#

+title: Orise Tech OTM8009A 3.97" 480x800 TFT LCD panel (MIPI-DSI video mode)

+maintainers:

• • Philippe CORNU philippe.cornu@st.com

+description: |

•         The Orise Tech OTM8009A is a 3.97" 480x800 TFT LCD panel connected using
  

•         a MIPI-DSI video interface. Its backlight is managed through the DSI link.
  

+allOf:

• • $ref: panel-common.yaml#

+properties:

• compatible:
• const: orisetech,otm8009a
• reg:
• maxItems: 1
• description: DSI virtual channel
• enable-gpios: true
• port: true
• power-supply: true
• reset-gpios:
• maxItems: 1

+additionalProperties: false

+required:

• • compatible
• • reg

+examples:

• • |
• dsi@0 {

•  #address-cells = <1>;
  

•  #size-cells = <0>;
  

•  panel@0 {
  

•    compatible = "orisetech,otm8009a";
  

•    reg = <0>;
  

•    reset-gpios = <&gpiof 15 0>;
  

•    power-supply = <&v1v8>;
  

•  };
  

• };

+...

Reviewed-by: Philippe Cornu philippe.cornu@st.com Thank you Philippe :-)