mbox series

[GIT,PULL,FOR,v6.6] Remove the old videobuf framework

Message ID 004b9f44-62db-4a65-380f-80129476f2f2@xs4all.nl
State New
Headers show
Series [GIT,PULL,FOR,v6.6] Remove the old videobuf framework | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.6l

Message

Hans Verkuil Aug. 16, 2023, 8:45 a.m. UTC
After converting the last driver (bttv) that used the old videobuf
framework, we can now drop that framework completely.

Regards,

	Hans

The following changes since commit b7ec3212a73abc987e8f33aa42988e6c39c38c92:

  media: bttv: convert to vb2 (2023-08-14 20:27:59 +0200)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.6l

for you to fetch changes up to c0bb3ecbc9a0c37cd0bb0e9156151ad48f5ef414:

  media: remove the old videobuf framework (2023-08-16 10:32:30 +0200)

----------------------------------------------------------------
Tag branch

----------------------------------------------------------------
Hans Verkuil (2):
      media: atomisp: remove left-over videobuf include
      media: remove the old videobuf framework

 Documentation/driver-api/media/v4l2-core.rst                    |    1 -
 Documentation/driver-api/media/v4l2-dev.rst                     |    8 -
 Documentation/driver-api/media/v4l2-videobuf.rst                |  403 ---------------
 Documentation/translations/zh_CN/video4linux/v4l2-framework.txt |   12 -
 drivers/media/v4l2-core/Kconfig                                 |   16 -
 drivers/media/v4l2-core/Makefile                                |    5 -
 drivers/media/v4l2-core/videobuf-core.c                         | 1198 ---------------------------------------------
 drivers/media/v4l2-core/videobuf-dma-contig.c                   |  402 ---------------
 drivers/media/v4l2-core/videobuf-dma-sg.c                       |  681 --------------------------
 drivers/media/v4l2-core/videobuf-vmalloc.c                      |  326 ------------
 drivers/staging/media/atomisp/pci/atomisp_compat_css20.c        |    1 -
 include/media/videobuf-core.h                                   |  233 ---------
 include/media/videobuf-dma-contig.h                             |   30 --
 include/media/videobuf-dma-sg.h                                 |  102 ----
 include/media/videobuf-vmalloc.h                                |   43 --
 15 files changed, 3461 deletions(-)
 delete mode 100644 Documentation/driver-api/media/v4l2-videobuf.rst
 delete mode 100644 drivers/media/v4l2-core/videobuf-core.c
 delete mode 100644 drivers/media/v4l2-core/videobuf-dma-contig.c
 delete mode 100644 drivers/media/v4l2-core/videobuf-dma-sg.c
 delete mode 100644 drivers/media/v4l2-core/videobuf-vmalloc.c
 delete mode 100644 include/media/videobuf-core.h
 delete mode 100644 include/media/videobuf-dma-contig.h
 delete mode 100644 include/media/videobuf-dma-sg.h
 delete mode 100644 include/media/videobuf-vmalloc.h

Comments

Jenkins Aug. 16, 2023, 9:14 a.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/004b9f44-62db-4a65-380f-80129476f2f2@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/331247/
Build time: 00:20:51
Link: https://lore.kernel.org/linux-media/004b9f44-62db-4a65-380f-80129476f2f2@xs4all.nl

gpg: Signature made Wed 16 Aug 2023 08:32:57 AM UTC
gpg:                using EDDSA key 52ADCAAE8A4F70B99ACD8D726B425DF79B1C1E76
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg:                 aka "Hans Verkuil <hverkuil@xs4all.nl>" [full]

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

Error/warnings:

patches/0001-media-atomisp-remove-left-over-videobuf-include.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
	../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:447 gc0310_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/pci/atomisp_fops.c: ../drivers/staging/media/atomisp/pci/atomisp_fops.c:517 atomisp_open() warn: missing unwind goto?
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2801 atomisp_cp_dvs_6axis_config() warn: missing unwind goto?
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2900 atomisp_cp_morph_table() warn: missing unwind goto?

    allyesconfig: return code #0:
	../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto?
	../drivers/media/dvb-frontends/mb86a16.c: ../drivers/media/dvb-frontends/mb86a16.c:1483 mb86a16_set_fe() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2567 dvb_register() error: we previously assumed 'fe1->dvb.frontend' could be null (see line 1743)
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2590 dvb_register() parse error: OOM: 3003836Kb sm_state_count = 1958831
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2590 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2590 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 62 seconds
	../drivers/media/pci/ivtv/ivtvfb.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../drivers/media/pci/ivtv/ivtv-driver.h):
	SPARSE:../drivers/media/pci/ivtv/ivtvfb.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
	../drivers/media/i2c/ov5645.c: ../drivers/media/i2c/ov5645.c:687 ov5645_set_power_on() warn: 'ov5645->xclk' from clk_prepare_enable() not released on lines: 687.
	../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2771 mxc_jpeg_probe() warn: missing unwind goto?
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	../drivers/media/usb/dvb-usb-v2/af9035.c: ../drivers/media/usb/dvb-usb-v2/af9035.c:467 af9035_i2c_master_xfer() warn: inconsistent returns '&d->i2c_mutex'.
	  Locked on  : 326,387
	  Unlocked on: 465,467
	../drivers/media/usb/uvc/uvc_v4l2.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../include/linux/rcuwait.h, ...):
	SPARSE:../drivers/media/usb/uvc/uvc_v4l2.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression

patches/0002-media-remove-the-old-videobuf-framework.patch:

   checkpatch.pl:
	$ cat patches/0002-media-remove-the-old-videobuf-framework.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:70: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?