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

[Fabian Wüthrich]

Hi Jacopo

I missed that one. Thanks for fixing it.

Tested-by: Fabian Wüthrich <me at fabwu.ch>

On 23.02.21 14:53, 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>
> ---
>  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
>