mbox series

[GIT,PULL,for,5.12] V4L2 camera sensor and other patches

Message ID 20210104120612.GB850@valkosipuli.retiisi.org.uk (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,for,5.12] V4L2 camera sensor and other patches | expand

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/for-5.12-1.2-signed

Message

Sakari Ailus Jan. 4, 2021, 12:06 p.m. UTC
Hi Mauro,

Here's a large chunk of V4L2 core IOCTL handling, documentation, camera
sensor and IPU3 patches for 5.12. Notable additions are drivers for ov5648
and ov8856 drivers from Paul Kocialkowski.

A few patches changing V4L2 core IOCTL handling address a long-standing
memory leak issue.

Documentation patches improve documentation e.g. on V4L2 sub-device ops,
add an entity function for ISPs and document clock handling for camera
sensors and document relation between pixel rate and link frequency on
CSI-2 with C-PHY.

There are also noteworthy improvements to the ov5647 driver from Jacopo
Mondi and Dave Stevenson.

Please pull.


The following changes since commit 5c8fe583cce542aa0b84adc939ce85293de36e5e:

  Linux 5.11-rc1 (2020-12-27 15:30:22 -0800)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.12-1.2-signed

for you to fetch changes up to 4e3e9a6501921f3525b9e16e26a969718d9f30c0:

  media: mt9v111: Remove unneeded device-managed puts (2021-01-04 13:51:12 +0200)

----------------------------------------------------------------
V4L2 patches for 5.12

----------------------------------------------------------------
Andy Shevchenko (1):
      media: ipu3-cio2: Build only for x86

Dave Stevenson (8):
      media: ov5647: Add support for PWDN GPIO.
      media: ov5647: Add support for non-continuous clock mode
      media: ov5647: Add set_fmt and get_fmt calls.
      media: ov5647: Add support for get_selection()
      media: ov5647: Set V4L2_SUBDEV_FL_HAS_EVENTS flag
      media: ov5647: Support V4L2_CID_PIXEL_RATE
      media: ov5647: Support V4L2_CID_VBLANK control
      media: ov5647: Advertise the correct exposure range

David Plowman (1):
      media: ov5647: Support gain, exposure and AWB controls

Ezequiel Garcia (1):
      media: mt9v111: Remove unneeded device-managed puts

Hans Verkuil (1):
      media: i2c: imx219: take lock in imx219_enum_mbus_code/frame_size

Jacopo Mondi (22):
      dt-bindings: media: i2c: Rename ov5647.yaml
      media: ov5647: Fix format initialization
      media: ov5647: Fix style issues
      media: ov5647: Replace license with SPDX identifier
      media: ov5647: Fix return value from read/write
      media: ov5647: Program mode at s_stream(1) time
      media: ov5647: Implement enum_frame_size()
      media: ov5647: Protect s_stream() with mutex
      media: ov5647: Rationalize driver structure name
      media: ov5647: Break out format handling
      media: ov5647: Rename SBGGR8 VGA mode
      media: ov5647: Add SGGBR10_1X10 modes
      media: ov5647: Use SBGGR10_1X10 640x480 as default
      media: ov5647: Implement set_fmt pad operation
      media: ov5647: Support V4L2_CID_HBLANK control
      media: ov5647: Use pm_runtime infrastructure
      media: ov5647: Rework s_stream() operation
      media: ov5647: Apply controls only when powered
      media: ov5647: Constify oe_enable/disable reglist
      media: ov5647: Support VIDIOC_SUBSCRIBE_EVENT
      media: ov5647: Remove 640x480 SBGGR8 mode
      media: i2c: ov5670: Fix PIXEL_RATE minimum value

Paul Kocialkowski (4):
      dt-bindings: media: i2c: Add OV5648 bindings documentation
      media: i2c: Add support for the OV5648 image sensor
      dt-bindings: media: i2c: Add OV8865 bindings documentation
      media: i2c: Add support for the OV8865 image sensor

Pavel Machek (1):
      media: ipu3-cio2: Fix mbus_code processing in cio2_subdev_set_fmt()

