All of lore.kernel.org
 help / color / mirror / Atom feed
From: petrcvekcz@gmail.com
To: hans.verkuil@cisco.com, jacopo@jmondi.org, mchehab@kernel.org,
	marek.vasut@gmail.com
Cc: Petr Cvek <petrcvekcz@gmail.com>,
	linux-media@vger.kernel.org, robert.jarzmik@free.fr,
	slapin@ossfans.org, philipp.zabel@gmail.com, sakari.ailus@iki.fi
Subject: [PATCH v3 0/8] media: soc_camera: ov9640: switch driver to v4l2_async
Date: Thu, 13 Dec 2018 16:39:11 +0100	[thread overview]
Message-ID: <cover.1544713575.git.petrcvekcz@gmail.com> (raw)

From: Petr Cvek <petrcvekcz@gmail.com>

This patch series transfer the ov9640 driver from the soc_camera subsystem
into a standalone v4l2 driver. There is no changes except the required
v4l2_async calls, GPIO allocation, deletion of now unused variables,
a change from mdelay() to msleep() and an addition of SPDX identifiers
(as suggested in the v1 version RFC).

The config symbol has been changed from CONFIG_SOC_CAMERA_OV9640 to
CONFIG_VIDEO_OV9640.

Also as the drivers of the soc_camera seems to be orphaned I'm volunteering
as a maintainer of the driver (I own the hardware).

I've found the ov9640 seems to be used at least in (the future) HTC
Magician and Palm Zire72. These will need to define power and reset GPIOs
and remove the soc_camera definitions. I'm debugging it on magician now
(ov9640 was unusable on them since the pxa_camera switched from
the soc_camera).

Additional fixes (from v2) are: a fix of the probe error path, variables
change to an unsigned type (indexes, lengths), a redefinition of formats
array to const and better clarity of code near returns.

Petr Cvek (8):
  media: soc_camera: ov9640: move ov9640 out of soc_camera
  media: i2c: ov9640: drop soc_camera code and switch to v4l2_async
  MAINTAINERS: add Petr Cvek as a maintainer for the ov9640 driver
  media: i2c: ov9640: add missing SPDX identifiers
  media: i2c: ov9640: change array index or length variables to unsigned
  media: i2c: ov9640: add space before return for better clarity
  media: i2c: ov9640: make array of supported formats constant
  media: i2c: ov9640: fix missing error handling in probe

 MAINTAINERS                          |   6 +
 drivers/media/i2c/Kconfig            |   7 +
 drivers/media/i2c/Makefile           |   1 +
 drivers/media/i2c/ov9640.c           | 776 +++++++++++++++++++++++++++
 drivers/media/i2c/ov9640.h           | 207 +++++++
 drivers/media/i2c/soc_camera/Kconfig |   6 +-
 6 files changed, 1001 insertions(+), 2 deletions(-)
 create mode 100644 drivers/media/i2c/ov9640.c
 create mode 100644 drivers/media/i2c/ov9640.h

-- 
2.20.0


             reply	other threads:[~2018-12-13 15:39 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-13 15:39 petrcvekcz [this message]
2018-12-13 15:39 ` [PATCH v3 1/8] media: soc_camera: ov9640: move ov9640 out of soc_camera petrcvekcz
2018-12-13 15:39 ` [PATCH v3 2/8] media: i2c: ov9640: drop soc_camera code and switch to v4l2_async petrcvekcz
2018-12-13 15:39 ` [PATCH v3 3/8] MAINTAINERS: add Petr Cvek as a maintainer for the ov9640 driver petrcvekcz
2018-12-13 15:39 ` [PATCH v3 4/8] media: i2c: ov9640: add missing SPDX identifiers petrcvekcz
2018-12-13 15:39 ` [PATCH v3 5/8] media: i2c: ov9640: change array index or length variables to unsigned petrcvekcz
2018-12-13 15:39 ` [PATCH v3 6/8] media: i2c: ov9640: add space before return for better clarity petrcvekcz
2018-12-13 15:39 ` [PATCH v3 7/8] media: i2c: ov9640: make array of supported formats constant petrcvekcz
2018-12-13 15:39 ` [PATCH v3 8/8] media: i2c: ov9640: fix missing error handling in probe petrcvekcz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1544713575.git.petrcvekcz@gmail.com \
    --to=petrcvekcz@gmail.com \
    --cc=hans.verkuil@cisco.com \
    --cc=jacopo@jmondi.org \
    --cc=linux-media@vger.kernel.org \
    --cc=marek.vasut@gmail.com \
    --cc=mchehab@kernel.org \
    --cc=philipp.zabel@gmail.com \
    --cc=robert.jarzmik@free.fr \
    --cc=sakari.ailus@iki.fi \
    --cc=slapin@ossfans.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.