| Message ID | 20210205083847.GE3@valkosipuli.retiisi.org.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,v4,for,5.12] More V4L2 patches | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20210205083847.GE3@valkosipuli.retiisi.org.uk/
Build log: https://builder.linuxtv.org/job/patchwork/88889/
Build time: 00:16:14
Link: https://lore.kernel.org/linux-media/20210205083847.GE3@valkosipuli.retiisi.org.uk
gpg: Signature made Fri 05 Feb 2021 08:36:41 AM 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
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 Since v2: - Add missing assigned-clock-parents property in IMX334 binding example. Since v3: - Add missing semicolon in IMX334 binding example. 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.3-signed for you to fetch changes up to b28f16655455df353760bf01508c5087a2e8bfc5: media: i2c: Add imx334 camera sensor driver (2021-02-05 10:36:32 +0200) ---------------------------------------------------------------- More V4L2 patches! ---------------------------------------------------------------- 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 | 91 ++ 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, 2620 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