On Mon, Feb 14, 2011 at 01:21:34PM +0100, Laurent Pinchart wrote: > The OMAP3 ISP CCP2 and CSI2 receivers provide an interface to connect > serial MIPI sensors to the device. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx> > Signed-off-by: David Cohen <dacohen@xxxxxxxxx> > Signed-off-by: Stanimir Varbanov <svarbanov@xxxxxxxxxx> > Signed-off-by: Vimarsh Zutshi <vimarsh.zutshi@xxxxxxxxx> > Signed-off-by: Tuukka Toivonen <tuukkat76@xxxxxxxxx> > Signed-off-by: Sergio Aguirre <saaguirre@xxxxxx> > Signed-off-by: Antti Koskipaa <akoskipa@xxxxxxxxx> > Signed-off-by: Ivan T. Ivanov <iivanov@xxxxxxxxxx> > Signed-off-by: RaniSuneela <r-m@xxxxxx> > Signed-off-by: Atanas Filipov <afilipov@xxxxxxxxxx> > Signed-off-by: Gjorgji Rosikopulos <grosikopulos@xxxxxxxxxx> > Signed-off-by: Hiroshi DOYU <Hiroshi.DOYU@xxxxxxxxx> > Signed-off-by: Nayden Kanchev <nkanchev@xxxxxxxxxx> > Signed-off-by: Phil Carmody <ext-phil.2.carmody@xxxxxxxxx> > Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@xxxxxxxxx> > Signed-off-by: Dominic Curran <dcurran@xxxxxx> > Signed-off-by: Ilkka Myllyperkio <ilkka.myllyperkio@xxxxxxxxx> > Signed-off-by: Pallavi Kulkarni <p-kulkarni@xxxxxx> > Signed-off-by: Vaibhav Hiremath <hvaibhav@xxxxxx> checkpatch still complains a bit about this one: CHECK: struct mutex definition without comment #1368: FILE: drivers/media/video/omap3-isp/ispqueue.h:157: + struct mutex lock; CHECK: spinlock_t definition without comment #1369: FILE: drivers/media/video/omap3-isp/ispqueue.h:158: + spinlock_t irqlock; WARNING: please, no space before tabs #2723: FILE: drivers/media/video/omap3-isp/ispvideo.h:49: + * ^Ibits. Identical to @code if the format is 10 bits wide or less.$ WARNING: please, no space before tabs #2725: FILE: drivers/media/video/omap3-isp/ispvideo.h:51: + * ^Iformat. Identical to @code if the format is not DPCM compressed.$ CHECK: spinlock_t definition without comment #2762: FILE: drivers/media/video/omap3-isp/ispvideo.h:88: + spinlock_t lock; CHECK: struct mutex definition without comment #2823: FILE: drivers/media/video/omap3-isp/ispvideo.h:149: + struct mutex mutex; CHECK: struct mutex definition without comment #2840: FILE: drivers/media/video/omap3-isp/ispvideo.h:166: + struct mutex stream_lock; total: 0 errors, 2 warnings, 5 checks, 2806 lines checked /home/balbi/tst.diff has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. does it make sense to fix those ? -- balbi -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html