mbox series

[GIT,PULL,FOR,5.20] More V4L2 patches

Message ID Ys8kICeZqicfr0mC@valkosipuli.retiisi.eu (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,5.20] More V4L2 patches | expand

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/for-5.20-2.1-signed

Message

Sakari Ailus July 13, 2022, 7:59 p.m. UTC 
Hi Mauro,

Here's another set of V4L2 patches for 5.20. Hopefully it's not too late
for that yet.

Here are a few fixes and improvements to existing APIs and drivers. No new
drivers are included.

Please pull.


The following changes since commit e670f5d672ef3d00b0b8c69eff09a019e6dd4ef9:

  media: amphion: only insert the first sequence startcode for vc1l format (2022-07-08 18:18:49 +0100)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.20-2.1-signed

for you to fetch changes up to e927c9847d7aeb12c5853a4d84d58723f5b5db36:

  media: ov5693: add ov5693_of_match, dts support (2022-07-13 22:57:29 +0300)

----------------------------------------------------------------
More V4L2 patches for 5.20

----------------------------------------------------------------
Daniel Scally (2):
      media: entity: Add iterator for entity data links
      media: entity: Use dedicated data link iterator

Hans Verkuil (1):
      ar0521: fix Kconfig: VIDEO_V4L2 -> VIDEO_DEV

Laurent Pinchart (1):
      media: Replace dependency on VIDEO_V4L2_SUBDEV_API with select

Marek Vasut (1):
      media: mt9p031: Implement crop bounds get selection

Sakari Ailus (1):
      media: sunxi: Depend on GENERIC_PHY_MIPI_DPHY

Tomi Valkeinen (2):
      media: subdev: Add v4l2_subdev_call_state_try() macro
      media: stm32: dcmi: Fix subdev op call with uninitialized state

Tommaso Merciai (6):
      media: ov5693: count num_supplies using array_size
      media: ov5693: add dvdd into ov5693_supply_names array
      media: ov5693: rename clk into xvclk
      media: ov5693: add support for acpi clock-frequency prop
      media: dt-bindings: ov5693: document YAML binding
      media: ov5693: add ov5693_of_match, dts support

Yang Yingliang (1):
      media: ov7251: add missing disable functions on error in ov7251_set_power_on()

 .../devicetree/bindings/media/i2c/ovti,ov5693.yaml | 124 +++++++++++++++++++++
 MAINTAINERS                                        |   1 +
 drivers/media/i2c/Kconfig                          |   6 +-
 drivers/media/i2c/mt9p031.c                        |  20 +++-
 drivers/media/i2c/ov5693.c                         |  57 ++++++----
 drivers/media/i2c/ov7251.c                         |   2 +
 drivers/media/mc/mc-entity.c                       |  22 +++-
 drivers/media/platform/atmel/Kconfig               |   4 +-
 drivers/media/platform/st/stm32/stm32-dcmi.c       |  14 +--
 .../media/platform/sunxi/sun6i-mipi-csi2/Kconfig   |   1 +
 .../platform/sunxi/sun8i-a83t-mipi-csi2/Kconfig    |   1 +
 drivers/media/v4l2-core/Kconfig                    |   6 +-
 include/media/media-entity.h                       |  30 +++++
 include/media/v4l2-subdev.h                        |  34 ++++++
 14 files changed, 278 insertions(+), 44 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/ovti,ov5693.yaml

Comments

Jenkins July 13, 2022, 9:26 p.m. UTC | #1 
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/Ys8kICeZqicfr0mC@valkosipuli.retiisi.eu/
Build log: https://builder.linuxtv.org/job/patchwork/224599/
Build time: 00:23:08
Link: https://lore.kernel.org/linux-media/Ys8kICeZqicfr0mC@valkosipuli.retiisi.eu

gpg: Signature made Wed 13 Jul 2022 07:57:51 PM UTC
gpg:                using DSA key F0D0377A0D4F25A79238EFE56D40361B6E28C193
gpg:                issuer "sakari.ailus@linux.intel.com"
gpg: Good signature from "Sakari Ailus <sakari.ailus@linux.intel.com>" [full]

Summary: got 5/15 patches with issues, being 3 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-media-sunxi-Depend-on-GENERIC_PHY_MIPI_DPHY.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/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:5253 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	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/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:678 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 674)
	../drivers/media/platform/sunxi/sun6i-mipi-csi2/sun6i_mipi_csi2.c:193 sun6i_mipi_csi2_s_stream() warn: missing error code 'ret'
	../drivers/media/platform/sunxi/sun8i-a83t-mipi-csi2/sun8i_a83t_mipi_csi2.c:225 sun8i_a83t_mipi_csi2_s_stream() warn: missing error code 'ret'
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2344 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: 3000020Kb sm_state_count = 1724941
	../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 68 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/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:2846 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0005-media-subdev-Add-v4l2_subdev_call_state_try-macro.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:5121 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/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:678 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 674)
	../drivers/media/platform/sunxi/sun6i-mipi-csi2/sun6i_mipi_csi2.c:193 sun6i_mipi_csi2_s_stream() warn: missing error code 'ret'
	../drivers/media/platform/sunxi/sun8i-a83t-mipi-csi2/sun8i_a83t_mipi_csi2.c:225 sun8i_a83t_mipi_csi2_s_stream() warn: missing error code 'ret'
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2328 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 = 1724941
	../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 70 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:2831 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

   checkpatch.pl:
	$ cat patches/0005-media-subdev-Add-v4l2_subdev_call_state_try-macro.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:54: CHECK: Macro argument reuse 'sd' - possible side-effects?

patches/0008-media-entity-Add-iterator-for-entity-data-links.patch:

    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:5143 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/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:678 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 674)
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2362 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 = 1724887
	../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 68 seconds
	../drivers/media/dvb-frontends/mb86a16.c: ../drivers/media/dvb-frontends/mb86a16.c:1471 mb86a16_set_fe() 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/platform/sunxi/sun6i-mipi-csi2/sun6i_mipi_csi2.c:193 sun6i_mipi_csi2_s_stream() warn: missing error code 'ret'
	../drivers/media/platform/sunxi/sun8i-a83t-mipi-csi2/sun8i_a83t_mipi_csi2.c:225 sun8i_a83t_mipi_csi2_s_stream() warn: missing error code 'ret'
	../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

   checkpatch.pl:
	$ cat patches/0008-media-entity-Add-iterator-for-entity-data-links.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:81: CHECK: Macro argument reuse 'entity' - possible side-effects?
	-:81: CHECK: Macro argument reuse 'link' - possible side-effects?

patches/0009-media-entity-Use-dedicated-data-link-iterator.patch:

   checkpatch.pl:
	$ cat patches/0009-media-entity-Use-dedicated-data-link-iterator.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:8: WARNING: Possible repeated word: 'that'

patches/0013-media-ov5693-add-support-for-acpi-clock-frequency-pr.patch:

   checkpatch.pl:
	$ cat patches/0013-media-ov5693-add-support-for-acpi-clock-frequency-pr.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:39: CHECK: Alignment should match open parenthesis


Error #512 when building PDF docs