Re: [PATCH v4] v4l: Add driver for Micron MT9M032 camera sensor

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

 



On Wed, Dec 21, 2011 at 02:06:20AM +0100, Laurent Pinchart wrote:
> Hi Martin,
> 
> Thanks for the patch.
> 
> On Saturday 17 December 2011 11:10:55 Martin Hostettler wrote:
> > The MT9M032 is a parallel 1.6MP sensor from Micron controlled through I2C.
> > 
> > The driver creates a V4L2 subdevice. It currently supports cropping, gain,
> > exposure and v/h flipping controls in monochrome mode with an
> > external pixel clock.
> 
> There are still several small issues with this driver. Things like not using 
> the module_i2c_driver() macro, some indentation, magic values in registers 
> (I'm trying to get more documentation), PLL setup (although that can be fixed 
> later, it's not a requirement for the driver to be mainlined), ...
> 
> Would you be fine if I took the patch in my tree, fixed the remaining issues 
> and pushed it to mainline for v3.4 (the time frame is too short for v3.3) ? 

Sure, that would be much appreciated. Thanks for reviewing and takeing
these patches!

Best regards, 
 - Martin Hostettler


> Authorship will of course be preserved. The alternative would be to go through 
> review/modification cycles, and I don't want to waste too much of your time 
> :-)
> 
> -- 
> Best 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