2024-04-12 09:57:30

by Alexander Stein

Subject: [PATCH v2 1/1] media: dt-bindings: nxp,imx8-isi: Refuse port@1 for single pipeline models

In case the hardware only supports just one pipeline, explicitly refuse
port@1 in ports node.

Signed-off-by: Alexander Stein <[email protected]>
Acked-by: Krzysztof Kozlowski <[email protected]>
---
This is v2 to patch series [1]. With 2 patches dropped, only this one
is left, hence no cover letter.

Changes in v2:
* Dropped v1 patches 2 & 3
* Collected Krzysztof's A-b

[1] https://lore.kernel.org/all/[email protected]/

Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml b/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
index e4665469a86c..4d5348d456a1 100644
--- a/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
+++ b/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
@@ -84,6 +84,7 @@ allOf:
properties:
port@0:
description: MIPI CSI-2 RX
+ port@1: false
required:
- port@0

--
2.34.1

2024-04-12 14:25:55

by Laurent Pinchart

Subject: Re: [PATCH v2 1/1] media: dt-bindings: nxp,imx8-isi: Refuse port@1 for single pipeline models

Hi Alexander,

Thank you for the patch.

On Fri, Apr 12, 2024 at 11:55:49AM +0200, Alexander Stein wrote:
> In case the hardware only supports just one pipeline, explicitly refuse
> port@1 in ports node.
>
> Signed-off-by: Alexander Stein <[email protected]>
> Acked-by: Krzysztof Kozlowski <[email protected]>

Reviewed-by: Laurent Pinchart <[email protected]>

> ---
> This is v2 to patch series [1]. With 2 patches dropped, only this one
> is left, hence no cover letter.
>
> Changes in v2:
> * Dropped v1 patches 2 & 3
> * Collected Krzysztof's A-b
>
> [1] https://lore.kernel.org/all/[email protected]/
>
> Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml b/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
> index e4665469a86c..4d5348d456a1 100644
> --- a/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
> +++ b/Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
> @@ -84,6 +84,7 @@ allOf:
> properties:
> port@0:
> description: MIPI CSI-2 RX
> + port@1: false
> required:
> - port@0
>

--
Regards,

Laurent Pinchart