mbox series

[GIT,PULL,v2,for,5.4] More sensor driver, omap3isp and documentation patches

Message ID 20190814103251.GA3504@valkosipuli.retiisi.org.uk (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,v2,for,5.4] More sensor driver, omap3isp and documentation patches | expand

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/for-5.4-6-signed

Message

Sakari Ailus Aug. 14, 2019, 10:32 a.m. UTC
Hi Mauro,

Here's a bunch of sensor driver, omap3isp and documentatino patches.
Noteworthy bit is perhaps a driver for the ov5675 sensor.

There are some checkpatch warnings there (patches 1, 6 and 8), the first
two seem false positives whereas the last one comes from the documentation
license header.

I've since v1 added a few more patches; IMX driver improvements as well as
a MAINTAINERS entry for ov5670.

Please pull.


The following changes since commit 0dc99e042a4cfbc1e27572d523d2a1dbaf402cbf:

  media: MAINTAINERS: Remove zoran driver (2019-08-13 11:55:34 -0300)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.4-6-signed

for you to fetch changes up to 852a51507ac865d7fbbf1305e2f485e7a4131b95:

  MAINTAINERS: Add entry for the ov5670 driver (2019-08-14 12:59:16 +0300)

----------------------------------------------------------------
Camera driver + framework patches for 5.4

----------------------------------------------------------------
Ezequiel Garcia (2):
      media: i2c: ov5645: Fix power sequence
      media: imx: mipi csi-2: Don't fail if initial state times-out

Luca Weiss (1):
      media: ov5640: Add support for flash and lens devices

Sakari Ailus (6):
      ov8856: Check reading clock frequency succeeded
      omap3isp: Set device on omap3isp subdevs
      omap3isp: Don't set streaming state on random subdevs
      v4l: Documentation: Serial busses use parallel mbus codes
      v4l: Documentation: Raw Bayer formats are not RGB formats
      MAINTAINERS: Add entry for the ov5670 driver

Shawn Tu (1):
      ov5675: Add support for OV5675 sensor

Steve Longerbeam (1):
      media: docs-rst: Clarify duration of LP-11 mode

Sébastien Szymanski (2):
      media: dt-bindings: imx7-csi: add i.MX6UL/L support
      media: imx7-media-csi: add i.MX6UL support

 .../devicetree/bindings/media/imx7-csi.txt         |    2 +-
 Documentation/media/kapi/csi2.rst                  |   17 +-
 Documentation/media/uapi/v4l/control.rst           |    2 +-
 Documentation/media/uapi/v4l/hist-v4l2.rst         |    2 +-
 Documentation/media/uapi/v4l/pixfmt-bayer.rst      |   38 +
 Documentation/media/uapi/v4l/pixfmt-packed-rgb.rst | 1306 --------------------
 Documentation/media/uapi/v4l/pixfmt-rgb.rst        | 1302 ++++++++++++++++++-
 Documentation/media/uapi/v4l/pixfmt-v4l2.rst       |    2 +-
 Documentation/media/uapi/v4l/pixfmt.rst            |    1 +
 Documentation/media/uapi/v4l/subdev-formats.rst    |    8 +
 MAINTAINERS                                        |   15 +
 drivers/media/i2c/Kconfig                          |   13 +
 drivers/media/i2c/Makefile                         |    1 +
 drivers/media/i2c/ov5640.c                         |    2 +-
 drivers/media/i2c/ov5645.c                         |   26 +-
 drivers/media/i2c/ov5675.c                         | 1183 ++++++++++++++++++
 drivers/media/i2c/ov8856.c                         |    5 +-
 drivers/media/platform/omap3isp/isp.c              |    8 +
 drivers/media/platform/omap3isp/ispccdc.c          |    1 +
 drivers/media/platform/omap3isp/ispccp2.c          |    1 +
 drivers/media/platform/omap3isp/ispcsi2.c          |    1 +
 drivers/media/platform/omap3isp/isppreview.c       |    1 +
 drivers/media/platform/omap3isp/ispresizer.c       |    1 +
 drivers/media/platform/omap3isp/ispstat.c          |    2 +
 drivers/staging/media/imx/Kconfig                  |    4 +-
 drivers/staging/media/imx/imx6-mipi-csi2.c         |   12 +-
 drivers/staging/media/imx/imx7-media-csi.c         |   30 +-
 27 files changed, 2630 insertions(+), 1356 deletions(-)
 create mode 100644 Documentation/media/uapi/v4l/pixfmt-bayer.rst
 delete mode 100644 Documentation/media/uapi/v4l/pixfmt-packed-rgb.rst
 create mode 100644 drivers/media/i2c/ov5675.c

