mbox series

[GIT,PULL,FOR,v6.1] dw100: Add i.MX8MP dw100 dewarper driver

Message ID 507d4249-5061-b6c4-12d9-aa77ec7a4d48@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v6.1] dw100: Add i.MX8MP dw100 dewarper driver | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.1b

Message

Hans Verkuil Aug. 10, 2022, 8:16 a.m. UTC
Note: patches for v4l-utils to test v4l2_ctrl_modify_dimensions using
the vivid driver pixel_array test control are available here:

https://git.linuxtv.org/hverkuil/v4l-utils.git/log/?h=array

These will be applied once support for this is merged.

Regards,

	Hans

The following changes since commit 485ade76c95ac5ccaa52fee9d712471c9211b989:

  media: hantro: Remove dedicated control documentation (2022-07-27 22:38:47 +0200)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.1b

for you to fetch changes up to 4b891726d7cd450be6b7a0681a51b773fe646f7b:

  media: MAINTAINERS: add entry for i.MX8MP DW100 v4l2 mem2mem driver (2022-08-10 09:23:52 +0200)

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

----------------------------------------------------------------
Hans Verkuil (6):
      v4l2-ctrls: allocate space for arrays
      v4l2-ctrls: alloc arrays in ctrl_ref
      v4l2-ctrls: add v4l2_ctrl_modify_dimensions
      v4l2-ctrls: add change flag for when dimensions change
      vivid: add pixel_array test control
      v4l2-ctrls: optimize type_ops for arrays

Xavier Roumegue (7):
      v4l2-ctrls: Export default v4l2_ctrl_type_ops callbacks
      media: Documentation: dw100: Add user documentation for the DW100 driver
      media: v4l: uapi: Add user control base for DW100 controls
      media: uapi: Add a control for DW100 driver
      media: dt-bindings: media: Add i.MX8MP DW100 binding
      media: dw100: Add i.MX8MP dw100 dewarper driver
      media: MAINTAINERS: add entry for i.MX8MP DW100 v4l2 mem2mem driver

 Documentation/devicetree/bindings/media/nxp,dw100.yaml       |   69 ++
 Documentation/userspace-api/media/drivers/dw100.rst          |   84 +++
 Documentation/userspace-api/media/drivers/index.rst          |    1 +
 Documentation/userspace-api/media/v4l/vidioc-dqevent.rst     |    5 +
 Documentation/userspace-api/media/videodev2.h.rst.exceptions |    1 +
 MAINTAINERS                                                  |    9 +
 drivers/media/platform/nxp/Kconfig                           |    1 +
 drivers/media/platform/nxp/Makefile                          |    1 +
 drivers/media/platform/nxp/dw100/Kconfig                     |   16 +
 drivers/media/platform/nxp/dw100/Makefile                    |    3 +
 drivers/media/platform/nxp/dw100/dw100.c                     | 1706 +++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/media/platform/nxp/dw100/dw100_regs.h                |  117 ++++
 drivers/media/test-drivers/vivid/vivid-core.h                |    5 +-
 drivers/media/test-drivers/vivid/vivid-ctrls.c               |   14 +
 drivers/media/test-drivers/vivid/vivid-vid-cap.c             |    4 +
 drivers/media/v4l2-core/v4l2-ctrls-api.c                     |   62 +-
 drivers/media/v4l2-core/v4l2-ctrls-core.c                    |  214 ++++---
 include/media/v4l2-ctrls.h                                   |  140 +++-
 include/uapi/linux/dw100.h                                   |   14 +
 include/uapi/linux/v4l2-controls.h                           |    6 +
 include/uapi/linux/videodev2.h                               |    1 +
 21 files changed, 2356 insertions(+), 117 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/media/nxp,dw100.yaml
 create mode 100644 Documentation/userspace-api/media/drivers/dw100.rst
 create mode 100644 drivers/media/platform/nxp/dw100/Kconfig
 create mode 100644 drivers/media/platform/nxp/dw100/Makefile
 create mode 100644 drivers/media/platform/nxp/dw100/dw100.c
 create mode 100644 drivers/media/platform/nxp/dw100/dw100_regs.h
 create mode 100644 include/uapi/linux/dw100.h

Comments

Jenkins Aug. 10, 2022, 11:15 a.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/507d4249-5061-b6c4-12d9-aa77ec7a4d48@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/232381/
Build time: 00:50:55
Link: https://lore.kernel.org/linux-media/507d4249-5061-b6c4-12d9-aa77ec7a4d48@xs4all.nl

