On Tuesday 04 May 2010 10:42:08 pm Randy Dunlap wrote: > On 05/04/10 12:12, Greg KH wrote: > > On Tue, May 04, 2010 at 11:36:04AM -0700, Randy Dunlap wrote: > >> On Tue, 4 May 2010 16:23:05 +1000 Stephen Rothwell wrote: > >> > >>> Hi all, > >>> > >>> Changes since 20100503: > >> > >> > >> drivers/staging/dt3155v4l/dt3155v4l.c:73: error: implicit declaration of function 'udelay' > >> drivers/staging/dt3155v4l/dt3155v4l.c:616: warning: assignment from incompatible pointer type > >> drivers/staging/dt3155v4l/dt3155v4l.c:1270: error: implicit declaration of function 'msleep' > > > > Ick. What arch was this? > > That is on x86_64. > > The config is attached. > It's my fault, sorry. Here is a patch solving the problem. Compiles with Randy's config, but does not link drivers/staging/built-in.o: In function `format_by_fourcc': (.text+0x211da): multiple definition of `format_by_fourcc' drivers/media/built-in.o:(.text+0x2a744): first defined here drivers/media/common/saa7146_video.c:struct saa7146_format* format_by_fourcc(struct saa7146_dev *dev, int fourcc) drivers/staging/cx25821/cx25821-video.c:struct cx25821_fmt *format_by_fourcc(unsigned int fourcc) Marin MItov ========================================= [PATCH] Adding the missing linux/delay.h The prototypes of udelay() and msleep() are in linux/delay.h, so include it. Signed-off-by: Marin Mitov <mitov@xxxxxxxxxxx> ========================================================================== --- a/drivers/staging/dt3155v4l/dt3155v4l.c 2010-05-05 08:55:30.000000000 +0300 +++ b/drivers/staging/dt3155v4l/dt3155v4l.c 2010-05-05 08:57:00.000000000 +0300 @@ -4,6 +4,7 @@ #include <linux/pci.h> #include <linux/version.h> #include <linux/stringify.h> +#include <linux/delay.h> #include <media/videobuf-dma-contig.h> #include <linux/kthread.h> -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
