From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F2FD0C07E9D for ; Tue, 27 Sep 2022 11:10:14 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230214AbiI0LKN (ORCPT ); Tue, 27 Sep 2022 07:10:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48844 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230251AbiI0LJs (ORCPT ); Tue, 27 Sep 2022 07:09:48 -0400 Received: from mx07-00178001.pphosted.com (mx07-00178001.pphosted.com [185.132.182.106]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 50DB82A972; Tue, 27 Sep 2022 04:07:09 -0700 (PDT) Received: from pps.filterd (m0288072.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.17.1.5/8.17.1.5) with ESMTP id 28RAmGVl003667; Tue, 27 Sep 2022 13:06:46 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foss.st.com; h=message-id : date : mime-version : subject : to : cc : references : from : in-reply-to : content-type : content-transfer-encoding; s=selector1; bh=NVaUJpjHrQrEUjnpIi35k4EW+HMBmjlZbQ25GQ9xbOE=; b=UnC5jXoEiM3urBI4oruxT30/aEccUFGlUwQjCMIoJ5Ls5VvfeYw0IFp5sA5cfj6jVIa0 G8jy0EFK47upi0yv+z1c0/Mz5djjdXpYYFHbTRAb3m19tyyYCz1k7MqryOEyxCjw3PWf ET3ME85i+eeUuk704LPSrxTcTYzlrgJOqj3xEOd4ImX7S4IVILxSaBSeTQJ7IkP5D1ac QcIMYB7DNlqTYMhZXOiyp723Yiuupuyg2G7rm4ESVfCcc705ce4xmSb1NrkAJqbbT9+t EPCOplsLjqZmx1pRu9kN8x1YhdV02aWw8/phbcnQNhix32M+odtgctW++tlyk54m/1G4 rQ== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx07-00178001.pphosted.com (PPS) with ESMTPS id 3jsqam9brr-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 27 Sep 2022 13:06:46 +0200 Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 8B80E10002A; Tue, 27 Sep 2022 13:06:44 +0200 (CEST) Received: from Webmail-eu.st.com (shfdag1node1.st.com [10.75.129.69]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id 7C32621D384; Tue, 27 Sep 2022 13:06:44 +0200 (CEST) Received: from [10.252.29.213] (10.75.127.49) by SHFDAG1NODE1.st.com (10.75.129.69) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.2375.31; Tue, 27 Sep 2022 13:06:44 +0200 Message-ID: <7420b97c-1421-03a5-dd47-044b5e1739f1@foss.st.com> Date: Tue, 27 Sep 2022 13:06:43 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH] dt-bindings: media: st,stmipid02: Convert the text bindings to YAML Content-Language: en-US To: Marek Vasut , CC: Alexandre Torgue , Krzysztof Kozlowski , Mauro Carvalho Chehab , Maxime Coquelin , Rob Herring , Sylvain Petinot , , , References: <20220927005538.690997-1-marex@denx.de> From: Benjamin MUGNIER In-Reply-To: <20220927005538.690997-1-marex@denx.de> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.75.127.49] X-ClientProxiedBy: SFHDAG2NODE3.st.com (10.75.127.6) To SHFDAG1NODE1.st.com (10.75.129.69) X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.528,FMLib:17.11.122.1 definitions=2022-09-27_03,2022-09-22_02,2022-06-22_01 Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org 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 > --- > Cc: Alexandre Torgue > Cc: Benjamin Mugnier > Cc: Krzysztof Kozlowski > Cc: Mauro Carvalho Chehab > Cc: Maxime Coquelin > Cc: Rob Herring > Cc: Sylvain Petinot > Cc: devicetree@vger.kernel.org > Cc: linux-media@vger.kernel.org > Cc: linux-stm32@st-md-mailman.stormreply.com > To: linux-arm-kernel@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@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@0 { > - reg = <0>; > - > - ep0: endpoint { > - data-lanes = <1 2>; > - remote-endpoint = <&mipi_csi2_in>; > - }; > - }; > - port@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 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 - Sylvain Petinot 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 > + > +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@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@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@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@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@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@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@0 { > + reg = <0>; > + > + ep0: endpoint { > + data-lanes = <1 2>; > + remote-endpoint = <&mipi_csi2_in>; > + }; > + }; > + port@2 { > + reg = <2>; > + > + ep2: endpoint { > + bus-width = <8>; > + hsync-active = <0>; > + vsync-active = <0>; > + remote-endpoint = <¶llel_out>; > + }; > + }; > + }; > + }; > + }; > + > +...