Sakari Ailus (8):
      Documentation: v4l: Remove reference to video ops
      Documentation: v4l: Document that link_validate op is valid for sink only
      Documentation: media: Update pixel rate formula for C-PHY
      v4l: fwnode: v4l2_async_notifier_parse_fwnode_endpoints is deprecated
      media: uapi: Add an entity type for Image Signal Processors
      v4l: ioctl: Fix memory leak in video_usercopy
      v4l: ioctl: Use kmalloc to allocate a small chunk of memory
      Documentation: media: Document clock handling in camera sensor drivers

 .../media/i2c/{ov5647.yaml => ovti,ov5647.yaml}    |    0
 .../devicetree/bindings/media/i2c/ovti,ov5648.yaml |  115 +
 .../devicetree/bindings/media/i2c/ovti,ov8865.yaml |  124 +
 Documentation/driver-api/media/camera-sensor.rst   |   20 +-
 Documentation/driver-api/media/csi2.rst            |    4 +-
 Documentation/driver-api/media/v4l2-subdev.rst     |   15 +-
 .../userspace-api/media/mediactl/media-types.rst   |    7 +
 MAINTAINERS                                        |    2 +-
 drivers/media/i2c/Kconfig                          |   26 +
 drivers/media/i2c/Makefile                         |    2 +
 drivers/media/i2c/imx219.c                         |    8 +-
 drivers/media/i2c/mt9v111.c                        |    6 -
 drivers/media/i2c/ov5647.c                         | 1259 +++++++--
 drivers/media/i2c/ov5648.c                         | 2623 +++++++++++++++++
 drivers/media/i2c/ov5670.c                         |    3 +-
 drivers/media/i2c/ov8865.c                         | 2972 ++++++++++++++++++++
 drivers/media/pci/intel/ipu3/Kconfig               |    3 +-
 drivers/media/pci/intel/ipu3/ipu3-cio2.c           |    2 +-
 drivers/media/v4l2-core/v4l2-ioctl.c               |   36 +-
 include/media/v4l2-fwnode.h                        |    4 +
 include/uapi/linux/media.h                         |    1 +
 21 files changed, 6981 insertions(+), 251 deletions(-)
 rename Documentation/devicetree/bindings/media/i2c/{ov5647.yaml => ovti,ov5647.yaml} (100%)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/ovti,ov5648.yaml
 create mode 100644 Documentation/devicetree/bindings/media/i2c/ovti,ov8865.yaml
 create mode 100644 drivers/media/i2c/ov5648.c
 create mode 100644 drivers/media/i2c/ov8865.c

Comments

Jenkins Jan. 4, 2021, 12:49 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20210104120612.GB850@valkosipuli.retiisi.org.uk/
Build log: https://builder.linuxtv.org/job/patchwork/84315/
Build time: 00:24:32
Link: https://lore.kernel.org/linux-media/20210104120612.GB850@valkosipuli.retiisi.org.uk

gpg: Signature made Mon 04 Jan 2021 11:51:32 AM UTC
gpg:                using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg:                issuer "sakari.ailus@linux.intel.com"
gpg: Can't check signature: No public key

Summary: got 7/47 patches with issues, being 0 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0020-media-ov5647-Add-SGGBR10_1X10-modes.patch:

   checkpatch.pl:
	$ cat patches/0020-media-ov5647-Add-SGGBR10_1X10-modes.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:16: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
	-:16: WARNING: Unknown commit id '581dfda6d0a62', maybe rebased or not pulled?

patches/0025-media-ov5647-Support-V4L2_CID_HBLANK-control.patch:

   checkpatch.pl:
	$ cat patches/0025-media-ov5647-Support-V4L2_CID_HBLANK-control.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:9: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
	-:9: WARNING: Unknown commit id 'd82f202156605', maybe rebased or not pulled?

patches/0037-v4l-ioctl-Fix-memory-leak-in-video_usercopy.patch:

   checkpatch.pl:
	$ cat patches/0037-v4l-ioctl-Fix-memory-leak-in-video_usercopy.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:53: CHECK: Comparison to NULL could be written "!array_buf"

patches/0043-dt-bindings-media-i2c-Add-OV5648-bindings-documentat.patch:

   checkpatch.pl:
	$ cat patches/0043-dt-bindings-media-i2c-Add-OV5648-bindings-documentat.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:18: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0044-media-i2c-Add-support-for-the-OV5648-image-sensor.patch:

   checkpatch.pl:
	$ cat patches/0044-media-i2c-Add-support-for-the-OV5648-image-sensor.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:65: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:567: CHECK: Macro argument 'c' may be better as '(c)' to avoid precedence issues
	-:726: CHECK: struct mutex definition without comment
	-:2375: CHECK: multiple assignments should be avoided
	-:2376: CHECK: multiple assignments should be avoided

patches/0045-dt-bindings-media-i2c-Add-OV8865-bindings-documentat.patch:

   checkpatch.pl:
	$ cat patches/0045-dt-bindings-media-i2c-Add-OV8865-bindings-documentat.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:23: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0046-media-i2c-Add-support-for-the-OV8865-image-sensor.patch:

   checkpatch.pl:
	$ cat patches/0046-media-i2c-Add-support-for-the-OV8865-image-sensor.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:73: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:172: CHECK: Prefer using the BIT macro
	-:303: CHECK: Please don't use multiple blank lines
	-:349: CHECK: Prefer using the BIT macro
	-:539: CHECK: Macro argument 'c' may be better as '(c)' to avoid precedence issues
	-:750: CHECK: struct mutex definition without comment
	-:1678: CHECK: Alignment should match open parenthesis
	-:2739: CHECK: multiple assignments should be avoided
	-:2740: CHECK: multiple assignments should be avoided


Error #512 when building PDF docs