On Sun, 5 Apr 2009 12:26:45 +0200 (CEST) Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote: > Hi Mauro, > > On Sun, 5 Apr 2009, Mauro Carvalho Chehab wrote: > > > On Fri, 3 Apr 2009 21:34:59 +0200 (CEST) > > Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote: > > > > > Hi Mauro, > > > > > > > > 04/04: Add camera (CSI) driver for MX1 > > > http://linuxtv.org/hg/~gliakhovetski/v4l-dvb?cmd=changeset;node=f115ddf29d64 > > > > There are two checkpatch.pl warnings on the last one. Please add later a patch to fix. > > > > WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable > > #128: FILE: linux/arch/arm/mach-mx1/ksym_mx1.c:17: > > +EXPORT_SYMBOL(mx1_camera_sof_fiq_start); > > > > WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable > > #129: FILE: linux/arch/arm/mach-mx1/ksym_mx1.c:18: > > +EXPORT_SYMBOL(mx1_camera_sof_fiq_end); > > They are irrelevant. These two symbols are defined in assembly, so, you > cannot put EXPORT_SYMBOL() there. OK. Cheers, Mauro -- 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
