[PATCH 2/3] media: dt-bindings: media: i2c: Add IMX462 to the IMX290 binding

Dave Stevenson posted 3 patches 1 week, 1 day ago
There is a newer version of this series
[PATCH 2/3] media: dt-bindings: media: i2c: Add IMX462 to the IMX290 binding
Posted by Dave Stevenson 1 week, 1 day ago
IMX462 is the successor to IMX290, which is supportable by
the existing IMX290 driver via a new compatible string.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
---
 Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
index bf05ca48601a..fa69bd21c8da 100644
--- a/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
@@ -33,6 +33,8 @@ properties:
           - sony,imx290lqr # Colour
           - sony,imx290llr # Monochrome
           - sony,imx327lqr # Colour
+          - sony,imx462lqr # Colour
+          - sony,imx462llr # Monochrome
       - const: sony,imx290
         deprecated: true
 

-- 
2.34.1
Re: [PATCH 2/3] media: dt-bindings: media: i2c: Add IMX462 to the IMX290 binding
Posted by Laurent Pinchart 1 week, 1 day ago
Hi Dave,

Thank you for the patch.

On Thu, Nov 14, 2024 at 04:01:14PM +0000, Dave Stevenson wrote:
> IMX462 is the successor to IMX290, which is supportable by
> the existing IMX290 driver via a new compatible string.
> 
> Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>

With one of the "media:" dropped from the subject line,

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
> index bf05ca48601a..fa69bd21c8da 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
> @@ -33,6 +33,8 @@ properties:
>            - sony,imx290lqr # Colour
>            - sony,imx290llr # Monochrome
>            - sony,imx327lqr # Colour
> +          - sony,imx462lqr # Colour
> +          - sony,imx462llr # Monochrome
>        - const: sony,imx290
>          deprecated: true
>  

-- 
Regards,

Laurent Pinchart
Re: [PATCH 2/3] media: dt-bindings: media: i2c: Add IMX462 to the IMX290 binding
Posted by Conor Dooley 1 week, 1 day ago
On Thu, Nov 14, 2024 at 04:01:14PM +0000, Dave Stevenson wrote:
> IMX462 is the successor to IMX290, which is supportable by
> the existing IMX290 driver via a new compatible string.
> 
> Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>

You've got "media: " twice in $subject.
Acked-by: Conor Dooley <conor.dooley@microchip.com>