Re: [PATCH 0/3] V4L2_CID_PIXEL_RATE support in sensor drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sakari,

On Monday 14 May 2012 00:53:36 Sakari Ailus wrote:
> On Wed, May 09, 2012 at 02:55:56PM +0200, Laurent Pinchart wrote:
> > Hi everybody,
> > 
> > This patch implements support for the V4L2_CID_PIXEL_RATE control in the
> > mt9t001, mt9p031 and mt9m032 sensor drivers.
> > 
> > Recent changes to the OMAP3 ISP driver (see the media-for-3.5 branch in
> > the http://git.linuxtv.org/sailus/media_tree.git repository) made support
> > for that control mandatory for sensor drivers that are to be used with the
> > OMAP3 ISP.
> > 
> > Sakari, would you like to take these patches in your tree and push them
> > for v3.5 ? V4L2_CID_PIXEL_RATE support for the mt9v032 driver is also
> > still missing, the patch you've sent to the list two months ago needs to
> > be rebased. Will you do that or should I do it ? I'm also wondering
> > whether that patch shouldn't mark the V4L2_CID_PIXEL_RATE control as
> > volatile, calling v4l2_s_ext_ctrls() from inside the driver looks quite
> > hackish to> me. The alternative would be to create a 64-bit version of
> > v4l2_ctrl_s_ctrl().
> >
> > Laurent Pinchart (3):
> >   mt9t001: Implement V4L2_CID_PIXEL_RATE control
> >   mt9p031: Implement V4L2_CID_PIXEL_RATE control
> >   mt9m032: Implement V4L2_CID_PIXEL_RATE control
> >  
> >  drivers/media/video/mt9m032.c |   13 +++++++++++--
> >  drivers/media/video/mt9p031.c |    5 ++++-
> >  drivers/media/video/mt9t001.c |   13 +++++++++++--
> >  include/media/mt9t001.h       |    1 +
> >  4 files changed, 27 insertions(+), 5 deletions(-)
> 
> Acked-by: Sakari Ailus <sakari.ailus@xxxxxx>

Thank you. Ad the patches depend on the V4L2_CID_PIXEL_RATE control, could you 
please take them in your tree and push them for v3.5 ?

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux