[libcamera-devel] [PATCH] libcamera: v4l2_videodevice: Return correct format for metadata

Niklas Söderlund niklas.soderlund at ragnatech.se
Fri Apr 17 02:07:57 CEST 2020


Hi Laurent,

Thanks for your work.

On 2020-04-17 01:05:58 +0300, Laurent Pinchart wrote:
> When setting format on a metadata video device, the returned format
> isn't updated with the actual set format due to a typo. Fix it.
> 
> Fixes: 629e9301c518 ("libcamera: v4l2_device: Add META support in g/s_fmt")
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

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

> ---
>  src/libcamera/v4l2_videodevice.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/libcamera/v4l2_videodevice.cpp b/src/libcamera/v4l2_videodevice.cpp
> index 439a9c90dbf7..a959cfe65c43 100644
> --- a/src/libcamera/v4l2_videodevice.cpp
> +++ b/src/libcamera/v4l2_videodevice.cpp
> @@ -848,7 +848,7 @@ int V4L2VideoDevice::setFormatMeta(V4L2DeviceFormat *format)
>  	 */
>  	format->size.width = 0;
>  	format->size.height = 0;
> -	format->fourcc = format->fourcc;
> +	format->fourcc = V4L2PixelFormat(pix->dataformat);
>  	format->planesCount = 1;
>  	format->planes[0].bpl = pix->buffersize;
>  	format->planes[0].size = pix->buffersize;
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund


More information about the libcamera-devel mailing list