linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux-media@vger.kernel.org
Cc: Manivannan Sadhasivam <mani@kernel.org>,
	Sakari Ailus <sakari.ailus@linux.intel.com>
Subject: [PATCH v3 0/2] media: i2c: IMX296 camera sensor support
Date: Tue, 23 Aug 2022 03:58:20 +0300	[thread overview]
Message-ID: <20220823005822.13805-1-laurent.pinchart@ideasonboard.com> (raw)

Hello,

This series adds support for the Sony IMX296 camera sensor driver, with
DT bindings in patch 1/2 and a driver in patch 2/2.

The DT bindings originate from a series posted by Manivannan ([1]). They
have been updated, see patch 1/2 for a detailed changelog.

The most notable changes compared to v2 ([1]) are

- Support for color/monochrome model identification through separate
  compatible strings
- Usage of the V4L2 active subdev state

Other small fixes and improvements are detailed in the changelog of each
patch.

Most review comments frmo v2 have been taken into account. The only
exception is the merge of the imx296_power_on() function with
imx296_runtime_resume() function (and similarly for power off and
runtime suspend) as requested by Sakari. I gave it a try, and it
conflicts with grouping all V4L2 subdev initialization in
imx296_subdev_init(). I really think keeping the functions separate
produces cleaner code, without any drawback.

[1] https://lore.kernel.org/all/20211219220948.28953-1-laurent.pinchart@ideasonboard.com

Laurent Pinchart (1):
  media: i2c: IMX296 camera sensor driver

Manivannan Sadhasivam (1):
  dt-bindings: media: i2c: Add IMX296 CMOS sensor binding

 .../bindings/media/i2c/sony,imx296.yaml       |  106 ++
 MAINTAINERS                                   |    9 +
 drivers/media/i2c/Kconfig                     |   13 +
 drivers/media/i2c/Makefile                    |    1 +
 drivers/media/i2c/imx296.c                    | 1174 +++++++++++++++++
 5 files changed, 1303 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx296.yaml
 create mode 100644 drivers/media/i2c/imx296.c

-- 
Regards,

Laurent Pinchart

             reply	other threads:[~2022-08-23  0:58 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-23  0:58 Laurent Pinchart [this message]
2022-08-23  0:58 ` [PATCH v3 1/2] dt-bindings: media: i2c: Add IMX296 CMOS sensor binding Laurent Pinchart
2022-08-23  0:58 ` [PATCH v3 2/2] media: i2c: IMX296 camera sensor driver Laurent Pinchart

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=20220823005822.13805-1-laurent.pinchart@ideasonboard.com \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=mani@kernel.org \
    --cc=sakari.ailus@linux.intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).