| Message ID | ZRwJNg5Ez6iSBINf@valkosipuli.retiisi.eu (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,6.7] Camera sensor patches, preparation for enabling streams API | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/ZRwJNg5Ez6iSBINf@valkosipuli.retiisi.eu/
Build log: https://builder.linuxtv.org/job/patchwork/345037/
Build time: 00:45:51
Link: https://lore.kernel.org/linux-media/ZRwJNg5Ez6iSBINf@valkosipuli.retiisi.eu
gpg: Signature made Tue 03 Oct 2023 12:22:28 PM UTC
gpg: using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg: issuer "sakari.ailus@linux.intel.com"
gpg: Good signature from "Sakari Ailus <sakari.ailus@linux.intel.com>" [full]
Summary: got 4/46 patches with issues, being 2 at build time
Error/warnings:
patches/0001-media-ov9282-Assign-maintenance-to-Dave.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
../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:446 gc0310_s_stream() warn: missing error code 'ret'
../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2779 atomisp_cp_dvs_6axis_config() warn: missing unwind goto?
../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2878 atomisp_cp_morph_table() warn: missing unwind goto?
allyesconfig: return code #0:
../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto?
SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
../drivers/media/usb/dvb-usb-v2/af9035.c: ../drivers/media/usb/dvb-usb-v2/af9035.c:467 af9035_i2c_master_xfer() warn: inconsistent returns '&d->i2c_mutex'.
Locked on : 326,387
Unlocked on: 465,467
../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2792 mxc_jpeg_probe() warn: missing unwind goto?
../drivers/media/i2c/ov5645.c: ../drivers/media/i2c/ov5645.c:687 ov5645_set_power_on() warn: 'ov5645->xclk' from clk_prepare_enable() not released on lines: 687.
../drivers/media/pci/cx25821/cx25821-medusa-video.c: ../drivers/media/pci/cx25821/cx25821-medusa-video.c:399 medusa_set_videostandard() parse error: OOM: 3003988Kb sm_state_count = 2158
../drivers/media/pci/cx25821/cx25821-medusa-video.c: ../drivers/media/pci/cx25821/cx25821-medusa-video.c:399 medusa_set_videostandard() parse error: __split_smt: function too hairy. Giving up after 4 seconds
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2517 dvb_register() parse error: OOM: 3014560Kb sm_state_count = 1757213
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2517 dvb_register() parse error: __split_smt: function too hairy. Giving up after 52 seconds
../drivers/media/usb/uvc/uvc_v4l2.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../include/linux/rcuwait.h, ...):
SPARSE:../drivers/media/usb/uvc/uvc_v4l2.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
../drivers/media/pci/ivtv/ivtvfb.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../drivers/media/pci/ivtv/ivtv-driver.h):
SPARSE:../drivers/media/pci/ivtv/ivtvfb.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
../drivers/media/pci/mgb4/mgb4_sysfs_out.c: ../drivers/media/pci/mgb4/mgb4_sysfs_out.c:118 video_source_store() warn: potential spectre issue 'mgbdev->vin' [r] (local cap)
../drivers/media/pci/mgb4/mgb4_sysfs_out.c: ../drivers/media/pci/mgb4/mgb4_sysfs_out.c:122 video_source_store() warn: possible spectre second half. 'loopin_new'
../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2854 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
patches/0007-media-i2c-Add-driver-for-onsemi-MT9M114-camera-senso.patch:
allyesconfig: return code #0:
../drivers/media/i2c/mt9m114.c: ../drivers/media/i2c/mt9m114.c:2381 mt9m114_probe() warn: missing unwind goto?
checkpatch.pl:
$ cat patches/0007-media-i2c-Add-driver-for-onsemi-MT9M114-camera-senso.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:183: CHECK: Prefer using the BIT macro
-:188: CHECK: Prefer using the BIT macro
-:196: CHECK: Prefer using the BIT macro
-:199: CHECK: Prefer using the BIT macro
-:225: CHECK: Prefer using the BIT macro
-:230: CHECK: Prefer using the BIT macro
-:235: CHECK: Prefer using the BIT macro
-:245: CHECK: Prefer using the BIT macro
-:348: CHECK: Prefer using the BIT macro
patches/0036-media-ccs-Use-sub-device-active-state.patch:
checkpatch.pl:
$ cat patches/0036-media-ccs-Use-sub-device-active-state.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:416: CHECK: multiple assignments should be avoided
patches/0046-media-Documentation-Split-camera-sensor-documentatio.patch:
checkpatch.pl:
$ cat patches/0046-media-Documentation-Split-camera-sensor-documentatio.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:194: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
Hi Mauro, Hans, This set contains cleanups and improvements for the imx219 driver from Laurent, additional mode for the hi556 driver from Bingbu, maintainer changes for some Intel-written sensor drivers as well as preparation for generic line based metadata support, including moving CCS and ov2740 drivers to use sub-device state. Please pull. The following changes since commit 1aa687a41c6b1ca28b45f230eb7ad2d3d6d35bc6: media: qcom: camss: Comment CSID dt_id field (2023-09-28 09:41:54 +0200) are available in the Git repository at: git://linuxtv.org/sailus/media_tree.git tags/for-6.7-2-signed for you to fetch changes up to 64079722be862e4e31b15f75503dee0315aca84e: media: Documentation: Split camera sensor documentation (2023-10-03 14:43:04 +0300) ---------------------------------------------------------------- V4L2 patches for 6.7 ---------------------------------------------------------------- Bingbu Cao (1): media: i2c/hi556: add a new mode 1296x722 settings Laurent Pinchart (22): media: dt-bindings: media: i2c: Add MT9M114 camera sensor binding media: i2c: Add driver for onsemi MT9M114 camera sensor media: i2c: imx219: Convert to CCI register access helpers media: i2c: imx219: Drop unused macros media: i2c: imx219: Replace register addresses with macros media: i2c: imx219: Drop IMX219_REG_CSI_LANE_MODE from common regs array media: i2c: imx219: Fix test pattern window for 640x480 mode media: i2c: imx219: Set mode registers programmatically media: i2c: imx219: Merge format and binning setting functions media: i2c: imx219: Initialize ycbcr_enc media: i2c: imx219: Use active crop rectangle to configure registers media: i2c: imx219: Infer binning settings from format and crop media: i2c: imx219: Access height from active format in imx219_set_ctrl media: i2c: imx219: Don't store the current mode in the imx219 structure media: i2c: imx219: Drop IMX219_VTS_* macros media: i2c: imx219: Group functions by purpose media: i2c: imx219: Remove unneeded goto media: i2c: imx219: Implement .init_cfg() using .set_fmt() media: i2c: imx219: Separate horizontal and vertical binning media: i2c: imx219: Calculate crop rectangle dynamically media: i2c: imx219: Name all subdev state variables 'state' media: i2c: imx219: Move variables to inner scope Sakari Ailus (23): media: ov9282: Assign maintenance to Dave media: imx412: Orphan the driver media: imx335: Orphan the driver media: imx334: Orphan the driver media: Documentation: Align numbered list, make it a proper ReST media: ccs: Fix driver quirk struct documentation media: ccs: Correctly initialise try compose rectangle media: ccs: Correct error handling in ccs_register_subdev media: ccs: Switch to init_cfg media: ccs: Rename ccs_create_subdev as ccs_init_subdev media: ccs: Move media_entity_pads_init to init from register media: ccs: Obtain media bus formats before initialising up sub-devices media: ccs: Use sub-device active state media: ccs: Partially revert "media: i2c: Use pm_runtime_resume_and_get()" media: ccs: Drop re-entrant s_stream support media: ov2740: Enable runtime PM before registering the async subdev media: ov2740: Use sub-device active state media: ov2740: Return -EPROBE_DEFER if no endpoint is found media: v4l: subdev: Clear frame descriptor before get_frame_desc media: v4l: subdev: Print debug information on frame descriptor media: mc: Check pad flag validity media: Add MIPI CSI-2 generic long packet type definition media: Documentation: Split camera sensor documentation .../bindings/media/i2c/onnn,mt9m114.yaml | 114 + Documentation/driver-api/media/camera-sensor.rst | 131 +- .../userspace-api/media/drivers/camera-sensor.rst | 104 + .../userspace-api/media/drivers/index.rst | 1 + Documentation/userspace-api/media/v4l/control.rst | 4 + .../userspace-api/media/v4l/dev-subdev.rst | 49 +- MAINTAINERS | 23 +- drivers/media/i2c/Kconfig | 11 + drivers/media/i2c/Makefile | 1 + drivers/media/i2c/ccs/ccs-core.c | 374 ++- drivers/media/i2c/ccs/ccs-quirk.h | 4 +- drivers/media/i2c/ccs/ccs.h | 4 +- drivers/media/i2c/ds90ub913.c | 2 - drivers/media/i2c/ds90ub953.c | 2 - drivers/media/i2c/ds90ub960.c | 2 - drivers/media/i2c/hi556.c | 64 +- drivers/media/i2c/imx219.c | 1247 +++++----- drivers/media/i2c/mt9m114.c | 2481 ++++++++++++++++++++ drivers/media/i2c/ov2740.c | 125 +- drivers/media/mc/mc-entity.c | 15 +- drivers/media/platform/nxp/imx-mipi-csis.c | 2 - drivers/media/v4l2-core/v4l2-subdev.c | 38 + include/media/mipi-csi2.h | 1 + 23 files changed, 3636 insertions(+), 1163 deletions(-) create mode 100644 Documentation/devicetree/bindings/media/i2c/onnn,mt9m114.yaml create mode 100644 Documentation/userspace-api/media/drivers/camera-sensor.rst create mode 100644 drivers/media/i2c/mt9m114.c