mbox series

[GIT,PULL,FOR,5.17] V4L2 patches

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

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/for-5.17-1-signed

Message

Sakari Ailus Nov. 24, 2021, 3:11 p.m. UTC
Hi Mauro,

Here are a bunch of V4L2 patches for 5.17.

In particular, this includes support for the ov5693 sensor, ACPI support
for the ov8865 driver as well as improvements to the imx274 driver.

Please pull.


The following changes since commit 0d2517b3765aa331aee0a95f9b8072062d6bb7e5:

  media: hantro: Support NV12 on the G2 core (2021-11-22 07:53:37 +0000)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.17-1-signed

for you to fetch changes up to bd697d6a91d20cd05860f14f672bd7e8237d4541:

  dt-bindings: media: renesas,jpu: Convert to json-schema (2021-11-24 11:15:14 +0200)

----------------------------------------------------------------
V4L2 patches for 5.17

----------------------------------------------------------------
Daniel Scally (19):
      media: ipu3-cio2: Toggle sensor streaming in pm runtime ops
      media: i2c: Add support for ov5693 sensor
      media: ipu3-cio2: Add link freq for INT33BE entry
      media: i2c: Re-order runtime pm initialisation
      media: i2c: Add ACPI support to ov8865
      media: i2c: Fix incorrect value in comment
      media: i2c: Defer probe if not endpoint found
      media: i2c: Support 19.2MHz input clock in ov8865
      media: i2c: Add .get_selection() support to ov8865
      media: i2c: Switch control to V4L2_CID_ANALOGUE_GAIN
      media: i2c: Add vblank control to ov8865
      media: i2c: Add hblank control to ov8865
      media: i2c: Update HTS values in ov8865
      media: i2c: cap exposure at height + vblank in ov8865
      media: i2c: Add controls from fwnode to ov8865
      media: i2c: Switch exposure control unit to lines
      media: i2c: Use dev_err_probe() in ov8865
      media: i2c: Fix max gain in ov8865
      media: ipu3-cio2: Add INT347A to cio2-bridge

Eugen Hristev (6):
      media: i2c: imx274: fix s_frame_interval runtime resume not requested
      media: i2c: imx274: fix trivial typo expsoure/exposure
      media: i2c: imx274: fix trivial typo obainted/obtained
      media: i2c: imx274: implement enum_mbus_code
      media: i2c: imx274: simplify probe function by adding local variable dev
      media: i2c: imx274: implement fwnode parsing

Geert Uytterhoeven (1):
      dt-bindings: media: renesas,jpu: Convert to json-schema

Hans de Goede (1):
      media: i2c: ov8865: Fix lockdep error

Kieran Bingham (1):
      media: i2c: max9286: Depend on VIDEO_V4L2

 .../devicetree/bindings/media/renesas,jpu.txt      |   25 -
 .../devicetree/bindings/media/renesas,jpu.yaml     |   65 +
 MAINTAINERS                                        |    7 +
 drivers/media/i2c/Kconfig                          |   12 +
 drivers/media/i2c/Makefile                         |    1 +
 drivers/media/i2c/imx274.c                         |  102 +-
 drivers/media/i2c/ov5693.c                         | 1537 ++++++++++++++++++++
 drivers/media/i2c/ov8865.c                         |  466 ++++--
 drivers/media/pci/intel/ipu3/cio2-bridge.c         |    4 +-
 drivers/media/pci/intel/ipu3/ipu3-cio2-main.c      |   17 +-
 10 files changed, 2052 insertions(+), 184 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/media/renesas,jpu.txt
 create mode 100644 Documentation/devicetree/bindings/media/renesas,jpu.yaml
 create mode 100644 drivers/media/i2c/ov5693.c

Comments

Jenkins Nov. 24, 2021, 3:45 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/YZ5WGhErn1kv4vlX@valkosipuli.retiisi.eu/
Build log: https://builder.linuxtv.org/job/patchwork/159659/
Build time: 00:28:12
Link: https://lore.kernel.org/linux-media/YZ5WGhErn1kv4vlX@valkosipuli.retiisi.eu

gpg: Signature made Wed 24 Nov 2021 09:15:47 AM 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 14/28 patches with issues, being 11 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-media-i2c-max9286-Depend-on-VIDEO_V4L2.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/cec/core/cec-adap.c: ../drivers/media/cec/core/cec-adap.c:926 cec_transmit_msg_fh() warn: '&data->list' not removed from list
	../drivers/media/rc/meson-ir-tx.c:22: warning: expecting prototype for meson(). Prototype was for DEVICE_NAME() instead
	../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
	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/usb/pvrusb2/pvrusb2-encoder.c: ../drivers/media/usb/pvrusb2/pvrusb2-encoder.c:288 pvr2_encoder_cmd() warn: inconsistent indenting
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:1730 pvr2_hdw_set_streaming() warn: inconsistent indenting
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3461 pvr2_hdw_cpufw_set_enabled() warn: inconsistent indenting
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3501 pvr2_hdw_cpufw_get() warn: inconsistent indenting
	../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: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:2874 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0002-media-i2c-imx274-fix-s_frame_interval-runtime-resume.patch:

   checkpatch.pl:
	$ cat patches/0002-media-i2c-imx274-fix-s_frame_interval-runtime-resume.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:13: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)

patches/0007-media-i2c-Add-support-for-ov5693-sensor.patch:

   checkpatch.pl:
	$ cat patches/0007-media-i2c-Add-support-for-ov5693-sensor.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:1365: CHECK: Lines should not end with a '('
	-:1368: CHECK: spaces preferred around that '-' (ctx:VxV)

patches/0015-media-i2c-Add-.get_selection-support-to-ov8865.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2799 ov8865_get_selection() warn: inconsistent indenting

patches/0016-media-i2c-Switch-control-to-V4L2_CID_ANALOGUE_GAIN.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2800 ov8865_get_selection() warn: inconsistent indenting

patches/0017-media-i2c-Add-vblank-control-to-ov8865.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2832 ov8865_get_selection() warn: inconsistent indenting

patches/0018-media-i2c-Add-hblank-control-to-ov8865.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2846 ov8865_get_selection() warn: inconsistent indenting

patches/0019-media-i2c-Update-HTS-values-in-ov8865.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2804 ov8865_get_selection() warn: inconsistent indenting

patches/0020-media-i2c-cap-exposure-at-height-vblank-in-ov8865.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2828 ov8865_get_selection() warn: inconsistent indenting

patches/0021-media-i2c-Add-controls-from-fwnode-to-ov8865.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2838 ov8865_get_selection() warn: inconsistent indenting

patches/0022-media-i2c-Switch-exposure-control-unit-to-lines.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2841 ov8865_get_selection() warn: inconsistent indenting

patches/0023-media-i2c-Use-dev_err_probe-in-ov8865.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2841 ov8865_get_selection() warn: inconsistent indenting

patches/0024-media-i2c-Fix-max-gain-in-ov8865.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2841 ov8865_get_selection() warn: inconsistent indenting

patches/0028-dt-bindings-media-renesas-jpu-Convert-to-json-schema.patch:

   checkpatch.pl:
	$ cat patches/0028-dt-bindings-media-renesas-jpu-Convert-to-json-schema.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:25: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:28: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
	-:55: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst


Error #512 when building PDF docs