| Message ID | YhdGXZL3xpfld/JQ@valkosipuli.retiisi.eu (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,5.18] More V4L2 patches | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/YhdGXZL3xpfld/JQ@valkosipuli.retiisi.eu/
Build log: https://builder.linuxtv.org/job/patchwork/185642/
Build time: 00:19:23
Link: https://lore.kernel.org/linux-media/YhdGXZL3xpfld/JQ@valkosipuli.retiisi.eu
gpg: Signature made Thu 24 Feb 2022 08:42:09 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 6/12 patches with issues, being 4 at build time, plus one error when buinding PDF document
Error/warnings:
patches/0001-media-ov5640-Fix-set-format-v4l2_mbus_pixelcode-not-.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
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)
patches/0005-media-media-entity-Add-media_pad_is_streaming-helper.patch:
allyesconfig: return code #0:
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:2894 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
patches/0006-media-media-entity-Simplify-media_pipeline_start.patch:
allyesconfig: return code #0:
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)
patches/0007-v4l-fwnode-Drop-redunant-ENODATA-check-in-property-r.patch:
checkpatch.pl:
$ cat patches/0007-v4l-fwnode-Drop-redunant-ENODATA-check-in-property-r.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:7: ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit c343bc2ce2c6 ("ACPI: properties: Align return codes of __acpi_node_get_property_reference()")'
patches/0009-media-dt-bindings-Add-Intersil-ISL79987-DT-bindings.patch:
checkpatch.pl:
$ cat patches/0009-media-dt-bindings-Add-Intersil-ISL79987-DT-bindings.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:24: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0010-media-i2c-isl7998x-Add-driver-for-Intersil-ISL7998x.patch:
allyesconfig: return code #0:
../drivers/media/i2c/isl7998x.c: ../drivers/media/i2c/isl7998x.c:561 isl7998x_norm_to_val() error: buffer overflow 'isl7998x_std_res' 9 <= 9 (assuming for loop doesn't break)
../drivers/media/i2c/isl7998x.c: ../drivers/media/i2c/isl7998x.c:576 isl7998x_norm_to_mode() error: buffer overflow 'isl7998x_std_res' 9 <= 9 (assuming for loop doesn't break)
../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
checkpatch.pl:
$ cat patches/0010-media-i2c-isl7998x-Add-driver-for-Intersil-ISL7998x.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:61: WARNING: please write a help paragraph that fully describes the config symbol
-:1252: ERROR: Macros with complex values should be enclosed in parentheses
-:1252: CHECK: Macro argument reuse 'page' - possible side-effects?
-:1280: ERROR: Macros with complex values should be enclosed in parentheses
-:1280: CHECK: Macro argument reuse 'page' - possible side-effects?
-:1314: ERROR: Macros with complex values should be enclosed in parentheses
-:1314: CHECK: Macro argument reuse 'page' - possible side-effects?
Error #512 when building PDF docs
Hi Mauro, Here's a nice set of mostly camera related patches for 5.18. Worth noting are especially new drivers for isl7998x analogue to CSI-2/BT.656 decoder and og01a1b camera sensor. Also included are cleanups for V4L2 fwnode reference parsing and stream control internals in MC, as well as various driver fixes and cleanups. Please pull. The following changes since commit 2b891d3980f6c255459d0e1c29ce2152ec7cf678: media: xilinx: csi2rxss: Use mipi-csi2.h (2022-02-17 10:32:26 +0100) are available in the Git repository at: git://linuxtv.org/sailus/media_tree.git tags/for-5.18-2.1-signed for you to fetch changes up to ba81a4fce9cdc59d32646d802a91a5164e77321c: media: i2c: imx274: Drop surplus includes (2022-02-23 22:05:19 +0200) ---------------------------------------------------------------- V4L2 patches for 5.18 ---------------------------------------------------------------- Daniel Scally (1): media: i2c: Fix pixel array positions in ov8865 Hans de Goede (1): media: i2c: ov5648: Fix lockdep error Laurent Pinchart (2): media: media-entity: Add media_pad_is_streaming() helper function media: media-entity: Simplify media_pipeline_start() Linus Walleij (2): media: i2c: ccs: Drop unused include media: i2c: imx274: Drop surplus includes Marek Vasut (2): media: dt-bindings: Add Intersil ISL79987 DT bindings media: i2c: isl7998x: Add driver for Intersil ISL7998x Mirela Rabulea (1): media: ov5640: Fix set format, v4l2_mbus_pixelcode not updated Sakari Ailus (2): v4l: fwnode: Drop redunant -ENODATA check in property reference parsing v4l: fwnode: Remove now-redundant loop from v4l2_fwnode_parse_reference() Shawn Tu (1): media: Add a driver for the og01a1b camera sensor .../bindings/media/i2c/isil,isl79987.yaml | 113 ++ MAINTAINERS | 14 + drivers/media/i2c/Kconfig | 23 + drivers/media/i2c/Makefile | 2 + drivers/media/i2c/ccs/ccs-core.c | 1 - drivers/media/i2c/imx274.c | 2 - drivers/media/i2c/isl7998x.c | 1630 ++++++++++++++++++++ drivers/media/i2c/og01a1b.c | 1128 ++++++++++++++ drivers/media/i2c/ov5640.c | 14 +- drivers/media/i2c/ov5648.c | 10 +- drivers/media/i2c/ov8865.c | 4 +- drivers/media/mc/mc-entity.c | 55 +- drivers/media/platform/exynos4-is/common.c | 5 +- drivers/media/platform/exynos4-is/fimc-isp.c | 2 +- drivers/media/platform/exynos4-is/fimc-lite.c | 6 +- drivers/media/platform/rcar-vin/rcar-core.c | 2 +- drivers/media/v4l2-core/v4l2-fwnode.c | 28 +- include/media/media-entity.h | 21 +- include/uapi/linux/v4l2-controls.h | 6 + 19 files changed, 2989 insertions(+), 77 deletions(-) create mode 100644 Documentation/devicetree/bindings/media/i2c/isil,isl79987.yaml create mode 100644 drivers/media/i2c/isl7998x.c create mode 100644 drivers/media/i2c/og01a1b.c