[PATCHv3,0/7] media: use vb2_video_unregister_device()

Message ID 20200713113048.1150542-1-hverkuil-cisco@xs4all.nl (mailing list archive)
Headers
Series media: use vb2_video_unregister_device() |

Message

Hans Verkuil July 13, 2020, 11:30 a.m. UTC
  This series supersedes this patch:

https://patchwork.linuxtv.org/project/linux-media/patch/7e7e9841-7f3a-468b-01c8-5921e5c14df8@xs4all.nl/

This series checks drivers that use vb2_queue_release() incorrectly
and fixes them, and calls vb2_video_unregister_device() as well if the
driver set vdev->queue.

There are a lot more drivers that set vdev->queue and can probably
benefit from using vb2_video_unregister_device(), but that's something
for a future series.

The main focus of this series is to check for incorrect use of
vb2_queue_release().

Changes since v2: add patches 2-7.

Regards,

	Hans

Hans Verkuil (7):
  videobuf2-v4l2.c: add vb2_video_unregister_device helper function
  qcom/camss: use vb2_video_unregister_device()
  media/pci: use vb2_video_unregister_device()
  media/platform: drop vb2_queue_release()
  media/usb: use vb2_video_unregister_device()
  vimc: use vb2_video_unregister_device()
  staging/media: drop vb2_queue_release()

 .../media/common/videobuf2/videobuf2-v4l2.c   | 51 ++++++++++++++++---
 drivers/media/pci/dt3155/dt3155.c             |  3 +-
 drivers/media/pci/intel/ipu3/ipu3-cio2.c      |  9 ++--
 drivers/media/pci/saa7134/saa7134-core.c      |  6 +--
 drivers/media/pci/saa7134/saa7134-empress.c   |  3 +-
 drivers/media/pci/saa7134/saa7134-go7007.c    |  2 +-
 drivers/media/pci/saa7134/saa7134-video.c     |  2 -
 drivers/media/pci/sta2x11/sta2x11_vip.c       |  6 +--
 drivers/media/pci/tw5864/tw5864-video.c       |  4 +-
 drivers/media/platform/aspeed-video.c         |  5 +-
 .../platform/mtk-vcodec/mtk_vcodec_dec.c      |  4 +-
 drivers/media/platform/qcom/camss/camss-vfe.c |  8 ---
 drivers/media/platform/qcom/camss/camss-vfe.h |  2 -
 .../media/platform/qcom/camss/camss-video.c   | 12 +----
 .../media/platform/qcom/camss/camss-video.h   |  2 -
 drivers/media/platform/qcom/camss/camss.c     |  5 --
 drivers/media/platform/qcom/venus/vdec.c      |  8 +--
 drivers/media/platform/qcom/venus/venc.c      |  8 +--
 .../platform/sunxi/sun4i-csi/sun4i_csi.c      |  1 +
 .../platform/sunxi/sun4i-csi/sun4i_dma.c      |  6 +--
 .../platform/sunxi/sun6i-csi/sun6i_video.c    |  7 +--
 .../media/test-drivers/vimc/vimc-capture.c    |  7 +--
 drivers/media/test-drivers/vivid/vivid-core.c | 32 ++++++------
 drivers/media/usb/au0828/au0828-video.c       | 12 ++---
 drivers/media/usb/dvb-usb/cxusb-analog.c      | 13 ++---
 drivers/media/usb/usbtv/usbtv-video.c         |  4 +-
 drivers/staging/media/meson/vdec/vdec.c       |  8 +--
 drivers/staging/media/rkisp1/rkisp1-capture.c |  2 +-
 drivers/staging/media/rkisp1/rkisp1-params.c  |  7 +--
 drivers/staging/media/rkisp1/rkisp1-stats.c   |  6 +--
 drivers/staging/media/tegra-video/vi.c        |  8 +--
 include/media/videobuf2-v4l2.h                | 17 +++++++
 32 files changed, 118 insertions(+), 152 deletions(-)