mbox series

[v8,0/5] media: i2c: Add RDACM21 camera module

Message ID 20210114170429.139762-1-jacopo+renesas@jmondi.org
Headers show
Series media: i2c: Add RDACM21 camera module | expand

Message

Jacopo Mondi Jan. 14, 2021, 5:04 p.m. UTC
One more iteration to squash in all the fixups sent in v7 and address
a comment from Sergei in [2/5] commit message.

All patches now reviewed and hopefully ready to be collected!

Thanks
  j

Jacopo Mondi (5):
  media: i2c: Add driver for RDACM21 camera module
  dt-bindings: media: max9286: Document
    'maxim,reverse-channel-microvolt'
  media: i2c: max9286: Break-out reverse channel setup
  media: i2c: max9286: Make channel amplitude programmable
  media: i2c: max9286: Configure reverse channel amplitude

 .../bindings/media/i2c/maxim,max9286.yaml     |  22 +
 MAINTAINERS                                   |  12 +
 drivers/media/i2c/Kconfig                     |  13 +
 drivers/media/i2c/Makefile                    |   2 +
 drivers/media/i2c/max9286.c                   |  60 +-
 drivers/media/i2c/rdacm21.c                   | 623 ++++++++++++++++++
 6 files changed, 719 insertions(+), 13 deletions(-)
 create mode 100644 drivers/media/i2c/rdacm21.c

--
2.29.2

Comments

Jacopo Mondi Feb. 1, 2021, 8:54 a.m. UTC | #1
Hi Sakari,

On Thu, Jan 14, 2021 at 06:04:24PM +0100, Jacopo Mondi wrote:
> One more iteration to squash in all the fixups sent in v7 and address

> a comment from Sergei in [2/5] commit message.

>

> All patches now reviewed and hopefully ready to be collected!


All patches seems reviewed, do you think we can still collect this for
the v5.12 merge window ?

Thanks
  j

>

> Thanks

>   j

>

> Jacopo Mondi (5):

>   media: i2c: Add driver for RDACM21 camera module

>   dt-bindings: media: max9286: Document

>     'maxim,reverse-channel-microvolt'

>   media: i2c: max9286: Break-out reverse channel setup

>   media: i2c: max9286: Make channel amplitude programmable

>   media: i2c: max9286: Configure reverse channel amplitude

>

>  .../bindings/media/i2c/maxim,max9286.yaml     |  22 +

>  MAINTAINERS                                   |  12 +

>  drivers/media/i2c/Kconfig                     |  13 +

>  drivers/media/i2c/Makefile                    |   2 +

>  drivers/media/i2c/max9286.c                   |  60 +-

>  drivers/media/i2c/rdacm21.c                   | 623 ++++++++++++++++++

>  6 files changed, 719 insertions(+), 13 deletions(-)

>  create mode 100644 drivers/media/i2c/rdacm21.c

>

> --

> 2.29.2

>
Sakari Ailus Feb. 1, 2021, 1 p.m. UTC | #2
Hi Jacopo,

On Mon, Feb 01, 2021 at 09:54:40AM +0100, Jacopo Mondi wrote:
> Hi Sakari,

> 

> On Thu, Jan 14, 2021 at 06:04:24PM +0100, Jacopo Mondi wrote:

> > One more iteration to squash in all the fixups sent in v7 and address

> > a comment from Sergei in [2/5] commit message.

> >

> > All patches now reviewed and hopefully ready to be collected!

> 

> All patches seems reviewed, do you think we can still collect this for

> the v5.12 merge window ?


The set seems good to me. There was some fuzz in the DT binding patch; I
hope the resolution is ok:

<URL:https://git.linuxtv.org/sailus/media_tree.git/commit/?id=c9930c965596af73c61e1a6a9ef2d2128582ef38>

Feel free to cc me on the next time. :-)

-- 
Regards,

Sakari Ailus
Jacopo Mondi Feb. 1, 2021, 6:14 p.m. UTC | #3
Hi Sakari,

On Mon, Feb 01, 2021 at 03:00:29PM +0200, Sakari Ailus wrote:
> Hi Jacopo,

>

> On Mon, Feb 01, 2021 at 09:54:40AM +0100, Jacopo Mondi wrote:

> > Hi Sakari,

> >

> > On Thu, Jan 14, 2021 at 06:04:24PM +0100, Jacopo Mondi wrote:

> > > One more iteration to squash in all the fixups sent in v7 and address

> > > a comment from Sergei in [2/5] commit message.

> > >

> > > All patches now reviewed and hopefully ready to be collected!

> >

> > All patches seems reviewed, do you think we can still collect this for

> > the v5.12 merge window ?

>

> The set seems good to me. There was some fuzz in the DT binding patch; I

> hope the resolution is ok:

>

> <URL:https://git.linuxtv.org/sailus/media_tree.git/commit/?id=c9930c965596af73c61e1a6a9ef2d2128582ef38>


Yes, looks good!

>

> Feel free to cc me on the next time. :-)


yeah sorry, I went through too many iterations :)

Thanks
  j

>

> --

> Regards,

>

> Sakari Ailus