mbox series

[GIT,PULL,v3,FOR,5.18] V4L2 patches

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

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/for-5.18-1.2-signed

Message

Sakari Ailus Feb. 2, 2022, 2:22 p.m. UTC
Hi Mauro,

Here's a bunch of patches again for 5.18. Most notably there's V4L2 fwnode
/ mbus_config cleanup by Laurent, the hi847 camera sensor driver from Shawn
Tu and the od08d10 camera sensor driver by Jimmy Su. Fixes elsewhere are
included, too.

Since v1, a few more patches have been added and I've dropped a camss patch
already picked by Hans.

Since v2, I've added a new patch to prepare the ov6650 driver for
get_mbus_config() changes by Laurent. Also other fixes have been added,
effectively the rest of the patches. This is now on the media stage tree as
it depends on other patches there.

Please pull.


The following changes since commit 68a99f6a0ebfe9101ea79ba5af1c407a5ad4f629:

  media: lirc: report ir receiver overflow (2022-01-28 19:32:50 +0100)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.18-1.2-signed

for you to fetch changes up to 63e1437181c45a2b74d2e1f2c852381f0ccff20b:

  media: i2c: ov08d10: Unlock on error in ov08d10_enum_frame_size() (2022-02-02 15:44:33 +0200)

----------------------------------------------------------------
More V4L2 camera patches

----------------------------------------------------------------
Bingbu Cao (1):
      media: ov5675: use group write to update digital gain

Dan Carpenter (1):
      media: i2c: ov08d10: Unlock on error in ov08d10_enum_frame_size()

Kees Cook (1):
      media: omap3isp: Use struct_group() for memcpy() region

Laurent Pinchart (4):
      media: v4l2-mediabus: Use structures to describe bus configuration
      media: v4l2-mediabus: Drop legacy V4L2_MBUS_CSI2_*_LANE flags
      media: v4l2-mediabus: Drop legacy V4L2_MBUS_CSI2_CHANNEL_* flags
      media: v4l2-mediabus: Drop V4L2_MBUS_CSI2_CONTINUOUS_CLOCK flag

Martin Kepplinger (2):
      dt-binding: media: hynix,hi846: use $defs/port-base port description
      dt-bindings: media: hynix,hi846: add link-frequencies description

Sakari Ailus (1):
      microchip-csi2dc: Remove VC support for now

 .../devicetree/bindings/media/i2c/hynix,hi846.yaml |  6 ++-
 drivers/gpu/ipu-v3/ipu-csi.c                       |  6 +--
 drivers/media/i2c/adv7180.c                        | 10 ++---
 drivers/media/i2c/adv748x/adv748x-csi2.c           | 18 +--------
 drivers/media/i2c/ml86v7667.c                      |  5 ++-
 drivers/media/i2c/mt9m001.c                        |  8 ++--
 drivers/media/i2c/mt9m111.c                        | 14 ++++---
 drivers/media/i2c/ov08d10.c                        |  4 +-
 drivers/media/i2c/ov5675.c                         | 32 ++++++++++++++--
 drivers/media/i2c/ov6650.c                         | 14 +++----
 drivers/media/i2c/ov9640.c                         |  8 ++--
 drivers/media/i2c/tc358743.c                       | 24 ++----------
 drivers/media/i2c/tvp5150.c                        |  6 ++-
 drivers/media/platform/atmel/microchip-csi2dc.c    | 20 ++--------
 drivers/media/platform/omap3isp/ispstat.c          |  5 ++-
 drivers/media/platform/pxa_camera.c                | 12 +++---
 drivers/media/platform/rcar-vin/rcar-csi2.c        | 16 ++------
 drivers/media/v4l2-core/v4l2-fwnode.c              |  6 +--
 drivers/staging/media/imx/imx-media-csi.c          |  7 ++--
 drivers/staging/media/imx/imx6-mipi-csi2.c         | 25 ++----------
 include/media/v4l2-mediabus.h                      | 44 +++++++++-------------
 include/uapi/linux/omap3isp.h                      | 21 +++++++----
 22 files changed, 138 insertions(+), 173 deletions(-)

Comments

Jenkins Feb. 2, 2022, 3:48 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/YfqTl5GBCDRDLXl1@valkosipuli.retiisi.eu/
Build log: https://builder.linuxtv.org/job/patchwork/179449/
Build time: 00:13:11
Link: https://lore.kernel.org/linux-media/YfqTl5GBCDRDLXl1@valkosipuli.retiisi.eu

gpg: Signature made Wed 02 Feb 2022 01:49:23 PM UTC
gpg:                using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg:                issuer "sakari.ailus@linux.intel.com"
gpg: Good signature from "Sakari Ailus <sakari.ailus@linux.intel.com>" [unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: F0D0 377A 0D4F 25A7 9238  EFE5 6D40 361B 6E28 C193
     Subkey fingerprint: 53AC 58A5 F594 8636 C04A  1BF8 141D FA54 A1EC 8DEA

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

Error/warnings:

patches/0001-microchip-csi2dc-Remove-VC-support-for-now.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
	../drivers/media/i2c/ov08d10.c: ../drivers/media/i2c/ov08d10.c:1276 ov08d10_enum_frame_size() warn: inconsistent returns '&ov08d10->mutex'.
	  Locked on  : 1268
	  Unlocked on: 1276
	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/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)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2900 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0002-media-v4l2-mediabus-Use-structures-to-describe-bus-c.patch:

    allyesconfig: return code #0:
	../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/i2c/ov08d10.c: ../drivers/media/i2c/ov08d10.c:1276 ov08d10_enum_frame_size() warn: inconsistent returns '&ov08d10->mutex'.
	  Locked on  : 1268
	  Unlocked on: 1276
	../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)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2888 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0003-media-v4l2-mediabus-Drop-legacy-V4L2_MBUS_CSI2_-_LAN.patch:

    allyesconfig: return code #0:
	../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/i2c/ov08d10.c: ../drivers/media/i2c/ov08d10.c:1276 ov08d10_enum_frame_size() warn: inconsistent returns '&ov08d10->mutex'.
	  Locked on  : 1268
	  Unlocked on: 1276
	../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)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2900 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0004-media-v4l2-mediabus-Drop-legacy-V4L2_MBUS_CSI2_CHANN.patch:

    allyesconfig: return code #0:
	../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)
	../drivers/media/i2c/ov08d10.c: ../drivers/media/i2c/ov08d10.c:1276 ov08d10_enum_frame_size() warn: inconsistent returns '&ov08d10->mutex'.
	  Locked on  : 1268
	  Unlocked on: 1276
	../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/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2900 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0005-media-v4l2-mediabus-Drop-V4L2_MBUS_CSI2_CONTINUOUS_C.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov08d10.c: ../drivers/media/i2c/ov08d10.c:1276 ov08d10_enum_frame_size() warn: inconsistent returns '&ov08d10->mutex'.
	  Locked on  : 1268
	  Unlocked on: 1276
	../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)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2894 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0006-media-omap3isp-Use-struct_group-for-memcpy-region.patch:

   checkpatch.pl:
	$ cat patches/0006-media-omap3isp-Use-struct_group-for-memcpy-region.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:30: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
	-:105: CHECK: Alignment should match open parenthesis
	-:122: CHECK: Alignment should match open parenthesis

patches/0007-dt-binding-media-hynix-hi846-use-defs-port-base-port.patch:

   checkpatch.pl:
	$ cat patches/0007-dt-binding-media-hynix-hi846-use-defs-port-base-port.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:8: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)


Error #512 when building PDF docs