[PATCHv5] mx2_camera: Add soc_camera support for i.MX25/i.MX27

Guennadi Liakhovetski g.liakhovetski at gmx.de
Thu Jul 1 10:23:23 EDT 2010


Hi Uwe

On Thu, 1 Jul 2010, Uwe Kleine-König wrote:

> > diff --git a/drivers/media/video/Kconfig b/drivers/media/video/Kconfig
> > index bdbc9d3..27e2acc 100644
> > --- a/drivers/media/video/Kconfig
> > +++ b/drivers/media/video/Kconfig
> > @@ -969,6 +969,19 @@ config VIDEO_OMAP2
> >  	---help---
> >  	  This is a v4l2 driver for the TI OMAP2 camera capture interface
> >  
> > +config VIDEO_MX2_HOSTSUPPORT
> > +        bool
> > +
> > +config VIDEO_MX2
> > +	tristate "i.MX27/i.MX25 Camera Sensor Interface driver"
> > +	depends on VIDEO_DEV && SOC_CAMERA && (MACH_MX27 || ARCH_MX25)
> > +	select VIDEOBUF_DMA_CONTIG
> CONTIG?

What exactly was your question here?

> > diff --git a/drivers/media/video/mx2_camera.c b/drivers/media/video/mx2_camera.c
> > new file mode 100644
> > index 0000000..98c93fa
> > --- /dev/null
> > +++ b/drivers/media/video/mx2_camera.c
> > @@ -0,0 +1,1513 @@
> 
> [...snip...]

> > +static struct platform_driver mx2_camera_driver = {
> > +	.driver 	= {
> > +		.name	= MX2_CAM_DRV_NAME,
> I'm always unsure if you need
> 
> 		.owner  = THIS_MODULE,
> 
> here.

It is not needed in this case. See the "owner" field in struct 
soc_camera_host_ops mx2_soc_camera_host_ops.

But that's not the reason why I'm replying. What I didn't like in these 
your reviews, was the fact, that this driver has been submitted a number 
of times to the arm-kernel ML, it has "mx2" in its subject, so, you had 
enough chances to review it, just like Sascha did. Instead, you review it 
now, making the author create new versions of his patch. You have been 
asked for advise, because this patch potentially collided with other your 
patches, your help in resolving this question is appreciated. But then you 
suddenly decide to review the whole patch... Several of my patches have 
been treated similarly in the past, so, I know how annoying it is to have 
to re-iterate them because at v5 someone suddenly decided to take part in 
the patch review process too...

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/



More information about the linux-arm-kernel mailing list