Comments

Jenkins Aug. 14, 2019, 12:08 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/patch/58128/
Build log: https://builder.linuxtv.org/job/patchwork/8485/
Build time: 00:09:49
Link: https://lore.kernel.org/linux-media/20190814103251.GA3504@valkosipuli.retiisi.org.uk
Summary: 3 issues, being 0 at build time

gpg: Signature made Wed 14 Aug 2019 10:10:12 AM UTC
gpg:                using DSA key F0D0377A0D4F25A79238EFE56D40361B6E28C193
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


Error/warnings:

patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:660: CHECK: Alignment should match open parenthesis
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:709: CHECK: spaces preferred around that '/' (ctx:VxV)
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:774: CHECK: Alignment should match open parenthesis
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:781: CHECK: Alignment should match open parenthesis
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:786: CHECK: Alignment should match open parenthesis
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:791: CHECK: Alignment should match open parenthesis

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0001-ov5675-Add-support-for-OV5675-sensor.patch
patches/0006-media-i2c-ov5645-Fix-power-sequence.patch:8: ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit aa4bb8b8838f ("media: ov5640: Re-work MIPI startup sequence")'

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0006-media-i2c-ov5645-Fix-power-sequence.patch
patches/0008-v4l-Documentation-Raw-Bayer-formats-are-not-RGB-form.patch:53: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0008-v4l-Documentation-Raw-Bayer-formats-are-not-RGB-form.patch:58: WARNING: Missing or malformed SPDX-License-Identifier tag in line 1

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0008-v4l-Documentation-Raw-Bayer-formats-are-not-RGB-form.patch
Jenkins Aug. 14, 2019, 5:42 p.m. UTC | #2
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/patch/58128/
Build log: https://builder.linuxtv.org/job/patchwork/8689/
Build time: 00:08:45
Link: https://lore.kernel.org/linux-media/20190814103251.GA3504@valkosipuli.retiisi.org.uk
Summary: 3 issues, being 0 at build time

gpg: Signature made Wed 14 Aug 2019 10:10:12 AM UTC
gpg:                using DSA key F0D0377A0D4F25A79238EFE56D40361B6E28C193
gpg:                issuer "sakari.ailus@linux.intel.com"
gpg: Good signature from "Sakari Ailus <sakari.ailus@linux.intel.com>" [full]


Error/warnings:

patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:660: CHECK: Alignment should match open parenthesis
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:709: CHECK: spaces preferred around that '/' (ctx:VxV)
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:774: CHECK: Alignment should match open parenthesis
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:781: CHECK: Alignment should match open parenthesis
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:786: CHECK: Alignment should match open parenthesis
patches/0001-ov5675-Add-support-for-OV5675-sensor.patch:791: CHECK: Alignment should match open parenthesis

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0001-ov5675-Add-support-for-OV5675-sensor.patch
patches/0006-media-i2c-ov5645-Fix-power-sequence.patch:8: ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit aa4bb8b8838f ("media: ov5640: Re-work MIPI startup sequence")'

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0006-media-i2c-ov5645-Fix-power-sequence.patch
patches/0008-v4l-Documentation-Raw-Bayer-formats-are-not-RGB-form.patch:53: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0008-v4l-Documentation-Raw-Bayer-formats-are-not-RGB-form.patch:58: WARNING: Missing or malformed SPDX-License-Identifier tag in line 1

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0008-v4l-Documentation-Raw-Bayer-formats-are-not-RGB-form.patch