[libcamera-devel] [PATCH] libcamera: ipu3: Initialize V|H flip support

[Niklas Söderlund]

Hi Jacopo,

Thanks for your patch.

On 2021-02-23 14:53:31 +0100, Jacopo Mondi wrote:
> The flag that is used to record if the sensor driver supports the H/V flip
> controls is initialized at camera creation time, but it was not
> initialized by the IPU3Camera constructor, resulting in erroneous
> values that might break capture for sensor that do not support flipping.
> 
> Fix this by initializing the flag in the class constructor.
> 
> Fixes: 6c4ce7de30c8 ("libcamera: ipu3: Add rotation to ipu3 pipeline")
> Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  src/libcamera/pipeline/ipu3/ipu3.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp
> index daace29f71ea..2838196a4b21 100644
> --- a/src/libcamera/pipeline/ipu3/ipu3.cpp
> +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp
> @@ -56,7 +56,7 @@ class IPU3CameraData : public CameraData
>  {
>  public:
>  	IPU3CameraData(PipelineHandler *pipe)
> -		: CameraData(pipe), exposureTime_(0)
> +		: CameraData(pipe), exposureTime_(0), supportsFlips_(false)
>  	{
>  	}
> 
> --
> 2.30.0
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund