mbox series

[GIT,PULL,v2,for,5.12] More V4L2 patches

Message ID 20210203213507.GA3@valkosipuli.retiisi.org.uk (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,v2,for,5.12] More V4L2 patches | expand

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/for-5.12-5.1-signed

Message

Sakari Ailus Feb. 3, 2021, 9:35 p.m. UTC
Hi Mauro,

This pull request contains quite a bunch of different patches, but to name
a few, there are:

- Ezequiel's V4L2 async cleanup and V4L2 clock removal serieses,
- RDACM21 camera module driver,
- Imx258 raw camera driver,
- Nice improvement for V4L2 async subdev struct allocation by Laurent and
- Random fixes here and there.

Since v1:

- Add IMX334 driver + DT bindings
- Add ov8856 driver fix
- Add LINK_FREQ to IMX219 driver
- Fix vflip/hflip in ov5675 driver
- Dropped a minor documentation fix from me, merged with another bigger
  patch not yet in this pull request

Please pull.


The following changes since commit 0b9112a58836ad6a7e84eebec06a2de9778b7573:

  Merge tag 'v5.11-rc6' into patchwork (2021-02-01 10:03:45 +0100)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.12-5.1-signed

for you to fetch changes up to 6f20874cd363061c0f6cab3442ed914147bf6382:

  media: i2c: Add imx334 camera sensor driver (2021-02-03 22:35:24 +0200)

----------------------------------------------------------------
More V4L2 patches for 5.12

----------------------------------------------------------------
Andrey Konovalov (1):
      media: i2c: imx219: Implement V4L2_CID_LINK_FREQ control

Colin Ian King (1):
      media: i2c: fix spelling mistakes: "enpoint" -> "endpoint"

Dan Carpenter (1):
      media: i2c: ov5648: remove unnecessary NULL check

Ezequiel Garcia (19):
      media: v4l2-async: Remove V4L2_ASYNC_MATCH_DEVNAME
      media: v4l2-async: Clean v4l2_async_notifier_add_fwnode_remote_subdev
      media: atmel: Use v4l2_async_notifier_add_fwnode_remote_subdev
      media: stm32: Use v4l2_async_notifier_add_fwnode_remote_subdev
      media: exynos4-is: Use v4l2_async_notifier_add_fwnode_remote_subdev
      media: st-mipid02: Use v4l2_async_notifier_add_fwnode_remote_subdev
      media: cadence: Use v4l2_async_notifier_add_fwnode_remote_subdev
      media: marvell-ccic: Use v4l2_async_notifier_add_*_subdev
      media: renesas-ceu: Use v4l2_async_notifier_add_*_subdev
      media: pxa-camera: Use v4l2_async_notifier_add_*_subdev
      media: davinci: vpif_display: Remove unused v4l2-async code
      media: v4l2-async: Discourage use of v4l2_async_notifier_add_subdev
      media: Clarify v4l2-async subdevice addition API
      media: mach-pxa: Register the camera sensor fixed-rate clock
      media: pxa_camera: Drop the v4l2-clk clock register
      media: ov9640: Use the generic clock framework
      media: mt9m111: Use the generic clock framework
      media: ov6650: Use the generic clock framework
      media: Remove the legacy v4l2-clk API

Jacopo Mondi (5):
      media: i2c: Add driver for RDACM21 camera module
      dt-bindings: media: max9286: Document 'maxim,reverse-channel-microvolt'
      media: i2c: max9286: Break-out reverse channel setup
      media: i2c: max9286: Make channel amplitude programmable
      media: i2c: max9286: Configure reverse channel amplitude

Krzysztof Kozlowski (4):
      dt-bindings: media: imx258: add bindings for IMX258 sensor
      media: i2c: imx258: add support for binding via device tree
      media: i2c: imx258: simplify getting state container
      media: i2c: imx258: get clock from device properties and enable it via runtime PM

Laurent Pinchart (1):
      media: v4l2-async: Improve v4l2_async_notifier_add_*_subdev() API

Lubomir Rintel (1):
      media: marvell-ccic: power up the device on mclk enable

Martina Krasteva (2):
      dt-bindings: media: Add bindings for imx334
      media: i2c: Add imx334 camera sensor driver

Ricardo Ribalda (2):
      media: ipu3-cio2: Do not zero reserved fields
      media: staging/intel-ipu3: Do not zero reserved fields

Rikard Falkeborn (1):
      media: i2c: rdacm20: Constify static structs

Robert Foss (1):
      media: ov8856: Configure sensor for GRBG Bayer for all modes

Sakari Ailus (2):
      dt-bindings: Remove old ov5647.yaml file, update ovti,ov5647.yaml
      ipu3-cio2: Build bridge only if ACPI is enabled

Shawn Tu (1):
      ov5675: fix vflip/hflip control

Tomi Valkeinen (1):
      media: i2c: max9286: fix access to unallocated memory

 .../devicetree/bindings/media/i2c/imx258.yaml      |  134 +++
 .../bindings/media/i2c/maxim,max9286.yaml          |   22 +
 .../devicetree/bindings/media/i2c/ov5647.yaml      |   76 --
 .../devicetree/bindings/media/i2c/ovti,ov5647.yaml |   20 +-
 .../devicetree/bindings/media/i2c/sony,imx334.yaml |   90 ++
 Documentation/driver-api/media/v4l2-clocks.rst     |   31 -
 Documentation/driver-api/media/v4l2-core.rst       |    1 -
 Documentation/driver-api/media/v4l2-subdev.rst     |   48 +-
 MAINTAINERS                                        |   22 +
 arch/arm/mach-pxa/devices.c                        |    8 +
 drivers/media/i2c/Kconfig                          |   27 +
 drivers/media/i2c/Makefile                         |    3 +
 drivers/media/i2c/imx219.c                         |   15 +-
 drivers/media/i2c/imx258.c                         |   82 +-
 drivers/media/i2c/imx334.c                         | 1131 ++++++++++++++++++++
 drivers/media/i2c/max9286.c                        |   74 +-
 drivers/media/i2c/mt9m111.c                        |   17 +-
 drivers/media/i2c/ov5648.c                         |    4 +-
 drivers/media/i2c/ov5675.c                         |    6 +-
 drivers/media/i2c/ov6650.c                         |   28 +-
 drivers/media/i2c/ov8856.c                         |    4 +-
 drivers/media/i2c/ov8865.c                         |    2 +-
 drivers/media/i2c/ov9640.c                         |   15 +-
 drivers/media/i2c/ov9640.h                         |    2 +-
 drivers/media/i2c/rdacm20.c                        |    4 +-
 drivers/media/i2c/rdacm21.c                        |  623 +++++++++++
 drivers/media/i2c/st-mipid02.c                     |   21 +-
 drivers/media/pci/intel/ipu3/Kconfig               |    2 +-
 drivers/media/pci/intel/ipu3/ipu3-cio2-main.c      |   18 +-
 drivers/media/platform/am437x/am437x-vpfe.c        |    2 +-
 drivers/media/platform/atmel/atmel-isc.h           |    1 +
 drivers/media/platform/atmel/atmel-isi.c           |   46 +-
 drivers/media/platform/atmel/atmel-sama5d2-isc.c   |   44 +-
 drivers/media/platform/cadence/cdns-csi2rx.c       |   17 +-
 drivers/media/platform/davinci/vpif_capture.c      |    2 +-
 drivers/media/platform/davinci/vpif_display.c      |   86 +-
 drivers/media/platform/davinci/vpif_display.h      |    1 -
 drivers/media/platform/exynos4-is/media-dev.c      |   25 +-
 drivers/media/platform/exynos4-is/media-dev.h      |    2 +-
 drivers/media/platform/marvell-ccic/cafe-driver.c  |   14 +-
 drivers/media/platform/marvell-ccic/mcam-core.c    |   12 +-
 drivers/media/platform/marvell-ccic/mcam-core.h    |    1 -
 drivers/media/platform/marvell-ccic/mmp-driver.c   |   11 +-
 drivers/media/platform/omap3isp/isp.c              |   74 +-
 drivers/media/platform/pxa_camera.c                |   83 +-
 drivers/media/platform/qcom/camss/camss.c          |   11 +-
 drivers/media/platform/rcar-vin/rcar-core.c        |    5 +-
 drivers/media/platform/rcar-vin/rcar-csi2.c        |    2 +-
 drivers/media/platform/rcar_drif.c                 |    2 +-
 drivers/media/platform/renesas-ceu.c               |   56 +-
 .../media/platform/rockchip/rkisp1/rkisp1-dev.c    |   15 +-
 drivers/media/platform/stm32/stm32-dcmi.c          |   87 +-
 drivers/media/platform/sunxi/sun4i-csi/sun4i_csi.c |    9 +-
 drivers/media/platform/sunxi/sun4i-csi/sun4i_csi.h |    1 -
 drivers/media/platform/ti-vpe/cal.c                |   12 +-
 drivers/media/platform/video-mux.c                 |   14 +-
 drivers/media/platform/xilinx/xilinx-vipp.c        |   10 +-
 drivers/media/v4l2-core/Makefile                   |    2 +-
 drivers/media/v4l2-core/v4l2-async.c               |   95 +-
 drivers/media/v4l2-core/v4l2-clk.c                 |  321 ------
 drivers/media/v4l2-core/v4l2-fwnode.c              |    6 +-
 drivers/staging/media/imx/imx-media-csi.c          |   14 +-
 drivers/staging/media/imx/imx-media-of.c           |    2 +-
 drivers/staging/media/imx/imx6-mipi-csi2.c         |   19 +-
 drivers/staging/media/imx/imx7-media-csi.c         |   16 +-
 drivers/staging/media/imx/imx7-mipi-csis.c         |   15 +-
 drivers/staging/media/ipu3/ipu3-v4l2.c             |    3 -
 drivers/staging/media/tegra-video/vi.c             |   10 +-
 include/media/davinci/vpif_types.h                 |    2 -
 include/media/v4l2-async.h                         |   89 +-
 include/media/v4l2-clk.h                           |   73 --
 71 files changed, 2619 insertions(+), 1223 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/imx258.yaml
 delete mode 100644 Documentation/devicetree/bindings/media/i2c/ov5647.yaml
 create mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml
 delete mode 100644 Documentation/driver-api/media/v4l2-clocks.rst
 create mode 100644 drivers/media/i2c/imx334.c
 create mode 100644 drivers/media/i2c/rdacm21.c
 delete mode 100644 drivers/media/v4l2-core/v4l2-clk.c
 delete mode 100644 include/media/v4l2-clk.h

Comments

Jenkins Feb. 3, 2021, 9:57 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20210203213507.GA3@valkosipuli.retiisi.org.uk/
Build log: https://builder.linuxtv.org/job/patchwork/88678/
Build time: 00:16:05
Link: https://lore.kernel.org/linux-media/20210203213507.GA3@valkosipuli.retiisi.org.uk

gpg: Signature made Wed 03 Feb 2021 09:30:39 PM UTC
gpg:                using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg:                issuer "sakari.ailus@linux.intel.com"
gpg: Can't check signature: No public key

Summary: got 17/43 patches with issues, being 0 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0007-media-v4l2-async-Clean-v4l2_async_notifier_add_fwnod.patch:

   checkpatch.pl:
	$ cat patches/0007-media-v4l2-async-Clean-v4l2_async_notifier_add_fwnod.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:58: CHECK: Lines should not end with a '('
	-:132: CHECK: Lines should not end with a '('
	-:179: CHECK: Lines should not end with a '('
	-:230: CHECK: Alignment should match open parenthesis
	-:313: CHECK: Lines should not end with a '('
	-:384: CHECK: Lines should not end with a '('
	-:424: CHECK: Lines should not end with a '('
	-:467: CHECK: Lines should not end with a '('
	-:499: CHECK: Lines should not end with a '('

patches/0008-media-atmel-Use-v4l2_async_notifier_add_fwnode_remot.patch:

   checkpatch.pl:
	$ cat patches/0008-media-atmel-Use-v4l2_async_notifier_add_fwnode_remot.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:96: CHECK: Lines should not end with a '('
	-:188: CHECK: Lines should not end with a '('

patches/0009-media-stm32-Use-v4l2_async_notifier_add_fwnode_remot.patch:

   checkpatch.pl:
	$ cat patches/0009-media-stm32-Use-v4l2_async_notifier_add_fwnode_remot.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:238: CHECK: Lines should not end with a '('

patches/0010-media-exynos4-is-Use-v4l2_async_notifier_add_fwnode_.patch:

   checkpatch.pl:
	$ cat patches/0010-media-exynos4-is-Use-v4l2_async_notifier_add_fwnode_.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:69: CHECK: Lines should not end with a '('

