diff mbox series

media: dt-bindings: imx7-csi: Make power-domains not required for imx8mq

Message ID 20231004201105.2323758-1-festevam@gmail.com
State New
Headers show
Series media: dt-bindings: imx7-csi: Make power-domains not required for imx8mq | expand

Commit Message

Fabio Estevam Oct. 4, 2023, 8:11 p.m. UTC
From: Fabio Estevam <festevam@denx.de>

On i.MX8MQ the MIPI CSI block does have an associated power-domain, but
the CSI bridge does not.

Remove the power-domains requirement from the i.MX8MQ CSI bridge
to fix the following schema warning:

imx8mq-librem5-r4.dtb: csi@30a90000: 'power-domains' is a required property
from schema $id: http://devicetree.org/schemas/media/nxp,imx7-csi.yaml#

Fixes: de655386845a ("media: dt-bindings: media: imx7-csi: Document i.MX8M power-domains property")
Signed-off-by: Fabio Estevam <festevam@denx.de>
---

 Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml | 1 -
 1 file changed, 1 deletion(-)

Comments

Marek Vasut Oct. 5, 2023, 2:20 a.m. UTC | #1
On 10/4/23 22:11, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> On i.MX8MQ the MIPI CSI block does have an associated power-domain, but
> the CSI bridge does not.
> 
> Remove the power-domains requirement from the i.MX8MQ CSI bridge
> to fix the following schema warning:
> 
> imx8mq-librem5-r4.dtb: csi@30a90000: 'power-domains' is a required property
> from schema $id: http://devicetree.org/schemas/media/nxp,imx7-csi.yaml#
> 
> Fixes: de655386845a ("media: dt-bindings: media: imx7-csi: Document i.MX8M power-domains property")
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Reviewed-by: Marek Vasut <marex@denx.de>

Thanks !
Conor Dooley Oct. 5, 2023, 10:44 a.m. UTC | #2
On Wed, Oct 04, 2023 at 05:11:05PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> On i.MX8MQ the MIPI CSI block does have an associated power-domain, but
> the CSI bridge does not.
> 
> Remove the power-domains requirement from the i.MX8MQ CSI bridge
> to fix the following schema warning:
> 
> imx8mq-librem5-r4.dtb: csi@30a90000: 'power-domains' is a required property
> from schema $id: http://devicetree.org/schemas/media/nxp,imx7-csi.yaml#
> 
> Fixes: de655386845a ("media: dt-bindings: media: imx7-csi: Document i.MX8M power-domains property")
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.

> ---
> 
>  Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml b/Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml
> index 358019e85d90..326284e151f6 100644
> --- a/Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml
> +++ b/Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml
> @@ -59,7 +59,6 @@ allOf:
>          compatible:
>            contains:
>              enum:
> -              - fsl,imx8mq-csi
>                - fsl,imx8mm-csi
>      then:
>        required:
> -- 
> 2.34.1
>
Rob Herring Oct. 6, 2023, 4:55 p.m. UTC | #3
On Wed, 04 Oct 2023 17:11:05 -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> On i.MX8MQ the MIPI CSI block does have an associated power-domain, but
> the CSI bridge does not.
> 
> Remove the power-domains requirement from the i.MX8MQ CSI bridge
> to fix the following schema warning:
> 
> imx8mq-librem5-r4.dtb: csi@30a90000: 'power-domains' is a required property
> from schema $id: http://devicetree.org/schemas/media/nxp,imx7-csi.yaml#
> 
> Fixes: de655386845a ("media: dt-bindings: media: imx7-csi: Document i.MX8M power-domains property")
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
> 
>  Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 

Applied, thanks!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml b/Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml
index 358019e85d90..326284e151f6 100644
--- a/Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml
+++ b/Documentation/devicetree/bindings/media/nxp,imx7-csi.yaml
@@ -59,7 +59,6 @@  allOf:
         compatible:
           contains:
             enum:
-              - fsl,imx8mq-csi
               - fsl,imx8mm-csi
     then:
       required: