[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251105172703.GC6046@pendragon.ideasonboard.com>
Date: Wed, 5 Nov 2025 19:27:03 +0200
From: Laurent Pinchart <laurent.pinchart@...asonboard.com>
To: Guoniu Zhou <guoniu.zhou@....nxp.com>
Cc: Mauro Carvalho Chehab <mchehab@...nel.org>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>, Frank Li <Frank.Li@....com>,
linux-media@...r.kernel.org, devicetree@...r.kernel.org,
imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org, Guoniu Zhou <guoniu.zhou@....com>
Subject: Re: [PATCH v3 1/3] media: dt-bindings: nxp,imx8-isi: Add i.MX95 ISI
compatible string
Hi Guoniu,
Thank you for the patch.
On Wed, Nov 05, 2025 at 01:55:10PM +0800, Guoniu Zhou wrote:
> From: Guoniu Zhou <guoniu.zhou@....com>
>
> The ISI module on i.MX95 supports up to eight channels and four link
> sources to obtain the image data for processing in its pipelines. It
> can process up to eight image sources at the same time.
>
> Signed-off-by: Guoniu Zhou <guoniu.zhou@....com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com>
> ---
> .../devicetree/bindings/media/nxp,imx8-isi.yaml | 27 +++++++++++++++++++++-
> 1 file changed, 26 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml b/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
> index f43b91984f0152fbbcf80db3b3bbad7e8ad6c11e..c5cf4a9787ee7f31273bd097d2200402fb346773 100644
> --- a/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
> +++ b/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
> @@ -23,6 +23,7 @@ properties:
> - fsl,imx8mp-isi
> - fsl,imx8ulp-isi
> - fsl,imx93-isi
> + - fsl,imx95-isi
>
> reg:
> maxItems: 1
> @@ -49,7 +50,7 @@ properties:
> interrupts:
> description: Processing pipeline interrupts, one per pipeline
> minItems: 1
> - maxItems: 2
> + maxItems: 8
>
> power-domains:
> maxItems: 1
> @@ -98,6 +99,7 @@ allOf:
> then:
> properties:
> interrupts:
> + minItems: 2
> maxItems: 2
> ports:
> properties:
> @@ -109,6 +111,29 @@ allOf:
> - port@0
> - port@1
>
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: fsl,imx95-isi
> + then:
> + properties:
> + interrupts:
> + minItems: 8
> + ports:
> + properties:
> + port@0:
> + description: Pixel Link Slave 0
> + port@1:
> + description: Pixel Link Slave 1
> + port@2:
> + description: MIPI CSI-2 RX 0
> + port@3:
> + description: MIPI CSI-2 RX 1
> + required:
> + - port@2
> + - port@3
> +
> additionalProperties: false
>
> examples:
--
Regards,
Laurent Pinchart
Powered by blists - more mailing lists