patches/0011-media-st-mipid02-Use-v4l2_async_notifier_add_fwnode_.patch:

   checkpatch.pl:
	$ cat patches/0011-media-st-mipid02-Use-v4l2_async_notifier_add_fwnode_.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:53: CHECK: Lines should not end with a '('

patches/0013-media-marvell-ccic-Use-v4l2_async_notifier_add_-_sub.patch:

   checkpatch.pl:
	$ cat patches/0013-media-marvell-ccic-Use-v4l2_async_notifier_add_-_sub.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:51: CHECK: Alignment should match open parenthesis

patches/0014-media-renesas-ceu-Use-v4l2_async_notifier_add_-_subd.patch:

   checkpatch.pl:
	$ cat patches/0014-media-renesas-ceu-Use-v4l2_async_notifier_add_-_subd.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:116: CHECK: Alignment should match open parenthesis
	-:156: CHECK: Lines should not end with a '('

patches/0015-media-pxa-camera-Use-v4l2_async_notifier_add_-_subde.patch:

   checkpatch.pl:
	$ cat patches/0015-media-pxa-camera-Use-v4l2_async_notifier_add_-_subde.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:63: CHECK: Lines should not end with a '('
	-:89: CHECK: Lines should not end with a '('

patches/0017-media-v4l2-async-Discourage-use-of-v4l2_async_notifi.patch:

   checkpatch.pl:
	$ cat patches/0017-media-v4l2-async-Discourage-use-of-v4l2_async_notifi.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:35: CHECK: Alignment should match open parenthesis
	-:104: CHECK: Alignment should match open parenthesis

patches/0018-media-v4l2-async-Improve-v4l2_async_notifier_add_-_s.patch:

   checkpatch.pl:
	$ cat patches/0018-media-v4l2-async-Improve-v4l2_async_notifier_add_-_s.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:124: CHECK: Lines should not end with a '('
	-:263: CHECK: Lines should not end with a '('
	-:280: CHECK: Lines should not end with a '('
	-:326: CHECK: Lines should not end with a '('
	-:410: CHECK: Alignment should match open parenthesis
	-:437: CHECK: Lines should not end with a '('
	-:474: CHECK: Alignment should match open parenthesis
	-:577: CHECK: Lines should not end with a '('
	-:770: CHECK: Alignment should match open parenthesis
	-:790: CHECK: Macro argument '__type' may be better as '(__type)' to avoid precedence issues
	-:806: CHECK: Macro argument '__type' may be better as '(__type)' to avoid precedence issues
	-:822: CHECK: Macro argument '__type' may be better as '(__type)' to avoid precedence issues

patches/0020-dt-bindings-Remove-old-ov5647.yaml-file-update-ovti-.patch:

   checkpatch.pl:
	$ cat patches/0020-dt-bindings-Remove-old-ov5647.yaml-file-update-ovti-.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:20: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:23: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
	-:101: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst

patches/0032-media-mach-pxa-Register-the-camera-sensor-fixed-rate.patch:

   checkpatch.pl:
	$ cat patches/0032-media-mach-pxa-Register-the-camera-sensor-fixed-rate.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:41: CHECK: Alignment should match open parenthesis

patches/0034-media-ov9640-Use-the-generic-clock-framework.patch:

   checkpatch.pl:
	$ cat patches/0034-media-ov9640-Use-the-generic-clock-framework.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:6: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)

patches/0036-media-ov6650-Use-the-generic-clock-framework.patch:

   checkpatch.pl:
	$ cat patches/0036-media-ov6650-Use-the-generic-clock-framework.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:6: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)

patches/0037-media-Remove-the-legacy-v4l2-clk-API.patch:

   checkpatch.pl:
	$ cat patches/0037-media-Remove-the-legacy-v4l2-clk-API.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:33: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0041-media-ov8856-Configure-sensor-for-GRBG-Bayer-for-all.patch:

   checkpatch.pl:
	$ cat patches/0041-media-ov8856-Configure-sensor-for-GRBG-Bayer-for-all.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:20: WARNING: Use a single space after To:
	-:20: ERROR: Unrecognized email address: ''
	-:57: ERROR: Missing Signed-off-by: line by nominal patch author ''

patches/0043-media-i2c-Add-imx334-camera-sensor-driver.patch:

   checkpatch.pl:
	$ cat patches/0043-media-i2c-Add-imx334-camera-sensor-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:228: CHECK: struct mutex definition without comment


Error #512 when building PDF docs