All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jacopo Mondi <jacopo+renesas@jmondi.org>
To: kieran.bingham+renesas@ideasonboard.com,
	laurent.pinchart+renesas@ideasonboard.com,
	niklas.soderlund+renesas@ragnatech.se, geert@linux-m68k.org
Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>,
	linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
	linux-kernel@vger.kernel.org, Hyun Kwon <hyunk@xilinx.com>,
	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
	sergei.shtylyov@gmail.com
Subject: [PATCH v5 0/8] media: i2c: Add RDACM21 camera module
Date: Mon, 16 Nov 2020 14:52:57 +0100	[thread overview]
Message-ID: <20201116135305.81319-1-jacopo+renesas@jmondi.org> (raw)

Hello,
   this v5 mostly changes how the rdacm21 driver handles writes to the
OV490 chip by handling the 'high' part of the sensor register through an
helper function as suggested by Kieran and Laurent.

The diff is available at:
https://paste.debian.net/1172700/

Minor fixes in patches subjects and in bindings as reported by Sergei and
Kieran.

Bindings have now been sent to DT people now that they've been reviewed
by Kieran.

v4: https://patchwork.linuxtv.org/project/linux-media/list/?series=3847
v3: https://patchwork.linuxtv.org/project/linux-media/list/?series=3657

Background in v1 cover letter:
https://www.spinics.net/lists/linux-renesas-soc/msg52886.html

I have tested on Eagle V3M with 4 RDACM21, but the whole point of
this series is to retain compatibility with RDACM20.

For this reason I have included 2 patches on top, not intended for merge
that re-propose DTS support for the MAXIM max9286 expansion board connected
to Salvator-X and add the newly introduced property to the DTS file.

Kieran, I know you have a working setup with RDACM20, the final patches are
meant for ease your testing. Can you give this series a spin ?

Series based on latest renesas-drivers tag: renesas-drivers-2020-11-10-v5.10-rc3

Thanks
  j

Jacopo Mondi (7):
  media: i2c: Add driver for RDACM21 camera module
  dt-bindings: media: max9286: Document
    'maxim,initial-reverse-channel-mV'
  media: i2c: max9286: Break-out reverse channel setup
  media: i2c: max9286: Make channel amplitude programmable
  media: i2c: max9286: Configure reverse channel amplitude
  [DNI] arm64: dts: renesas: salvator-x-max9286: Specify channel
    amplitude
  [DNI] arm64: dts: renesas: eagle: Specify channel amplitude

Laurent Pinchart (1):
  arm64: dts: renesas: salvator-x: Add MAX9286 expansion board

 .../bindings/media/i2c/maxim,max9286.yaml     |  23 +
 MAINTAINERS                                   |  12 +
 .../arm64/boot/dts/renesas/r8a77970-eagle.dts |   1 +
 .../boot/dts/renesas/salvator-x-max9286.dtsi  | 396 ++++++++++++
 drivers/media/i2c/Kconfig                     |  13 +
 drivers/media/i2c/Makefile                    |   2 +
 drivers/media/i2c/max9286.c                   |  58 +-
 drivers/media/i2c/rdacm21.c                   | 595 ++++++++++++++++++
 8 files changed, 1087 insertions(+), 13 deletions(-)
 create mode 100644 arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi
 create mode 100644 drivers/media/i2c/rdacm21.c

--
2.29.1


             reply	other threads:[~2020-11-16 13:53 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-16 13:52 Jacopo Mondi [this message]
2020-11-16 13:52 ` [PATCH v5 1/8] media: i2c: Add driver for RDACM21 camera module Jacopo Mondi
2020-11-16 14:47   ` Kieran Bingham
2020-11-17 13:36     ` Jacopo Mondi
2020-11-17 15:31       ` Kieran Bingham
2020-11-19 14:42         ` Jacopo Mondi
2020-11-16 13:52 ` [PATCH v5 2/8] dt-bindings: media: max9286: Document 'maxim,initial-reverse-channel-mV' Jacopo Mondi
2020-11-30 22:00   ` Rob Herring
2020-12-15 11:14     ` Jacopo Mondi
2020-12-15 11:44       ` Geert Uytterhoeven
2020-12-15 12:08         ` Jacopo Mondi
2020-12-15 13:49           ` Rob Herring
2020-11-16 13:53 ` [PATCH v5 3/8] media: i2c: max9286: Break-out reverse channel setup Jacopo Mondi
2020-11-16 13:53 ` [PATCH v5 4/8] media: i2c: max9286: Make channel amplitude programmable Jacopo Mondi
2020-11-16 13:53 ` [PATCH v5 5/8] media: i2c: max9286: Configure reverse channel amplitude Jacopo Mondi
2020-11-16 13:53 ` [PATCH v5 6/8] arm64: dts: renesas: salvator-x: Add MAX9286 expansion board Jacopo Mondi
2020-11-16 13:53 ` [PATCH v5 7/8] [DNI] arm64: dts: renesas: salvator-x-max9286: Specify channel amplitude Jacopo Mondi
2020-11-18 13:40   ` Kieran Bingham
2020-11-16 13:53 ` [PATCH v5 8/8] [DNI] arm64: dts: renesas: eagle: " Jacopo Mondi
2020-11-18 13:34 ` [PATCH v5 0/8] media: i2c: Add RDACM21 camera module Kieran Bingham
2020-11-19 14:38   ` Jacopo Mondi

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=20201116135305.81319-1-jacopo+renesas@jmondi.org \
    --to=jacopo+renesas@jmondi.org \
    --cc=geert@linux-m68k.org \
    --cc=hyunk@xilinx.com \
    --cc=kieran.bingham+renesas@ideasonboard.com \
    --cc=laurent.pinchart+renesas@ideasonboard.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=manivannan.sadhasivam@linaro.org \
    --cc=niklas.soderlund+renesas@ragnatech.se \
    --cc=sergei.shtylyov@gmail.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 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.