gpg: Signature made Wed 10 Aug 2022 08:13:01 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 11/13 patches with issues, being 7 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-v4l2-ctrls-allocate-space-for-arrays.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:
	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/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:5539 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	../drivers/media/dvb-frontends/sp887x.c: ../drivers/media/dvb-frontends/sp887x.c:178 sp887x_initial_setup() error: __memcpy() '&buf[2]' too small (30 vs 16384)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2878 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000008Kb sm_state_count = 1725655
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 49 seconds

patches/0002-v4l2-ctrls-alloc-arrays-in-ctrl_ref.patch:

    allyesconfig: return code #0:
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000008Kb sm_state_count = 1725655
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 48 seconds
	../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:5517 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2864 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0003-v4l2-ctrls-add-v4l2_ctrl_modify_dimensions.patch:

    allyesconfig: return code #0:
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000024Kb sm_state_count = 1725648
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 51 seconds
	../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:5429 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2846 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0004-v4l2-ctrls-add-change-flag-for-when-dimensions-chang.patch:

    allyesconfig: return code #0:
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1936 vivid_create_instance() parse error: turning off implications after 60 seconds
	../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:4593 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000020Kb sm_state_count = 1725648
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 60 seconds
	../drivers/media/pci/cx88/cx88-dvb.c: ../drivers/media/pci/cx88/cx88-dvb.c:1626 dvb_register() error: we previously assumed 'fe1->dvb.frontend' could be null (see line 1086)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2836 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0006-v4l2-ctrls-optimize-type_ops-for-arrays.patch:

    allyesconfig: return code #0:
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1981 vivid_create_instance() parse error: turning off implications after 60 seconds
	../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:5077 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2260 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000028Kb sm_state_count = 1724849
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 74 seconds
	../drivers/media/pci/cx88/cx88-dvb.c: ../drivers/media/pci/cx88/cx88-dvb.c:1626 dvb_register() error: we previously assumed 'fe1->dvb.frontend' could be null (see line 1086)
	../drivers/media/pci/saa7134/saa7134-dvb.c: ../drivers/media/pci/saa7134/saa7134-dvb.c:1933 dvb_init() parse error: turning off implications after 60 seconds
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2831 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0007-v4l2-ctrls-Export-default-v4l2_ctrl_type_ops-callbac.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:4835 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2128 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000008Kb sm_state_count = 1724319
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 76 seconds
	../drivers/media/pci/cx88/cx88-dvb.c: ../drivers/media/pci/cx88/cx88-dvb.c:1626 dvb_register() error: we previously assumed 'fe1->dvb.frontend' could be null (see line 1086)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2810 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1981 vivid_create_instance() parse error: turning off implications after 60 seconds

patches/0008-media-Documentation-dw100-Add-user-documentation-for.patch:

   checkpatch.pl:
	$ cat patches/0008-media-Documentation-dw100-Add-user-documentation-for.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:20: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0009-media-v4l-uapi-Add-user-control-base-for-DW100-contr.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:4857 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1981 vivid_create_instance() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:1832 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000008Kb sm_state_count = 1722506
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 100 seconds
	../drivers/media/pci/cx88/cx88-dvb.c: ../drivers/media/pci/cx88/cx88-dvb.c:1626 dvb_register() error: we previously assumed 'fe1->dvb.frontend' could be null (see line 1086)
	../drivers/media/pci/saa7134/saa7134-dvb.c: ../drivers/media/pci/saa7134/saa7134-dvb.c:1935 dvb_init() parse error: turning off implications after 60 seconds
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2799 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0010-media-uapi-Add-a-control-for-DW100-driver.patch:

   checkpatch.pl:
	$ cat patches/0010-media-uapi-Add-a-control-for-DW100-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:45: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0011-media-dt-bindings-media-Add-i.MX8MP-DW100-binding.patch:

   checkpatch.pl:
	$ cat patches/0011-media-dt-bindings-media-Add-i.MX8MP-DW100-binding.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:19: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0012-media-dw100-Add-i.MX8MP-dw100-dewarper-driver.patch:

   checkpatch.pl:
	$ cat patches/0012-media-dw100-Add-i.MX8MP-dw100-dewarper-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:59: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:1837: CHECK: Prefer using the BIT macro


Error #512 when building PDF docs