[PATCH] dt-bindings: media: st,stmipid02: Convert the text bindings to YAML
Benjamin MUGNIER
benjamin.mugnier at foss.st.com
Tue Sep 27 04:06:43 PDT 2022
Hi Marek,
Thank you for your patch.
On 9/27/22 02:55, Marek Vasut wrote:
> Convert the text STMIPID02 DT bindings to YAML DT format to permit
> validation of DTs using this I2C CSI-2 to CPI bridge.
>
> Signed-off-by: Marek Vasut <marex at denx.de>
> ---
> Cc: Alexandre Torgue <alexandre.torgue at foss.st.com>
> Cc: Benjamin Mugnier <benjamin.mugnier at foss.st.com>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> Cc: Mauro Carvalho Chehab <mchehab at kernel.org>
> Cc: Maxime Coquelin <mcoquelin.stm32 at gmail.com>
> Cc: Rob Herring <robh+dt at kernel.org>
> Cc: Sylvain Petinot <sylvain.petinot at foss.st.com>
> Cc: devicetree at vger.kernel.org
> Cc: linux-media at vger.kernel.org
> Cc: linux-stm32 at st-md-mailman.stormreply.com
> To: linux-arm-kernel at lists.infradead.org
> ---
> .../bindings/media/i2c/st,st-mipid02.txt | 82 --------
> .../bindings/media/i2c/st,st-mipid02.yaml | 175 ++++++++++++++++++
> 2 files changed, 175 insertions(+), 82 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/media/i2c/st,st-mipid02.txt
> create mode 100644 Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.txt b/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.txt
> deleted file mode 100644
> index 7976e6c40a80a..0000000000000
> --- a/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.txt
> +++ /dev/null
> @@ -1,82 +0,0 @@
> -STMicroelectronics MIPID02 CSI-2 to PARALLEL bridge
> -
> -MIPID02 has two CSI-2 input ports, only one of those ports can be active at a
> -time. Active port input stream will be de-serialized and its content outputted
> -through PARALLEL output port.
> -CSI-2 first input port is a dual lane 800Mbps per lane whereas CSI-2 second
> -input port is a single lane 800Mbps. Both ports support clock and data lane
> -polarity swap. First port also supports data lane swap.
> -PARALLEL output port has a maximum width of 12 bits.
> -Supported formats are RAW6, RAW7, RAW8, RAW10, RAW12, RGB565, RGB888, RGB444,
> -YUV420 8-bit, YUV422 8-bit and YUV420 10-bit.
> -
> -Required Properties:
> -- compatible: shall be "st,st-mipid02"
> -- clocks: reference to the xclk input clock.
> -- clock-names: shall be "xclk".
> -- VDDE-supply: sensor digital IO supply. Must be 1.8 volts.
> -- VDDIN-supply: sensor internal regulator supply. Must be 1.8 volts.
> -
> -Optional Properties:
> -- reset-gpios: reference to the GPIO connected to the xsdn pin, if any.
> - This is an active low signal to the mipid02.
> -
> -Required subnodes:
> - - ports: A ports node with one port child node per device input and output
> - port, in accordance with the video interface bindings defined in
> - Documentation/devicetree/bindings/media/video-interfaces.txt. The
> - port nodes are numbered as follows:
> -
> - Port Description
> - -----------------------------
> - 0 CSI-2 first input port
> - 1 CSI-2 second input port
> - 2 PARALLEL output
> -
> -Endpoint node required property for CSI-2 connection is:
> -- data-lanes: shall be <1> for Port 1. for Port 0 dual-lane operation shall be
> -<1 2> or <2 1>. For Port 0 single-lane operation shall be <1> or <2>.
> -Endpoint node optional property for CSI-2 connection is:
> -- lane-polarities: any lane can be inverted or not.
> -
> -Endpoint node required property for PARALLEL connection is:
> -- bus-width: shall be set to <6>, <7>, <8>, <10> or <12>.
> -Endpoint node optional properties for PARALLEL connection are:
> -- hsync-active: active state of the HSYNC signal, 0/1 for LOW/HIGH respectively.
> -LOW being the default.
> -- vsync-active: active state of the VSYNC signal, 0/1 for LOW/HIGH respectively.
> -LOW being the default.
> -
> -Example:
> -
> -mipid02: csi2rx at 14 {
> - compatible = "st,st-mipid02";
> - reg = <0x14>;
> - status = "okay";
> - clocks = <&clk_ext_camera_12>;
> - clock-names = "xclk";
> - VDDE-supply = <&vdd>;
> - VDDIN-supply = <&vdd>;
> - ports {
> - #address-cells = <1>;
> - #size-cells = <0>;
> - port at 0 {
> - reg = <0>;
> -
> - ep0: endpoint {
> - data-lanes = <1 2>;
> - remote-endpoint = <&mipi_csi2_in>;
> - };
> - };
> - port at 2 {
> - reg = <2>;
> -
> - ep2: endpoint {
> - bus-width = <8>;
> - hsync-active = <0>;
> - vsync-active = <0>;
> - remote-endpoint = <¶llel_out>;
> - };
> - };
> - };
> -};
> diff --git a/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml b/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml
> new file mode 100644
> index 0000000000000..2cb117d883368
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml
> @@ -0,0 +1,175 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/media/i2c/st,st-mipid02.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: STMicroelectronics MIPID02 CSI-2 to PARALLEL bridge
> +
> +maintainers:
> + - Mickael Guene <mickael.guene at foss.st.com>
As Mickael is not the maintainer of the driver anymore, there should be both current maintainers instead, as listed in the MAINTAINER file:
- Benjamin Mugnier <benjamin.mugnier at foss.st.com>
- Sylvain Petinot <sylvain.petinot at foss.st.com>
Can you change it while at it ? Thank you.
By the way you will also need to update the MAINTAINERS file to change the st,st-mipid02.txt file to st,st-mipid02.yaml here:
F: Documentation/devicetree/bindings/media/i2c/st,st-mipid02.txt
> + - Marek Vasut <marex at denx.de>
> +
> +description:
> + MIPID02 has two CSI-2 input ports, only one of those ports can be
> + active at a time. Active port input stream will be de-serialized
> + and its content outputted through PARALLEL output port.
> + CSI-2 first input port is a dual lane 800Mbps per lane whereas CSI-2
> + second input port is a single lane 800Mbps. Both ports support clock
> + and data lane polarity swap. First port also supports data lane swap.
> + PARALLEL output port has a maximum width of 12 bits.
> + Supported formats are RAW6, RAW7, RAW8, RAW10, RAW12, RGB565, RGB888,
> + RGB444, YUV420 8-bit, YUV422 8-bit and YUV420 10-bit.
> +
> +properties:
> + compatible:
> + const: st,st-mipid02
> +
> + reg:
> + maxItems: 1
> +
> + clocks:
> + maxItems: 1
> +
> + clock-names:
> + description:
> + Reference to the xclk input clock.
> + items:
> + - const: xclk
> +
> + VDDE-supply:
> + description:
> + Sensor digital IO supply. Must be 1.8 volts.
> +
> + VDDIN-supply:
> + description:
> + Sensor internal regulator supply. Must be 1.8 volts.
> +
> + reset-gpios:
> + description:
> + Reference to the GPIO connected to the xsdn pin, if any.
> + This is an active low signal to the mipid02.
> +
> + ports:
> + $ref: /schemas/graph.yaml#/properties/ports
> + properties:
> + port at 0:
> + $ref: /schemas/graph.yaml#/$defs/port-base
> + unevaluatedProperties: false
> + description: CSI-2 first input port
> + properties:
> + endpoint:
> + $ref: /schemas/media/video-interfaces.yaml#
> + unevaluatedProperties: false
> +
> + properties:
> + data-lanes:
> + description:
> + Single-lane operation shall be <1> or <2> .
> + Dual-lane operation shall be <1 2> or <2 1> .
> + minItems: 1
> + maxItems: 2
> + lane-polarity:
> + description:
> + Any lane can be inverted or not.
> + minItems: 1
> + maxItems: 2
> +
> + required:
> + - data-lanes
> +
> + port at 1:
> + $ref: /schemas/graph.yaml#/$defs/port-base
> + unevaluatedProperties: false
> + description: CSI-2 second input port
> + properties:
> + endpoint:
> + $ref: /schemas/media/video-interfaces.yaml#
> + unevaluatedProperties: false
> +
> + properties:
> + data-lanes:
> + description:
> + Single-lane operation shall be <1> or <2> .
> + maxItems: 1
> + lane-polarity:
> + description:
> + Any lane can be inverted or not.
> + maxItems: 1
> +
> + required:
> + - data-lanes
> +
> + port at 2:
> + $ref: /schemas/graph.yaml#/$defs/port-base
> + unevaluatedProperties: false
> + description: Output port
> + properties:
> + endpoint:
> + $ref: /schemas/media/video-interfaces.yaml#
> + unevaluatedProperties: false
> +
> + properties:
> + bus-width:
> + enum: [8, 10, 12, 14]
Shouldn't this be [6, 8, 10, 12] as in the original file ?
> + default: 8
As far as I understand there is no 'default' as this property is required anyway.
> + hsync-active: true
> + vsync-active: true
> +
> + required:
> + - bus-width
> +
> + required:
> + - port at 0
Actually it could either be port0 or port1, but not both at the same time, see the 'description' part.
I'm not sure how to describe it in device tree binding though.
> + - port at 2
> +
> +additionalProperties: false
> +
> +required:
> + - compatible
> + - reg
I noticed this was missing in the original txt file. Thanks for taking care of it.
Regards,
Benjamin
> + - clocks
> + - clock-names
> + - VDDE-supply
> + - VDDIN-supply
> + - ports
> +
> +examples:
> + - |
> + i2c {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + mipid02: csi2rx at 14 {
> + compatible = "st,st-mipid02";
> + reg = <0x14>;
> + status = "okay";
> + clocks = <&clk_ext_camera_12>;
> + clock-names = "xclk";
> + VDDE-supply = <&vdd>;
> + VDDIN-supply = <&vdd>;
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + port at 0 {
> + reg = <0>;
> +
> + ep0: endpoint {
> + data-lanes = <1 2>;
> + remote-endpoint = <&mipi_csi2_in>;
> + };
> + };
> + port at 2 {
> + reg = <2>;
> +
> + ep2: endpoint {
> + bus-width = <8>;
> + hsync-active = <0>;
> + vsync-active = <0>;
> + remote-endpoint = <¶llel_out>;
> + };
> + };
> + };
> + };
> + };
> +
> +...
More information about the linux-arm-kernel
mailing list