Hi Felipe, On Monday 14 February 2011 13:37:39 Felipe Balbi wrote: > 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 <[email protected]> > > Signed-off-by: Sakari Ailus <[email protected]> > > Signed-off-by: David Cohen <[email protected]> > > Signed-off-by: Stanimir Varbanov <[email protected]> > > Signed-off-by: Vimarsh Zutshi <[email protected]> > > Signed-off-by: Tuukka Toivonen <[email protected]> > > Signed-off-by: Sergio Aguirre <[email protected]> > > Signed-off-by: Antti Koskipaa <[email protected]> > > Signed-off-by: Ivan T. Ivanov <[email protected]> > > Signed-off-by: RaniSuneela <[email protected]> > > Signed-off-by: Atanas Filipov <[email protected]> > > Signed-off-by: Gjorgji Rosikopulos <[email protected]> > > Signed-off-by: Hiroshi DOYU <[email protected]> > > Signed-off-by: Nayden Kanchev <[email protected]> > > Signed-off-by: Phil Carmody <[email protected]> > > Signed-off-by: Artem Bityutskiy <[email protected]> > > Signed-off-by: Dominic Curran <[email protected]> > > Signed-off-by: Ilkka Myllyperkio <[email protected]> > > Signed-off-by: Pallavi Kulkarni <[email protected]> > > Signed-off-by: Vaibhav Hiremath <[email protected]> > > 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;
Those two have a kerneldoc comment in front of the structure definition. > 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 ? -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
