Re: [PATCH v6 07/10] omap3isp: CCP2/CSI2 receivers

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

 



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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux