| Message ID | 20210202204305.GE3@valkosipuli.retiisi.org.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,for,5.12] More V4L2 patches | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20210202204305.GE3@valkosipuli.retiisi.org.uk/
Build log: https://builder.linuxtv.org/job/patchwork/88530/
Build time: 00:28:02
Link: https://lore.kernel.org/linux-media/20210202204305.GE3@valkosipuli.retiisi.org.uk
gpg: Signature made Tue 02 Feb 2021 08:37:35 PM UTC
gpg: using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg: issuer "sakari.ailus@linux.intel.com"
gpg: Can't check signature: No public key
Summary: got 15/40 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
-:123: CHECK: Lines should not end with a '('
-:262: CHECK: Lines should not end with a '('
-:279: CHECK: Lines should not end with a '('
-:325: CHECK: Lines should not end with a '('
-:409: CHECK: Alignment should match open parenthesis
-:436: CHECK: Lines should not end with a '('
-:473: CHECK: Alignment should match open parenthesis
-:576: CHECK: Lines should not end with a '('
-:769: CHECK: Alignment should match open parenthesis
-:789: CHECK: Macro argument '__type' may be better as '(__type)' to avoid precedence issues
-:805: CHECK: Macro argument '__type' may be better as '(__type)' to avoid precedence issues
-:821: 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/0033-media-mach-pxa-Register-the-camera-sensor-fixed-rate.patch:
checkpatch.pl:
$ cat patches/0033-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/0035-media-ov9640-Use-the-generic-clock-framework.patch:
checkpatch.pl:
$ cat patches/0035-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/0037-media-ov6650-Use-the-generic-clock-framework.patch:
checkpatch.pl:
$ cat patches/0037-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/0038-media-Remove-the-legacy-v4l2-clk-API.patch:
checkpatch.pl:
$ cat patches/0038-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?
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. Please pull. The following changes since commit f0ddb4e9911665b9ad68fe94e0faaaff5953902e: media: venus: venc: set IDR period to FW only for H264 & HEVC (2021-01-27 16:22:36 +0100) are available in the Git repository at: git://linuxtv.org/sailus/media_tree.git tags/for-5.12-5-signed for you to fetch changes up to 294bac0c7d79761440dfb1aeaf17186a8ccf3208: ov5675: fix vflip/hflip control (2021-02-02 22:28:48 +0200) ---------------------------------------------------------------- V4L2 patches for 5.12 ---------------------------------------------------------------- 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 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 Sakari Ailus (3): dt-bindings: Remove old ov5647.yaml file, update ovti,ov5647.yaml Documentation: media: camera-sensor: The unit of hblank is pixels 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 +- Documentation/driver-api/media/camera-sensor.rst | 7 +- 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 | 13 + arch/arm/mach-pxa/devices.c | 8 + drivers/media/i2c/Kconfig | 13 + drivers/media/i2c/Makefile | 2 + drivers/media/i2c/imx258.c | 82 ++- 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/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 --- 68 files changed, 1362 insertions(+), 1223 deletions(-) create mode 100644 Documentation/devicetree/bindings/media/i2c/imx258.yaml delete mode 100644 Documentation/devicetree/bindings/media/i2c/ov5647.yaml delete mode 100644 Documentation/driver-api/media/v4l2-clocks.rst 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