| Message ID | 20230703113734.762307-1-guoniu.zhou@oss.nxp.com |
|---|---|
| Headers | show |
| Series | media: nxp: add i.MX93 MIPI CSI-2 support | expand |
Hi Guoniu, thanks for posting this driver. Am Montag, 3. Juli 2023, 13:37:33 CEST schrieb guoniu.zhou@oss.nxp.com: > ******************** > Achtung externe E-Mail: Öffnen Sie Anhänge und Links nur, wenn Sie wissen, > dass diese aus einer sicheren Quelle stammen und sicher sind. Leiten Sie > die E-Mail im Zweifelsfall zur Prüfung an den IT-Helpdesk weiter. > Attention external email: Open attachments and links only if you know that > they are from a secure source and are safe. In doubt forward the email to > the IT-Helpdesk to check it. ******************** > > From: "Guoniu.zhou" <guoniu.zhou@nxp.com> > > Add new binding documentation for DesignWare Core MIPI CSI-2 receiver > and DPHY found on NXP i.MX93. > > Signed-off-by: Guoniu.zhou <guoniu.zhou@nxp.com> > --- > .../bindings/media/nxp,dwc-mipi-csi2.yaml | 140 ++++++++++++++++++ > 1 file changed, 140 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml new file > mode 100644 > index 000000000000..ece6fb8991d4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > @@ -0,0 +1,140 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/media/nxp,dwc-mipi-csi2.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: NXP i.MX93 MIPI CSI-2 Host Controller receiver > + > +maintainers: > + - G.N. Zhou <guoniu.zhou@nxp.com> > + > +description: |- > + The MIPI CSI-2 receiver found on i.MX93 originates from Synopsys > + DesignWare Core and it implements the CSI-2 protocol on the host > + side and a DPHY configured as a Slave acts as the physical layer. > + Two data lanes are supported on i.MX93 family devices and the data > + rate of each lane support up to 1.5Gbps. > + > + While the CSI-2 receiver is separate from the MIPI D-PHY IP core, > + the PHY is completely wrapped by the CSI-2 controller and expose > + a control interface which only can communicate with CSI-2 controller > + This binding thus covers both IP cores. > + > +properties: > + compatible: > + enum: > + - fsl,imx93-mipi-csi2 > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + clocks: > + items: > + - description: The peripheral clock (a.k.a. APB clock) > + - description: The pixel clock > + - description: The MIPI D-PHY clock > + > + clock-names: > + items: > + - const: per > + - const: pixel > + - const: phy_cfg > + > + power-domains: > + maxItems: 1 > + > + ports: > + $ref: /schemas/graph.yaml#/properties/ports > + > + properties: > + port@0: > + $ref: /schemas/graph.yaml#/$defs/port-base > + unevaluatedProperties: false > + description: > + Input port node, single endpoint describing the CSI-2 > transmitter. + > + properties: > + endpoint: > + $ref: video-interfaces.yaml# > + unevaluatedProperties: false > + > + properties: > + data-lanes: > + minItems: 1 > + items: > + - const: 1 > + - const: 2 > + > + fsl,hsfreqrange: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: > + Used to select the desired high speed frequency range > + according to data lane bit rate. Please refer to i.MX93 > + reference manual MIPI CSI-2 DPHY chapter to get a valid > + value. If this is data lane bit rate specific, shouldn't it be set in s_stream callback or similar? Best regards, Alexander > + > + required: > + - data-lanes > + - fsl,hsfreqrange > + > + port@1: > + $ref: /schemas/graph.yaml#/properties/port > + description: > + Output port node > + > +required: > + - compatible > + - reg > + - interrupts > + - clocks > + - clock-names > + - power-domains > + - ports > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/clock/imx93-clock.h> > + #include <dt-bindings/interrupt-controller/arm-gic.h> > + #include <dt-bindings/interrupt-controller/irq.h> > + #include <dt-bindings/power/fsl,imx93-power.h> > + > + mipi-csi@4ae00000 { > + compatible = "fsl,imx93-mipi-csi2"; > + reg = <0x4ae00000 0x10000>; > + interrupts = <GIC_SPI 175 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&clks IMX93_CLK_MIPI_CSI_GATE>, > + <&clks IMX93_CLK_CAM_PIX>, > + <&clks IMX93_CLK_MIPI_PHY_CFG>; > + clock-names = "per", "pixel", "phy_cfg"; > + power-domains = <&media_blk_ctrl IMX93_MEDIABLK_PD_MIPI_CSI>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + mipi_from_sensor: endpoint { > + remote-endpoint = <&ap1302_to_mipi>; > + data-lanes = <1 2>; > + fsl,hsfreqrange = <0x2c>; > + }; > + }; > + > + port@1 { > + reg = <1>; > + > + mipi_to_isi: endpoint { > + remote-endpoint = <&isi_in>; > + }; > + }; > + }; > + }; > +...
Hey, I know little about media bindings, so only got a single comment for you. On Mon, Jul 03, 2023 at 07:37:33PM +0800, guoniu.zhou@oss.nxp.com wrote: > From: "Guoniu.zhou" <guoniu.zhou@nxp.com> > > Add new binding documentation for DesignWare Core MIPI CSI-2 receiver > and DPHY found on NXP i.MX93. > > Signed-off-by: Guoniu.zhou <guoniu.zhou@nxp.com> > --- > .../bindings/media/nxp,dwc-mipi-csi2.yaml | 140 ++++++++++++++++++ > 1 file changed, 140 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > new file mode 100644 > index 000000000000..ece6fb8991d4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml The filename of the binding should match the compatible. > @@ -0,0 +1,140 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/media/nxp,dwc-mipi-csi2.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: NXP i.MX93 MIPI CSI-2 Host Controller receiver > + > +maintainers: > + - G.N. Zhou <guoniu.zhou@nxp.com> > + > +description: |- > + The MIPI CSI-2 receiver found on i.MX93 originates from Synopsys > + DesignWare Core and it implements the CSI-2 protocol on the host > + side and a DPHY configured as a Slave acts as the physical layer. > + Two data lanes are supported on i.MX93 family devices and the data > + rate of each lane support up to 1.5Gbps. > + > + While the CSI-2 receiver is separate from the MIPI D-PHY IP core, > + the PHY is completely wrapped by the CSI-2 controller and expose > + a control interface which only can communicate with CSI-2 controller > + This binding thus covers both IP cores. > + > +properties: > + compatible: > + enum: > + - fsl,imx93-mipi-csi2 Everywhere else you say NXP, why use Freescale here? Cheers, Conor.
Hi Conor, Thanks for your comment. > -----Original Message----- > From: Conor Dooley <conor@kernel.org> > Sent: 2023年7月5日 0:54 > To: G.N. Zhou (OSS) <guoniu.zhou@oss.nxp.com> > Cc: linux-media@vger.kernel.org; devicetree@vger.kernel.org; dl-linux-imx > <linux-imx@nxp.com>; mchehab@kernel.org; > laurent.pinchart@ideasonboard.com; robh+dt@kernel.org; > krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; > jacopo.mondi@ideasonboard.com > Subject: Re: [PATCH 1/2] media: dt-bindings: Add binding doc for i.MX93 MIPI > CSI-2 > > Hey, > > I know little about media bindings, so only got a single comment for you. > > On Mon, Jul 03, 2023 at 07:37:33PM +0800, guoniu.zhou@oss.nxp.com wrote: > > From: "Guoniu.zhou" <guoniu.zhou@nxp.com> > > > > Add new binding documentation for DesignWare Core MIPI CSI-2 receiver > > and DPHY found on NXP i.MX93. > > > > Signed-off-by: Guoniu.zhou <guoniu.zhou@nxp.com> > > --- > > .../bindings/media/nxp,dwc-mipi-csi2.yaml | 140 > ++++++++++++++++++ > > 1 file changed, 140 insertions(+) > > > > diff --git > > a/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > new file mode 100644 > > index 000000000000..ece6fb8991d4 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > The filename of the binding should match the compatible. > > > @@ -0,0 +1,140 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/media/nxp,dwc-mipi-csi2.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: NXP i.MX93 MIPI CSI-2 Host Controller receiver > > + > > +maintainers: > > + - G.N. Zhou <guoniu.zhou@nxp.com> > > + > > +description: |- > > + The MIPI CSI-2 receiver found on i.MX93 originates from Synopsys > > + DesignWare Core and it implements the CSI-2 protocol on the host > > + side and a DPHY configured as a Slave acts as the physical layer. > > + Two data lanes are supported on i.MX93 family devices and the data > > + rate of each lane support up to 1.5Gbps. > > + > > + While the CSI-2 receiver is separate from the MIPI D-PHY IP core, > > + the PHY is completely wrapped by the CSI-2 controller and expose a > > + control interface which only can communicate with CSI-2 controller > > + This binding thus covers both IP cores. > > + > > +properties: > > + compatible: > > + enum: > > + - fsl,imx93-mipi-csi2 > > Everywhere else you say NXP, why use Freescale here? Due to history reason, all i.MX platforms of NXP use "fsl" as vendor abbreviation prefix. > > Cheers, > Conor.
On Wed, Jul 05, 2023 at 01:30:38AM +0000, G.N. Zhou (OSS) wrote: > > > diff --git > > > a/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > > b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > > new file mode 100644 > > > index 000000000000..ece6fb8991d4 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > > > The filename of the binding should match the compatible. > > > > > @@ -0,0 +1,140 @@ > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/media/nxp,dwc-mipi-csi2.yaml# > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > + > > > +title: NXP i.MX93 MIPI CSI-2 Host Controller receiver > > > + > > > +maintainers: > > > + - G.N. Zhou <guoniu.zhou@nxp.com> > > > + > > > +description: |- > > > + The MIPI CSI-2 receiver found on i.MX93 originates from Synopsys > > > + DesignWare Core and it implements the CSI-2 protocol on the host > > > + side and a DPHY configured as a Slave acts as the physical layer. > > > + Two data lanes are supported on i.MX93 family devices and the data > > > + rate of each lane support up to 1.5Gbps. > > > + > > > + While the CSI-2 receiver is separate from the MIPI D-PHY IP core, > > > + the PHY is completely wrapped by the CSI-2 controller and expose a > > > + control interface which only can communicate with CSI-2 controller > > > + This binding thus covers both IP cores. > > > + > > > +properties: > > > + compatible: > > > + enum: > > > + - fsl,imx93-mipi-csi2 > > > > Everywhere else you say NXP, why use Freescale here? > > Due to history reason, all i.MX platforms of NXP use "fsl" as vendor abbreviation prefix. Okay. Please update the filename to patch the "fsl" compatible then. Cheers, Conor.
On Wed, Jul 05, 2023 at 01:36:46AM +0000, G.N. Zhou (OSS) wrote: > Hi Alexander, > > Thanks for you comment. > > > -----Original Message----- > > From: Alexander Stein <alexander.stein@ew.tq-group.com> > > Sent: 2023年7月4日 16:39 > > To: linux-media@vger.kernel.org; devicetree@vger.kernel.org; dl-linux-imx > > <linux-imx@nxp.com>; G.N. Zhou (OSS) <guoniu.zhou@oss.nxp.com> > > Cc: mchehab@kernel.org; laurent.pinchart@ideasonboard.com; > > robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; > > jacopo.mondi@ideasonboard.com > > Subject: Re: [PATCH 1/2] media: dt-bindings: Add binding doc for i.MX93 MIPI > > CSI-2 > > > > Hi Guoniu, > > > > thanks for posting this driver. > > > > Am Montag, 3. Juli 2023, 13:37:33 CEST schrieb guoniu.zhou@oss.nxp.com: > > > > > > From: "Guoniu.zhou" <guoniu.zhou@nxp.com> > > > > > > Add new binding documentation for DesignWare Core MIPI CSI-2 receiver > > > and DPHY found on NXP i.MX93. > > > > > > Signed-off-by: Guoniu.zhou <guoniu.zhou@nxp.com> > > > --- > > > .../bindings/media/nxp,dwc-mipi-csi2.yaml | 140 ++++++++++++++++++ > > > 1 file changed, 140 insertions(+) > > > > > > diff --git > > > a/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > > b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml new > > > file mode 100644 index 000000000000..ece6fb8991d4 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > > @@ -0,0 +1,140 @@ > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/media/nxp,dwc-mipi-csi2.yaml# > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > + > > > +title: NXP i.MX93 MIPI CSI-2 Host Controller receiver > > > + > > > +maintainers: > > > + - G.N. Zhou <guoniu.zhou@nxp.com> > > > + > > > +description: |- > > > + The MIPI CSI-2 receiver found on i.MX93 originates from Synopsys > > > + DesignWare Core and it implements the CSI-2 protocol on the host > > > + side and a DPHY configured as a Slave acts as the physical layer. > > > + Two data lanes are supported on i.MX93 family devices and the data > > > + rate of each lane support up to 1.5Gbps. > > > + > > > + While the CSI-2 receiver is separate from the MIPI D-PHY IP core, > > > + the PHY is completely wrapped by the CSI-2 controller and expose a > > > + control interface which only can communicate with CSI-2 controller > > > + This binding thus covers both IP cores. > > > + > > > +properties: > > > + compatible: > > > + enum: > > > + - fsl,imx93-mipi-csi2 > > > + > > > + reg: > > > + maxItems: 1 > > > + > > > + interrupts: > > > + maxItems: 1 > > > + > > > + clocks: > > > + items: > > > + - description: The peripheral clock (a.k.a. APB clock) > > > + - description: The pixel clock > > > + - description: The MIPI D-PHY clock > > > + > > > + clock-names: > > > + items: > > > + - const: per > > > + - const: pixel > > > + - const: phy_cfg > > > + > > > + power-domains: > > > + maxItems: 1 > > > + > > > + ports: > > > + $ref: /schemas/graph.yaml#/properties/ports > > > + > > > + properties: > > > + port@0: > > > + $ref: /schemas/graph.yaml#/$defs/port-base > > > + unevaluatedProperties: false > > > + description: > > > + Input port node, single endpoint describing the CSI-2 transmitter. > > > + > > > + properties: > > > + endpoint: > > > + $ref: video-interfaces.yaml# > > > + unevaluatedProperties: false > > > + > > > + properties: > > > + data-lanes: > > > + minItems: 1 > > > + items: > > > + - const: 1 > > > + - const: 2 > > > + > > > + fsl,hsfreqrange: > > > + $ref: /schemas/types.yaml#/definitions/uint32 > > > + description: > > > + Used to select the desired high speed frequency range > > > + according to data lane bit rate. Please refer to i.MX93 > > > + reference manual MIPI CSI-2 DPHY chapter to get a valid > > > + value. > > > > If this is data lane bit rate specific, shouldn't it be set in s_stream callback or > > similar? > > That's correct if we have a formula to calculate it and get data rate from sensor. But Synopsys only > provide a table to search the valid hsfreqrange according to data rate and the values are nonlinear > so I export a property to handle this issue. We have multiple drivers in mainline that do the same, so it's not a problem, you can have a table of values in the driver and search for the right entry at runtime. See drivers/media/platform/renesas/rcar-vin/rcar-csi2.c for instance. > > > + > > > + required: > > > + - data-lanes > > > + - fsl,hsfreqrange > > > + > > > + port@1: > > > + $ref: /schemas/graph.yaml#/properties/port > > > + description: > > > + Output port node > > > + > > > +required: > > > + - compatible > > > + - reg > > > + - interrupts > > > + - clocks > > > + - clock-names > > > + - power-domains > > > + - ports > > > + > > > +additionalProperties: false > > > + > > > +examples: > > > + - | > > > + #include <dt-bindings/clock/imx93-clock.h> > > > + #include <dt-bindings/interrupt-controller/arm-gic.h> > > > + #include <dt-bindings/interrupt-controller/irq.h> > > > + #include <dt-bindings/power/fsl,imx93-power.h> > > > + > > > + mipi-csi@4ae00000 { > > > + compatible = "fsl,imx93-mipi-csi2"; > > > + reg = <0x4ae00000 0x10000>; > > > + interrupts = <GIC_SPI 175 IRQ_TYPE_LEVEL_HIGH>; > > > + clocks = <&clks IMX93_CLK_MIPI_CSI_GATE>, > > > + <&clks IMX93_CLK_CAM_PIX>, > > > + <&clks IMX93_CLK_MIPI_PHY_CFG>; > > > + clock-names = "per", "pixel", "phy_cfg"; > > > + power-domains = <&media_blk_ctrl IMX93_MEDIABLK_PD_MIPI_CSI>; > > > + > > > + ports { > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + > > > + port@0 { > > > + reg = <0>; > > > + > > > + mipi_from_sensor: endpoint { > > > + remote-endpoint = <&ap1302_to_mipi>; > > > + data-lanes = <1 2>; > > > + fsl,hsfreqrange = <0x2c>; > > > + }; > > > + }; > > > + > > > + port@1 { > > > + reg = <1>; > > > + > > > + mipi_to_isi: endpoint { > > > + remote-endpoint = <&isi_in>; > > > + }; > > > + }; > > > + }; > > > + }; > > > +...
Hi Laurent, Thanks for your comments. > -----Original Message----- > From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Sent: 2023年7月6日 5:24 > To: G.N. Zhou (OSS) <guoniu.zhou@oss.nxp.com> > Cc: Alexander Stein <alexander.stein@ew.tq-group.com>; > linux-media@vger.kernel.org; devicetree@vger.kernel.org; dl-linux-imx > <linux-imx@nxp.com>; mchehab@kernel.org; robh+dt@kernel.org; > krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; > jacopo.mondi@ideasonboard.com > Subject: Re: [PATCH 1/2] media: dt-bindings: Add binding doc for i.MX93 MIPI > CSI-2 > > Caution: This is an external email. Please take care when clicking links or opening > attachments. When in doubt, report the message using the 'Report this email' > button > > > On Wed, Jul 05, 2023 at 01:36:46AM +0000, G.N. Zhou (OSS) wrote: > > Hi Alexander, > > > > Thanks for you comment. > > > > > -----Original Message----- > > > From: Alexander Stein <alexander.stein@ew.tq-group.com> > > > Sent: 2023年7月4日 16:39 > > > To: linux-media@vger.kernel.org; devicetree@vger.kernel.org; > > > dl-linux-imx <linux-imx@nxp.com>; G.N. Zhou (OSS) > > > <guoniu.zhou@oss.nxp.com> > > > Cc: mchehab@kernel.org; laurent.pinchart@ideasonboard.com; > > > robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; > > > robh+conor+dt@kernel.org; > > > jacopo.mondi@ideasonboard.com > > > Subject: Re: [PATCH 1/2] media: dt-bindings: Add binding doc for > > > i.MX93 MIPI > > > CSI-2 > > > > > > Hi Guoniu, > > > > > > thanks for posting this driver. > > > > > > Am Montag, 3. Juli 2023, 13:37:33 CEST schrieb guoniu.zhou@oss.nxp.com: > > > > > > > > From: "Guoniu.zhou" <guoniu.zhou@nxp.com> > > > > > > > > Add new binding documentation for DesignWare Core MIPI CSI-2 > > > > receiver and DPHY found on NXP i.MX93. > > > > > > > > Signed-off-by: Guoniu.zhou <guoniu.zhou@nxp.com> > > > > --- > > > > .../bindings/media/nxp,dwc-mipi-csi2.yaml | 140 > ++++++++++++++++++ > > > > 1 file changed, 140 insertions(+) > > > > > > > > diff --git > > > > a/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > > > b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml > > > > new file mode 100644 index 000000000000..ece6fb8991d4 > > > > --- /dev/null > > > > +++ b/Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.ya > > > > +++ ml > > > > @@ -0,0 +1,140 @@ > > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML > > > > +1.2 > > > > +--- > > > > +$id: http://devicetree.org/schemas/media/nxp,dwc-mipi-csi2.yaml# > > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > > + > > > > +title: NXP i.MX93 MIPI CSI-2 Host Controller receiver > > > > + > > > > +maintainers: > > > > + - G.N. Zhou <guoniu.zhou@nxp.com> > > > > + > > > > +description: |- > > > > + The MIPI CSI-2 receiver found on i.MX93 originates from > > > > +Synopsys > > > > + DesignWare Core and it implements the CSI-2 protocol on the > > > > +host > > > > + side and a DPHY configured as a Slave acts as the physical layer. > > > > + Two data lanes are supported on i.MX93 family devices and the > > > > +data > > > > + rate of each lane support up to 1.5Gbps. > > > > + > > > > + While the CSI-2 receiver is separate from the MIPI D-PHY IP > > > > + core, the PHY is completely wrapped by the CSI-2 controller and > > > > + expose a control interface which only can communicate with > > > > + CSI-2 controller This binding thus covers both IP cores. > > > > + > > > > +properties: > > > > + compatible: > > > > + enum: > > > > + - fsl,imx93-mipi-csi2 > > > > + > > > > + reg: > > > > + maxItems: 1 > > > > + > > > > + interrupts: > > > > + maxItems: 1 > > > > + > > > > + clocks: > > > > + items: > > > > + - description: The peripheral clock (a.k.a. APB clock) > > > > + - description: The pixel clock > > > > + - description: The MIPI D-PHY clock > > > > + > > > > + clock-names: > > > > + items: > > > > + - const: per > > > > + - const: pixel > > > > + - const: phy_cfg > > > > + > > > > + power-domains: > > > > + maxItems: 1 > > > > + > > > > + ports: > > > > + $ref: /schemas/graph.yaml#/properties/ports > > > > + > > > > + properties: > > > > + port@0: > > > > + $ref: /schemas/graph.yaml#/$defs/port-base > > > > + unevaluatedProperties: false > > > > + description: > > > > + Input port node, single endpoint describing the CSI-2 > transmitter. > > > > + > > > > + properties: > > > > + endpoint: > > > > + $ref: video-interfaces.yaml# > > > > + unevaluatedProperties: false > > > > + > > > > + properties: > > > > + data-lanes: > > > > + minItems: 1 > > > > + items: > > > > + - const: 1 > > > > + - const: 2 > > > > + > > > > + fsl,hsfreqrange: > > > > + $ref: /schemas/types.yaml#/definitions/uint32 > > > > + description: > > > > + Used to select the desired high speed frequency > range > > > > + according to data lane bit rate. Please refer to > i.MX93 > > > > + reference manual MIPI CSI-2 DPHY chapter to get a > valid > > > > + value. > > > > > > If this is data lane bit rate specific, shouldn't it be set in > > > s_stream callback or similar? > > > > That's correct if we have a formula to calculate it and get data rate > > from sensor. But Synopsys only provide a table to search the valid > > hsfreqrange according to data rate and the values are nonlinear so I export a > property to handle this issue. > > We have multiple drivers in mainline that do the same, so it's not a problem, you > can have a table of values in the driver and search for the right entry at runtime. > See drivers/media/platform/renesas/rcar-vin/rcar-csi2.c for instance. Okay, I will update in next version. Thanks for your suggestion. > > > > > + > > > > + required: > > > > + - data-lanes > > > > + - fsl,hsfreqrange > > > > + > > > > + port@1: > > > > + $ref: /schemas/graph.yaml#/properties/port > > > > + description: > > > > + Output port node > > > > + > > > > +required: > > > > + - compatible > > > > + - reg > > > > + - interrupts > > > > + - clocks > > > > + - clock-names > > > > + - power-domains > > > > + - ports > > > > + > > > > +additionalProperties: false > > > > + > > > > +examples: > > > > + - | > > > > + #include <dt-bindings/clock/imx93-clock.h> > > > > + #include <dt-bindings/interrupt-controller/arm-gic.h> > > > > + #include <dt-bindings/interrupt-controller/irq.h> > > > > + #include <dt-bindings/power/fsl,imx93-power.h> > > > > + > > > > + mipi-csi@4ae00000 { > > > > + compatible = "fsl,imx93-mipi-csi2"; > > > > + reg = <0x4ae00000 0x10000>; > > > > + interrupts = <GIC_SPI 175 IRQ_TYPE_LEVEL_HIGH>; > > > > + clocks = <&clks IMX93_CLK_MIPI_CSI_GATE>, > > > > + <&clks IMX93_CLK_CAM_PIX>, > > > > + <&clks IMX93_CLK_MIPI_PHY_CFG>; > > > > + clock-names = "per", "pixel", "phy_cfg"; > > > > + power-domains = <&media_blk_ctrl > > > > + IMX93_MEDIABLK_PD_MIPI_CSI>; > > > > + > > > > + ports { > > > > + #address-cells = <1>; > > > > + #size-cells = <0>; > > > > + > > > > + port@0 { > > > > + reg = <0>; > > > > + > > > > + mipi_from_sensor: endpoint { > > > > + remote-endpoint = <&ap1302_to_mipi>; > > > > + data-lanes = <1 2>; > > > > + fsl,hsfreqrange = <0x2c>; > > > > + }; > > > > + }; > > > > + > > > > + port@1 { > > > > + reg = <1>; > > > > + > > > > + mipi_to_isi: endpoint { > > > > + remote-endpoint = <&isi_in>; > > > > + }; > > > > + }; > > > > + }; > > > > + }; > > > > +... > > -- > Regards, > > Laurent Pinchart
From: "Guoniu.zhou" <guoniu.zhou@nxp.com> Add MIPI CSI-2 and D-PHY driver support for NXP i.MX93. v4l2-compliance 1.23.0-4996, 64bits, 64-bit time_t v4l2-compliance SHA: 9431e4b26b48 2023-02-13 14:51:47 Compliance test for device /dev/v4l-subdev2: Driver Info: Driver version : 6.4.0 Capabilities : 0x00000000 Required ioctls: test VIDIOC_SUDBEV_QUERYCAP: OK test invalid ioctls: OK Allow for multiple opens: test second /dev/v4l-subdev2 open: OK test VIDIOC_SUBDEV_QUERYCAP: OK test for unlimited opens: OK Debug ioctls: test VIDIOC_LOG_STATUS: OK (Not Supported) Input ioctls: test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported) test VIDIOC_G/S_FREQUENCY: OK (Not Supported) test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported) test VIDIOC_ENUMAUDIO: OK (Not Supported) test VIDIOC_G/S/ENUMINPUT: OK (Not Supported) test VIDIOC_G/S_AUDIO: OK (Not Supported) Inputs: 0 Audio Inputs: 0 Tuners: 0 Output ioctls: test VIDIOC_G/S_MODULATOR: OK (Not Supported) test VIDIOC_G/S_FREQUENCY: OK (Not Supported) test VIDIOC_ENUMAUDOUT: OK (Not Supported) test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported) test VIDIOC_G/S_AUDOUT: OK (Not Supported) Outputs: 0 Audio Outputs: 0 Modulators: 0 Input/Output configuration ioctls: test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported) test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported) test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported) test VIDIOC_G/S_EDID: OK (Not Supported) Control ioctls: test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK (Not Supported) test VIDIOC_QUERYCTRL: OK (Not Supported) test VIDIOC_G/S_CTRL: OK (Not Supported) test VIDIOC_G/S/TRY_EXT_CTRLS: OK (Not Supported) test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK (Not Supported) test VIDIOC_G/S_JPEGCOMP: OK (Not Supported) Standard Controls: 0 Private Controls: 0 Format ioctls: test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK (Not Supported) test VIDIOC_G/S_PARM: OK (Not Supported) test VIDIOC_G_FBUF: OK (Not Supported) test VIDIOC_G_FMT: OK (Not Supported) test VIDIOC_TRY_FMT: OK (Not Supported) test VIDIOC_S_FMT: OK (Not Supported) test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported) test Cropping: OK (Not Supported) test Composing: OK (Not Supported) test Scaling: OK (Not Supported) Codec ioctls: test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported) test VIDIOC_G_ENC_INDEX: OK (Not Supported) test VIDIOC_(TRY_)DECODER_CMD: OK (Not Supported) Buffer ioctls: test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK (Not Supported) test VIDIOC_EXPBUF: OK (Not Supported) test Requests: OK (Not Supported) Total for device /dev/v4l-subdev2: 43, Succeeded: 43, Failed: 0, Warnings: 0 Guoniu.zhou (2): media: dt-bindings: Add binding doc for i.MX93 MIPI CSI-2 media: nxp: add driver for i.MX93 MIPI CSI-2 controller and D-PHY .../bindings/media/nxp,dwc-mipi-csi2.yaml | 140 ++ MAINTAINERS | 10 + drivers/media/platform/nxp/Kconfig | 11 + drivers/media/platform/nxp/Makefile | 3 + drivers/media/platform/nxp/dwc-mipi-csi2.c | 1384 +++++++++++++++++ drivers/media/platform/nxp/dwc-mipi-csi2.h | 289 ++++ drivers/media/platform/nxp/dwc-mipi-dphy.c | 195 +++ 7 files changed, 2032 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/nxp,dwc-mipi-csi2.yaml create mode 100644 drivers/media/platform/nxp/dwc-mipi-csi2.c create mode 100644 drivers/media/platform/nxp/dwc-mipi-csi2.h create mode 100644 drivers/media/platform/nxp/dwc-mipi-dphy.c