Re: [PATCH v4 0/5] media: uvcvideo: Automatically handle invalid uvc_versions

[Hans de Goede]

Hi,

On 28-Sep-26 16:31, Ricardo Ribalda wrote:
> This series changes the way we handle invalid uvc_versions. Instead of
> just failing it will try to continue probing.
> 
> Now that we are at it fix the way we handle device warnings.
> 
> Thanks!
> 
> Signed-off-by: Ricardo Ribalda <ribalda@xxxxxxxxxxxx>

Thank you for your patch.

I have merged patches 1 2 and 5 into

https://gitlab.freedesktop.org/linux-media/users/uvc/-/commits/for-next/

I've held off on merging patches 3 and 4 for now because of the ongoing
discussion about those.

Regards,

Hans


> ---
> Changes in v4:
> - Handle invalid dwMaxVideoFrameSize.
> - ctrl_size -> video_ctrl_size
> - CTRL_SIZE->VERSION
> - Link to v3: https://lore.kernel.org/r/20260911-uvc-version-v3-0-604328d8a0dd@xxxxxxxxxxxx
> 
> Changes in v3:
> - Move uvc_video_ctrl_size to uvc_video_init()
> - Fixes for uvc_warn_once
> - Remove bmVideoStandardParsing parsing
> - Link to v2: https://lore.kernel.org/r/20260911-uvc-version-v2-0-395f27b0444c@xxxxxxxxxxxx
> 
> Changes in v2:
> - Add quirk for Avermedia
> - Change the way the compliance error message is printed and cache
>   ctrl_size
> - Link to v1: https://lore.kernel.org/r/20260907-uvc-version-v1-1-34e45dfe04de@xxxxxxxxxxxx
> 
> ---
> Ricardo Ribalda (5):
>       media: uvcvideo: uvc_warn should warn not info
>       media: uvcvideo: Use uvc_warn_once when it make sense
>       media: uvcvideo: Automatically handle cameras with invalid uvc_version
>       media: uvcvideo: Do not parse bmVideoStandards
>       media: uvcvideo: Force UVC version for Avermedia GC515
> 
>  drivers/media/usb/uvc/uvc_driver.c | 17 ++++++++----
>  drivers/media/usb/uvc/uvc_video.c  | 56 ++++++++++++++++++++++++--------------
>  drivers/media/usb/uvc/uvcvideo.h   |  6 ++--
>  3 files changed, 52 insertions(+), 27 deletions(-)
> ---
> base-commit: 58348f64125e9a3e44d3abb275ca7f4e6c9641e5
> change-id: 20260907-uvc-version-aedefc4384ce
> 
> Best regards,