Re: [PATCH v8 0/5] media: uvcvideo: Introduce V4L2_META_FMT_UVC_MSXU_1_5 + other meta fixes

[Hans de Goede <hansg@kernel.org>]

Hi Ricardo,

On 7-Jul-25 8:34 PM, Ricardo Ribalda wrote:
> This series introduces a new metadata format for UVC cameras and adds a
> couple of improvements to the UVC metadata handling.
> 
> The new metadata format can be enabled in runtime with quirks.
> 
> Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
> ---
> Changes in v8:
> - Dynamically fill dev->meta_formats instead of be const.
> - Link to v7: https://lore.kernel.org/r/20250617-uvc-meta-v7-0-9c50623e2286@chromium.org

Thank you for the new version. I've merged this series
and pushed this to uvc/for-next now.

Regards,

Hans




> Changes in v7:
> - Add patch: Introduce dev->meta_formats
> - Link to v6: https://lore.kernel.org/r/20250604-uvc-meta-v6-0-7141d48c322c@chromium.org
> 
> Changes in v6 (Thanks Laurent):
> - Fix typo in metafmt-uvc.rst
> - Improve metafmt-uvc-msxu-1-5.rst
> - uvc_meta_v4l2_try_format() block MSXU format unless the quirk is
>   active
> - Refactor uvc_enable_msxu
> - Document uvc_meta_detect_msxu
> - Rebase series
> - Add R-b
> - Link to v5: https://lore.kernel.org/r/20250404-uvc-meta-v5-0-f79974fc2d20@chromium.org
> 
> Changes in v5:
> - Fix codestyle and kerneldoc warnings reported by media-ci
> - Link to v4: https://lore.kernel.org/r/20250403-uvc-meta-v4-0-877aa6475975@chromium.org
> 
> Changes in v4:
> - Rename format to V4L2_META_FMT_UVC_MSXU_1_5 (Thanks Mauro)
> - Flag the new format with a quirk.
> - Autodetect MSXU devices.
> - Link to v3: https://lore.kernel.org/linux-media/20250313-uvc-metadata-v3-0-c467af869c60@chromium.org/
> 
> Changes in v3:
> - Fix doc syntax errors.
> - Link to v2: https://lore.kernel.org/r/20250306-uvc-metadata-v2-0-7e939857cad5@chromium.org
> 
> Changes in v2:
> - Add metadata invalid fix
> - Move doc note to a separate patch
> - Introduce V4L2_META_FMT_UVC_CUSTOM (thanks HdG!).
> - Link to v1: https://lore.kernel.org/r/20250226-uvc-metadata-v1-1-6cd6fe5ec2cb@chromium.org
> 
> ---
> Ricardo Ribalda (5):
>       media: uvcvideo: Do not mark valid metadata as invalid
>       media: Documentation: Add note about UVCH length field
>       media: uvcvideo: Introduce dev->meta_formats
>       media: uvcvideo: Introduce V4L2_META_FMT_UVC_MSXU_1_5
>       media: uvcvideo: Auto-set UVC_QUIRK_MSXU_META
> 
>  .../userspace-api/media/v4l/meta-formats.rst       |   1 +
>  .../media/v4l/metafmt-uvc-msxu-1-5.rst             |  23 +++++
>  .../userspace-api/media/v4l/metafmt-uvc.rst        |   4 +-
>  MAINTAINERS                                        |   1 +
>  drivers/media/usb/uvc/uvc_driver.c                 |   7 ++
>  drivers/media/usb/uvc/uvc_metadata.c               | 115 +++++++++++++++++++--
>  drivers/media/usb/uvc/uvc_video.c                  |  12 +--
>  drivers/media/usb/uvc/uvcvideo.h                   |   7 ++
>  drivers/media/v4l2-core/v4l2-ioctl.c               |   1 +
>  include/linux/usb/uvc.h                            |   3 +
>  include/uapi/linux/videodev2.h                     |   1 +
>  11 files changed, 161 insertions(+), 14 deletions(-)
> ---
> base-commit: a8598c7de1bcd94461ca54c972efa9b4ea501fb9
> change-id: 20250403-uvc-meta-e556773d12ae
> 
> Best regards,