diff mbox series

[1/1] media: i2c: imx290: Fix IMX920 typo

Message ID 20240221071550.2769570-1-alexander.stein@ew.tq-group.com (mailing list archive)
State New, archived
Headers show
Series [1/1] media: i2c: imx290: Fix IMX920 typo | expand

Commit Message

Alexander Stein Feb. 21, 2024, 7:15 a.m. UTC 
Replace IMX920 by IMX290.

Fixes: b4ab57b07c5b9 ("media: i2c: imx290: Add crop selection targets support")
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
 drivers/media/i2c/imx290.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

Comments

Manivannan Sadhasivam Feb. 21, 2024, 7:17 a.m. UTC | #1 
On Wed, Feb 21, 2024 at 08:15:50AM +0100, Alexander Stein wrote:
> Replace IMX920 by IMX290.
> 
> Fixes: b4ab57b07c5b9 ("media: i2c: imx290: Add crop selection targets support")
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>

Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>

- Mani

> ---
>  drivers/media/i2c/imx290.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c
> index 89fcebe178395..d75e36018cdc0 100644
> --- a/drivers/media/i2c/imx290.c
> +++ b/drivers/media/i2c/imx290.c
> @@ -150,10 +150,10 @@
>  
>  #define IMX290_PIXEL_ARRAY_WIDTH			1945
>  #define IMX290_PIXEL_ARRAY_HEIGHT			1097
> -#define IMX920_PIXEL_ARRAY_MARGIN_LEFT			12
> -#define IMX920_PIXEL_ARRAY_MARGIN_RIGHT			13
> -#define IMX920_PIXEL_ARRAY_MARGIN_TOP			8
> -#define IMX920_PIXEL_ARRAY_MARGIN_BOTTOM		9
> +#define IMX290_PIXEL_ARRAY_MARGIN_LEFT			12
> +#define IMX290_PIXEL_ARRAY_MARGIN_RIGHT			13
> +#define IMX290_PIXEL_ARRAY_MARGIN_TOP			8
> +#define IMX290_PIXEL_ARRAY_MARGIN_BOTTOM		9
>  #define IMX290_PIXEL_ARRAY_RECORDING_WIDTH		1920
>  #define IMX290_PIXEL_ARRAY_RECORDING_HEIGHT		1080
>  
> @@ -1161,10 +1161,10 @@ static int imx290_get_selection(struct v4l2_subdev *sd,
>  		 * The sensor moves the readout by 1 pixel based on flips to
>  		 * keep the Bayer order the same.
>  		 */
> -		sel->r.top = IMX920_PIXEL_ARRAY_MARGIN_TOP
> +		sel->r.top = IMX290_PIXEL_ARRAY_MARGIN_TOP
>  			   + (IMX290_PIXEL_ARRAY_RECORDING_HEIGHT - format->height) / 2
>  			   + imx290->vflip->val;
> -		sel->r.left = IMX920_PIXEL_ARRAY_MARGIN_LEFT
> +		sel->r.left = IMX290_PIXEL_ARRAY_MARGIN_LEFT
>  			    + (IMX290_PIXEL_ARRAY_RECORDING_WIDTH - format->width) / 2
>  			    + imx290->hflip->val;
>  		sel->r.width = format->width;
> @@ -1183,8 +1183,8 @@ static int imx290_get_selection(struct v4l2_subdev *sd,
>  		return 0;
>  
>  	case V4L2_SEL_TGT_CROP_DEFAULT:
> -		sel->r.top = IMX920_PIXEL_ARRAY_MARGIN_TOP;
> -		sel->r.left = IMX920_PIXEL_ARRAY_MARGIN_LEFT;
> +		sel->r.top = IMX290_PIXEL_ARRAY_MARGIN_TOP;
> +		sel->r.left = IMX290_PIXEL_ARRAY_MARGIN_LEFT;
>  		sel->r.width = IMX290_PIXEL_ARRAY_RECORDING_WIDTH;
>  		sel->r.height = IMX290_PIXEL_ARRAY_RECORDING_HEIGHT;
>  
> -- 
> 2.34.1
>
Laurent Pinchart Feb. 21, 2024, 12:01 p.m. UTC | #2 
Hi Alexander,

Thank you for the patch.

On Wed, Feb 21, 2024 at 08:15:50AM +0100, Alexander Stein wrote:
> Replace IMX920 by IMX290.
> 
> Fixes: b4ab57b07c5b9 ("media: i2c: imx290: Add crop selection targets support")
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>

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

> ---
>  drivers/media/i2c/imx290.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c
> index 89fcebe178395..d75e36018cdc0 100644
> --- a/drivers/media/i2c/imx290.c
> +++ b/drivers/media/i2c/imx290.c
> @@ -150,10 +150,10 @@
>  
>  #define IMX290_PIXEL_ARRAY_WIDTH			1945
>  #define IMX290_PIXEL_ARRAY_HEIGHT			1097
> -#define IMX920_PIXEL_ARRAY_MARGIN_LEFT			12
> -#define IMX920_PIXEL_ARRAY_MARGIN_RIGHT			13
> -#define IMX920_PIXEL_ARRAY_MARGIN_TOP			8
> -#define IMX920_PIXEL_ARRAY_MARGIN_BOTTOM		9
> +#define IMX290_PIXEL_ARRAY_MARGIN_LEFT			12
> +#define IMX290_PIXEL_ARRAY_MARGIN_RIGHT			13
> +#define IMX290_PIXEL_ARRAY_MARGIN_TOP			8
> +#define IMX290_PIXEL_ARRAY_MARGIN_BOTTOM		9
>  #define IMX290_PIXEL_ARRAY_RECORDING_WIDTH		1920
>  #define IMX290_PIXEL_ARRAY_RECORDING_HEIGHT		1080
>  
> @@ -1161,10 +1161,10 @@ static int imx290_get_selection(struct v4l2_subdev *sd,
>  		 * The sensor moves the readout by 1 pixel based on flips to
>  		 * keep the Bayer order the same.
>  		 */
> -		sel->r.top = IMX920_PIXEL_ARRAY_MARGIN_TOP
> +		sel->r.top = IMX290_PIXEL_ARRAY_MARGIN_TOP
>  			   + (IMX290_PIXEL_ARRAY_RECORDING_HEIGHT - format->height) / 2
>  			   + imx290->vflip->val;
> -		sel->r.left = IMX920_PIXEL_ARRAY_MARGIN_LEFT
> +		sel->r.left = IMX290_PIXEL_ARRAY_MARGIN_LEFT
>  			    + (IMX290_PIXEL_ARRAY_RECORDING_WIDTH - format->width) / 2
>  			    + imx290->hflip->val;
>  		sel->r.width = format->width;
> @@ -1183,8 +1183,8 @@ static int imx290_get_selection(struct v4l2_subdev *sd,
>  		return 0;
>  
>  	case V4L2_SEL_TGT_CROP_DEFAULT:
> -		sel->r.top = IMX920_PIXEL_ARRAY_MARGIN_TOP;
> -		sel->r.left = IMX920_PIXEL_ARRAY_MARGIN_LEFT;
> +		sel->r.top = IMX290_PIXEL_ARRAY_MARGIN_TOP;
> +		sel->r.left = IMX290_PIXEL_ARRAY_MARGIN_LEFT;
>  		sel->r.width = IMX290_PIXEL_ARRAY_RECORDING_WIDTH;
>  		sel->r.height = IMX290_PIXEL_ARRAY_RECORDING_HEIGHT;
>
diff mbox series

Patch

diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c
index 89fcebe178395..d75e36018cdc0 100644
--- a/drivers/media/i2c/imx290.c
+++ b/drivers/media/i2c/imx290.c
@@ -150,10 +150,10 @@ 
 
 #define IMX290_PIXEL_ARRAY_WIDTH			1945
 #define IMX290_PIXEL_ARRAY_HEIGHT			1097
-#define IMX920_PIXEL_ARRAY_MARGIN_LEFT			12
-#define IMX920_PIXEL_ARRAY_MARGIN_RIGHT			13
-#define IMX920_PIXEL_ARRAY_MARGIN_TOP			8
-#define IMX920_PIXEL_ARRAY_MARGIN_BOTTOM		9
+#define IMX290_PIXEL_ARRAY_MARGIN_LEFT			12
+#define IMX290_PIXEL_ARRAY_MARGIN_RIGHT			13
+#define IMX290_PIXEL_ARRAY_MARGIN_TOP			8
+#define IMX290_PIXEL_ARRAY_MARGIN_BOTTOM		9
 #define IMX290_PIXEL_ARRAY_RECORDING_WIDTH		1920
 #define IMX290_PIXEL_ARRAY_RECORDING_HEIGHT		1080
 
@@ -1161,10 +1161,10 @@  static int imx290_get_selection(struct v4l2_subdev *sd,
 		 * The sensor moves the readout by 1 pixel based on flips to
 		 * keep the Bayer order the same.
 		 */
-		sel->r.top = IMX920_PIXEL_ARRAY_MARGIN_TOP
+		sel->r.top = IMX290_PIXEL_ARRAY_MARGIN_TOP
 			   + (IMX290_PIXEL_ARRAY_RECORDING_HEIGHT - format->height) / 2
 			   + imx290->vflip->val;
-		sel->r.left = IMX920_PIXEL_ARRAY_MARGIN_LEFT
+		sel->r.left = IMX290_PIXEL_ARRAY_MARGIN_LEFT
 			    + (IMX290_PIXEL_ARRAY_RECORDING_WIDTH - format->width) / 2
 			    + imx290->hflip->val;
 		sel->r.width = format->width;
@@ -1183,8 +1183,8 @@  static int imx290_get_selection(struct v4l2_subdev *sd,
 		return 0;
 
 	case V4L2_SEL_TGT_CROP_DEFAULT:
-		sel->r.top = IMX920_PIXEL_ARRAY_MARGIN_TOP;
-		sel->r.left = IMX920_PIXEL_ARRAY_MARGIN_LEFT;
+		sel->r.top = IMX290_PIXEL_ARRAY_MARGIN_TOP;
+		sel->r.left = IMX290_PIXEL_ARRAY_MARGIN_LEFT;
 		sel->r.width = IMX290_PIXEL_ARRAY_RECORDING_WIDTH;
 		sel->r.height = IMX290_PIXEL_ARRAY_RECORDING_HEIGHT;