[v4,0/3] media: i2c: imx219: Feature enhancements

Message ID 1583846229-6799-1-git-send-email-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive)
Headers
Series media: i2c: imx219: Feature enhancements |

Message

Prabhakar Mahadev Lad March 10, 2020, 1:17 p.m. UTC
  This patch series does the following:
1: Makes sure the sensor is LP11 state on power up
2: Adds support for RAW8
3: Adds support for 640x480 resolution

Changes for v4:
1: Fixed review comments for patch 2/2 as request by Sakari.

Changes for v3:
1: Only patch 3/3 was posted for review.

Changes for v2:
1: Dropped setting the format in probe to coax the sensor to enter LP11
   state.
2: Fixed switching between RAW8/RAW10 modes.
3: Fixed fps setting for 640x480 and switched to auto mode.

Lad Prabhakar (3):
  media: i2c: imx219: Fix power sequence
  media: i2c: imx219: Add support for RAW8 bit bayer format
  media: i2c: imx219: Add support for cropped 640x480 resolution

 drivers/media/i2c/imx219.c | 235 ++++++++++++++++++++++++++++++++++++++-------
 1 file changed, 202 insertions(+), 33 deletions(-)
  

Comments

Marcus Folkesson March 17, 2020, 8:01 a.m. UTC | #1
Hi,

To which tree does this patchset apply?

On Tue, Mar 10, 2020 at 01:17:06PM +0000, Lad Prabhakar wrote:
> This patch series does the following:
> 1: Makes sure the sensor is LP11 state on power up
> 2: Adds support for RAW8
> 3: Adds support for 640x480 resolution
> 
> Changes for v4:
> 1: Fixed review comments for patch 2/2 as request by Sakari.
> 
> Changes for v3:
> 1: Only patch 3/3 was posted for review.
> 
> Changes for v2:
> 1: Dropped setting the format in probe to coax the sensor to enter LP11
>    state.
> 2: Fixed switching between RAW8/RAW10 modes.
> 3: Fixed fps setting for 640x480 and switched to auto mode.
> 
> Lad Prabhakar (3):
>   media: i2c: imx219: Fix power sequence
>   media: i2c: imx219: Add support for RAW8 bit bayer format
>   media: i2c: imx219: Add support for cropped 640x480 resolution
> 
>  drivers/media/i2c/imx219.c | 235 ++++++++++++++++++++++++++++++++++++++-------
>  1 file changed, 202 insertions(+), 33 deletions(-)
> 
> -- 
> 2.7.4
> 


Thanks,
Marcus Folkesson
  
Prabhakar Mahadev Lad March 17, 2020, 8:03 a.m. UTC | #2
Hi Marcus,

> -----Original Message-----
> From: Marcus Folkesson <marcus.folkesson@gmail.com>
> Sent: 17 March 2020 08:02
> To: Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Cc: Dave Stevenson <dave.stevenson@raspberrypi.com>; Sakari Ailus
> <sakari.ailus@linux.intel.com>; Mauro Carvalho Chehab
> <mchehab@kernel.org>; linux-media@vger.kernel.org; linux-
> kernel@vger.kernel.org; Lad Prabhakar <prabhakar.csengg@gmail.com>
> Subject: Re: [PATCH v4 0/3] media: i2c: imx219: Feature enhancements
>
> Hi,
>
> To which tree does this patchset apply?
>
These patches apply to media_tree [1], btw v4 patches have been accepted and  merged into master branch of media_tree.

[1] https://git.linuxtv.org//media_tree.git

Cheers,
--Prabhakar

> On Tue, Mar 10, 2020 at 01:17:06PM +0000, Lad Prabhakar wrote:
> > This patch series does the following:
> > 1: Makes sure the sensor is LP11 state on power up
> > 2: Adds support for RAW8
> > 3: Adds support for 640x480 resolution
> >
> > Changes for v4:
> > 1: Fixed review comments for patch 2/2 as request by Sakari.
> >
> > Changes for v3:
> > 1: Only patch 3/3 was posted for review.
> >
> > Changes for v2:
> > 1: Dropped setting the format in probe to coax the sensor to enter LP11
> >    state.
> > 2: Fixed switching between RAW8/RAW10 modes.
> > 3: Fixed fps setting for 640x480 and switched to auto mode.
> >
> > Lad Prabhakar (3):
> >   media: i2c: imx219: Fix power sequence
> >   media: i2c: imx219: Add support for RAW8 bit bayer format
> >   media: i2c: imx219: Add support for cropped 640x480 resolution
> >
> >  drivers/media/i2c/imx219.c | 235
> ++++++++++++++++++++++++++++++++++++++-------
> >  1 file changed, 202 insertions(+), 33 deletions(-)
> >
> > --
> > 2.7.4
> >
>
>
> Thanks,
> Marcus Folkesson


Renesas Electronics Europe GmbH, Geschaeftsfuehrer/President: Carsten Jauch, Sitz der Gesellschaft/Registered office: Duesseldorf, Arcadiastrasse 10, 40472 Duesseldorf, Germany, Handelsregister/Commercial Register: Duesseldorf, HRB 3708 USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE reg. no.: DE 14978647
  
Marcus Folkesson March 17, 2020, 8:11 a.m. UTC | #3
Hi Prabhakar,

On Tue, Mar 17, 2020 at 08:03:49AM +0000, Prabhakar Mahadev Lad wrote:
> Hi Marcus,
> 
> > -----Original Message-----
> > From: Marcus Folkesson <marcus.folkesson@gmail.com>
> > Sent: 17 March 2020 08:02
> > To: Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > Cc: Dave Stevenson <dave.stevenson@raspberrypi.com>; Sakari Ailus
> > <sakari.ailus@linux.intel.com>; Mauro Carvalho Chehab
> > <mchehab@kernel.org>; linux-media@vger.kernel.org; linux-
> > kernel@vger.kernel.org; Lad Prabhakar <prabhakar.csengg@gmail.com>
> > Subject: Re: [PATCH v4 0/3] media: i2c: imx219: Feature enhancements
> >
> > Hi,
> >
> > To which tree does this patchset apply?
> >
> These patches apply to media_tree [1], btw v4 patches have been accepted and  merged into master branch of media_tree.

Thank you and good work!

> 
> [1] https://git.linuxtv.org//media_tree.git
> 
> Cheers,
> --Prabhakar

Best regards
Marcus Folkesson