mbox series

[GIT,PULL,FOR,6.7] Camera sensor patches, preparation for enabling streams API

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

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/for-6.7-2-signed

Message

Sakari Ailus Oct. 3, 2023, 12:29 p.m. UTC 
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

Comments

Jenkins Oct. 3, 2023, 1:19 p.m. UTC | #1 
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?