Re: [GIT PULL FOR v5.17] uvcvideo fixes

Re: [GIT PULL FOR v5.17] uvcvideo fixes

[Ricardo Ribalda]

Hi Laurent

Could you also consider this patch?
https://patchwork.linuxtv.org/project/linux-media/patch/20211008120914.69175-1-ribalda@chromium.org/


Thanks!
-- 
Ricardo Ribalda

[GIT PULL FOR v5.17] [v2] uvcvideo fixes

[Laurent Pinchart]

The following changes since commit 1f1517fafda598839a02e39968c5063ddcfa51fc:

  media: cx18: drop an unused macro (2021-11-22 08:32:02 +0000)

are available in the Git repository at:

  git://linuxtv.org/pinchartl/media.git tags/uvc-next-20211201

for you to fetch changes up to 525214b072dbdddd3d40d6e061dc3714798e8e23:

  media: uvcvideo: Avoid returning invalid controls (2021-12-01 07:42:35 +0200)

----------------------------------------------------------------
- uvcvideo fixes

----------------------------------------------------------------
Colin Ian King (1):
      media: uvcvideo: Fix memory leak of object map on error exit path

James Hilliard (1):
      media: uvcvideo: Increase UVC_CTRL_CONTROL_TIMEOUT to 5 seconds.

Johan Hovold (1):
      media: uvcvideo: fix division by zero at stream start

Ricardo Ribalda (3):
      media: uvcvideo: Set the colorspace as sRGB if undefined
      media: uvcvideo: Avoid invalid memory access
      media: uvcvideo: Avoid returning invalid controls

 drivers/media/usb/uvc/uvc_ctrl.c   | 4 ++--
 drivers/media/usb/uvc/uvc_driver.c | 6 ++++--
 drivers/media/usb/uvc/uvc_v4l2.c   | 6 ++++--
 drivers/media/usb/uvc/uvc_video.c  | 4 ++++
 drivers/media/usb/uvc/uvcvideo.h   | 2 +-
 5 files changed, 15 insertions(+), 7 deletions(-)

-- 
Regards,

Laurent Pinchart

Re: [GIT PULL FOR v5.17] [v2] uvcvideo fixes (#79016)

[Jenkins]

From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/YacOun3Diggsi05V@pendragon.ideasonboard.com/
Build log: https://builder.linuxtv.org/job/patchwork/161545/
Build time: 00:17:26
Link: https://lore.kernel.org/linux-media/YacOun3Diggsi05V@pendragon.ideasonboard.com

gpg: Signature made Wed 01 Dec 2021 05:54:00 AM UTC
gpg:                using RSA key CB9D6877529820CD53099B1B65F89C37BC54210D
gpg:                issuer "laurent.pinchart@ideasonboard.com"
gpg: Can't check signature: No public key

Summary: got 1/6 patches with issues, being 1 at build time

Error/warnings:

patches/0001-media-uvcvideo-Fix-memory-leak-of-object-map-on-erro.patch:

    allyesconfig: return code #0:
	../scripts/genksyms/parse.y: warning: 9 shift/reduce conflicts [-Wconflicts-sr]
	../scripts/genksyms/parse.y: warning: 5 reduce/reduce conflicts [-Wconflicts-rr]
	../scripts/genksyms/parse.y: note: rerun with option '-Wcounterexamples' to generate conflict counterexamples

    allyesconfig: return code #0:
	../drivers/media/rc/meson-ir-tx.c:22: warning: expecting prototype for meson(). Prototype was for DEVICE_NAME() instead
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	SPARSE:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: warning: array of flexible structures
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2841 ov8865_get_selection() warn: inconsistent indenting
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2625 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:658 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 654)

[GIT PULL FOR v5.17] uvcvideo fixes

[Laurent Pinchart]

Hi Mauro,

The following changes since commit 1f1517fafda598839a02e39968c5063ddcfa51fc:

  media: cx18: drop an unused macro (2021-11-22 08:32:02 +0000)

are available in the Git repository at:

  git://linuxtv.org/pinchartl/media.git tags/uvc-next-20211201

for you to fetch changes up to 4d5d51427c5983d73a3e424bf4c6e49fbbe7db80:

  media: uvcvideo: Avoid returning invalid controls (2021-12-01 05:37:41 +0200)

----------------------------------------------------------------
- uvcvideo fixes

----------------------------------------------------------------
Colin Ian King (1):
      media: uvcvideo: Fix memory leak of object map on error exit path

James Hilliard (1):
      media: uvcvideo: Increase UVC_CTRL_CONTROL_TIMEOUT to 5 seconds.

Johan Hovold (1):
      media: uvcvideo: fix division by zero at stream start

Ricardo Ribalda (2):
      media: uvcvideo: Set the colorspace as sRGB if undefined
      media: uvcvideo: Avoid returning invalid controls

 drivers/media/usb/uvc/uvc_ctrl.c   | 2 +-
 drivers/media/usb/uvc/uvc_driver.c | 6 ++++--
 drivers/media/usb/uvc/uvc_v4l2.c   | 6 ++++--
 drivers/media/usb/uvc/uvc_video.c  | 4 ++++
 drivers/media/usb/uvc/uvcvideo.h   | 2 +-
 5 files changed, 14 insertions(+), 6 deletions(-)

-- 
Regards,

Laurent Pinchart