Re: [PATCH 02/13] media: ipu6: Set minimum height to 1

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Sakari,

Thank you for the patch.

On Thu, Jun 19, 2025 at 11:15:35AM +0300, Sakari Ailus wrote:
> For image data generally 2 seems like a minimum height that surely won't
> cause any issues, but some sensors have metadata the height of which is
> just one line. Set the minimum height to 1.
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>

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

> ---
>  drivers/media/pci/intel/ipu6/ipu6-isys.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/media/pci/intel/ipu6/ipu6-isys.h b/drivers/media/pci/intel/ipu6/ipu6-isys.h
> index f488e782c26e..0e2c8b71edfc 100644
> --- a/drivers/media/pci/intel/ipu6/ipu6-isys.h
> +++ b/drivers/media/pci/intel/ipu6/ipu6-isys.h
> @@ -40,7 +40,7 @@ struct ipu6_bus_device;
>  #define IPU6_ISYS_NUM_RECV_QUEUE 1
>  
>  #define IPU6_ISYS_MIN_WIDTH		2U
> -#define IPU6_ISYS_MIN_HEIGHT		2U
> +#define IPU6_ISYS_MIN_HEIGHT		1U
>  #define IPU6_ISYS_MAX_WIDTH		4672U
>  #define IPU6_ISYS_MAX_HEIGHT		3416U
>  

-- 
Regards,

Laurent Pinchart