From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 24 Apr 2018 08:18:03 -0500 From: Benoit Parrot Subject: Re: [PATCH v11 3/4] dt-bindings: media: Add Cadence MIPI-CSI2 TX Device Tree bindings Message-ID: <20180424131802.GH3629@ti.com> References: <20180424122700.5387-1-maxime.ripard@bootlin.com> <20180424122700.5387-4-maxime.ripard@bootlin.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20180424122700.5387-4-maxime.ripard@bootlin.com> To: Maxime Ripard Cc: Mauro Carvalho Chehab , Mark Rutland , Rob Herring , Frank Rowand , Hans Verkuil , Laurent Pinchart , linux-media@vger.kernel.org, devicetree@vger.kernel.org, Richard Sproul , Alan Douglas , Steve Creaney , Thomas Petazzoni , Boris Brezillon , Niklas =?iso-8859-1?Q?S=F6derlund?= , Sakari Ailus , nm@ti.com, Simon Hatliff List-ID: Acked-by: Benoit Parrot Maxime Ripard wrote on Tue [2018-Apr-24 14:26:59 +0200]: > The Cadence MIPI-CSI2 TX controller is a CSI2 bridge that supports up to 4 > video streams and can output on up to 4 CSI-2 lanes, depending on the > hardware implementation. > > It can operate with an external D-PHY, an internal one or no D-PHY at all > in some configurations. > > Acked-by: Rob Herring > Acked-by: Sakari Ailus > Reviewed-by: Niklas S�derlund > Signed-off-by: Maxime Ripard > --- > .../devicetree/bindings/media/cdns,csi2tx.txt | 98 +++++++++++++++++++ > 1 file changed, 98 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/cdns,csi2tx.txt > > diff --git a/Documentation/devicetree/bindings/media/cdns,csi2tx.txt b/Documentation/devicetree/bindings/media/cdns,csi2tx.txt > new file mode 100644 > index 000000000000..459c6e332f52 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/cdns,csi2tx.txt > @@ -0,0 +1,98 @@ > +Cadence MIPI-CSI2 TX controller > +=============================== > + > +The Cadence MIPI-CSI2 TX controller is a CSI-2 bridge supporting up to > +4 CSI lanes in output, and up to 4 different pixel streams in input. > + > +Required properties: > + - compatible: must be set to "cdns,csi2tx" > + - reg: base address and size of the memory mapped region > + - clocks: phandles to the clocks driving the controller > + - clock-names: must contain: > + * esc_clk: escape mode clock > + * p_clk: register bank clock > + * pixel_if[0-3]_clk: pixel stream output clock, one for each stream > + implemented in hardware, between 0 and 3 > + > +Optional properties > + - phys: phandle to the D-PHY. If it is set, phy-names need to be set > + - phy-names: must contain "dphy" > + > +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 output > + 1 Stream 0 input > + 2 Stream 1 input > + 3 Stream 2 input > + 4 Stream 3 input > + > + The stream input port nodes are optional if they are not > + connected to anything at the hardware level or implemented > + in the design. Since there is only one endpoint per port, > + the endpoints are not numbered. > + > +Example: > + > +csi2tx: csi-bridge@0d0e1000 { > + compatible = "cdns,csi2tx"; > + reg = <0x0d0e1000 0x1000>; > + clocks = <&byteclock>, <&byteclock>, > + <&coreclock>, <&coreclock>, > + <&coreclock>, <&coreclock>; > + clock-names = "p_clk", "esc_clk", > + "pixel_if0_clk", "pixel_if1_clk", > + "pixel_if2_clk", "pixel_if3_clk"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + csi2tx_out: endpoint { > + remote-endpoint = <&remote_in>; > + clock-lanes = <0>; > + data-lanes = <1 2>; > + }; > + }; > + > + port@1 { > + reg = <1>; > + > + csi2tx_in_stream0: endpoint { > + remote-endpoint = <&stream0_out>; > + }; > + }; > + > + port@2 { > + reg = <2>; > + > + csi2tx_in_stream1: endpoint { > + remote-endpoint = <&stream1_out>; > + }; > + }; > + > + port@3 { > + reg = <3>; > + > + csi2tx_in_stream2: endpoint { > + remote-endpoint = <&stream2_out>; > + }; > + }; > + > + port@4 { > + reg = <4>; > + > + csi2tx_in_stream3: endpoint { > + remote-endpoint = <&stream3_out>; > + }; > + }; > + }; > +}; > -- > 2.17.0 > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from fllnx210.ext.ti.com ([198.47.19.17]:11936 "EHLO fllnx210.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933839AbeDXNSU (ORCPT ); Tue, 24 Apr 2018 09:18:20 -0400 Date: Tue, 24 Apr 2018 08:18:03 -0500 From: Benoit Parrot To: Maxime Ripard CC: Mauro Carvalho Chehab , Mark Rutland , Rob Herring , Frank Rowand , Hans Verkuil , Laurent Pinchart , , , Richard Sproul , Alan Douglas , Steve Creaney , Thomas Petazzoni , Boris Brezillon , Niklas =?iso-8859-1?Q?S=F6derlund?= , Sakari Ailus , , Simon Hatliff Subject: Re: [PATCH v11 3/4] dt-bindings: media: Add Cadence MIPI-CSI2 TX Device Tree bindings Message-ID: <20180424131802.GH3629@ti.com> References: <20180424122700.5387-1-maxime.ripard@bootlin.com> <20180424122700.5387-4-maxime.ripard@bootlin.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20180424122700.5387-4-maxime.ripard@bootlin.com> Sender: linux-media-owner@vger.kernel.org List-ID: Acked-by: Benoit Parrot Maxime Ripard wrote on Tue [2018-Apr-24 14:26:59 +0200]: > The Cadence MIPI-CSI2 TX controller is a CSI2 bridge that supports up to 4 > video streams and can output on up to 4 CSI-2 lanes, depending on the > hardware implementation. > > It can operate with an external D-PHY, an internal one or no D-PHY at all > in some configurations. > > Acked-by: Rob Herring > Acked-by: Sakari Ailus > Reviewed-by: Niklas Söderlund > Signed-off-by: Maxime Ripard > --- > .../devicetree/bindings/media/cdns,csi2tx.txt | 98 +++++++++++++++++++ > 1 file changed, 98 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/cdns,csi2tx.txt > > diff --git a/Documentation/devicetree/bindings/media/cdns,csi2tx.txt b/Documentation/devicetree/bindings/media/cdns,csi2tx.txt > new file mode 100644 > index 000000000000..459c6e332f52 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/cdns,csi2tx.txt > @@ -0,0 +1,98 @@ > +Cadence MIPI-CSI2 TX controller > +=============================== > + > +The Cadence MIPI-CSI2 TX controller is a CSI-2 bridge supporting up to > +4 CSI lanes in output, and up to 4 different pixel streams in input. > + > +Required properties: > + - compatible: must be set to "cdns,csi2tx" > + - reg: base address and size of the memory mapped region > + - clocks: phandles to the clocks driving the controller > + - clock-names: must contain: > + * esc_clk: escape mode clock > + * p_clk: register bank clock > + * pixel_if[0-3]_clk: pixel stream output clock, one for each stream > + implemented in hardware, between 0 and 3 > + > +Optional properties > + - phys: phandle to the D-PHY. If it is set, phy-names need to be set > + - phy-names: must contain "dphy" > + > +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 output > + 1 Stream 0 input > + 2 Stream 1 input > + 3 Stream 2 input > + 4 Stream 3 input > + > + The stream input port nodes are optional if they are not > + connected to anything at the hardware level or implemented > + in the design. Since there is only one endpoint per port, > + the endpoints are not numbered. > + > +Example: > + > +csi2tx: csi-bridge@0d0e1000 { > + compatible = "cdns,csi2tx"; > + reg = <0x0d0e1000 0x1000>; > + clocks = <&byteclock>, <&byteclock>, > + <&coreclock>, <&coreclock>, > + <&coreclock>, <&coreclock>; > + clock-names = "p_clk", "esc_clk", > + "pixel_if0_clk", "pixel_if1_clk", > + "pixel_if2_clk", "pixel_if3_clk"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + csi2tx_out: endpoint { > + remote-endpoint = <&remote_in>; > + clock-lanes = <0>; > + data-lanes = <1 2>; > + }; > + }; > + > + port@1 { > + reg = <1>; > + > + csi2tx_in_stream0: endpoint { > + remote-endpoint = <&stream0_out>; > + }; > + }; > + > + port@2 { > + reg = <2>; > + > + csi2tx_in_stream1: endpoint { > + remote-endpoint = <&stream1_out>; > + }; > + }; > + > + port@3 { > + reg = <3>; > + > + csi2tx_in_stream2: endpoint { > + remote-endpoint = <&stream2_out>; > + }; > + }; > + > + port@4 { > + reg = <4>; > + > + csi2tx_in_stream3: endpoint { > + remote-endpoint = <&stream3_out>; > + }; > + }; > + }; > +}; > -- > 2.17.0 >