[libcamera-devel] [PATCH] libcamera: v4l2_subdevice: Set format field to V4L2_FIELD_NONE

Kieran Bingham kieran.bingham at ideasonboard.com
Tue Oct 12 13:28:22 CEST 2021


Quoting Laurent Pinchart (2021-10-12 01:13:25)
> libcamera doesn't support interlaced formats, set the field to
> V4L2_FIELD_NONE explicitly instead of relying on drivers to interpret
> V4L2_FIELD_ANY the way we want it.
> 

And what about v4l2_videodevice.cpp: <git greps> Ah it's already there.

Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  src/libcamera/v4l2_subdevice.cpp | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/libcamera/v4l2_subdevice.cpp b/src/libcamera/v4l2_subdevice.cpp
> index 8fe5e45b31ee..023e2328893d 100644
> --- a/src/libcamera/v4l2_subdevice.cpp
> +++ b/src/libcamera/v4l2_subdevice.cpp
> @@ -425,6 +425,7 @@ int V4L2Subdevice::setFormat(unsigned int pad, V4L2SubdeviceFormat *format,
>         subdevFmt.format.width = format->size.width;
>         subdevFmt.format.height = format->size.height;
>         subdevFmt.format.code = format->mbus_code;
> +       subdevFmt.format.field = V4L2_FIELD_NONE;
>  
>         int ret = ioctl(VIDIOC_SUBDEV_S_FMT, &subdevFmt);
>         if (ret) {
> 
> base-commit: fa7a95bf32c6a8ee87b67159d4238cbc8e6dc3a9
> -- 
> Regards,
> 
> Laurent Pinchart
>


More information about the libcamera-devel mailing list