RE: [PATCH] videobuf-dma-contig.c: add missing #include

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

 



> -----Original Message-----
> From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Martin Michlmayr
> Sent: Wednesday, November 11, 2009 9:24 PM
> To: linux-media@xxxxxxxxxxxxxxx
> Cc: Mauro Carvalho Chehab
> Subject: Re: [PATCH] videobuf-dma-contig.c: add missing #include
> 
> Are there any comments regarding the build fix I submitted?  This
> issue is still there, as you can see at
> http://www.xs4all.nl/~hverkuil/logs/Tuesday.log
> 
[Hiremath, Vaibhav] Tested by me, and it is required, since this is breaking the build. Should get merged.

Thanks,
Vaibhav

> * Martin Michlmayr <tbm@xxxxxxxxxx> [2009-10-31 10:28]:
> > media/video/videobuf-dma-contig.c fails to compile on ARM
> Versatile
> > like this:
> >  | videobuf-dma-contig.c: In function
> 'videobuf_dma_contig_user_get':
> >  | videobuf-dma-contig.c:139: error: dereferencing pointer to
> incomplete type
> >  | videobuf-dma-contig.c:184: error: dereferencing pointer to
> incomplete type
> >  | make[8]: *** [drivers/media/video/videobuf-dma-contig.o] Error
> 1
> >
> > Looking at the preprocessed source, I noticed that there was no
> definition
> > for struct task_struct.
> >
> > Signed-off-by: Martin Michlmayr <tbm@xxxxxxxxxx>
> >
> > --- a/drivers/media/video/videobuf-dma-contig.c	2009-10-31
> 10:22:42.000000000 +0000
> > +++ b/drivers/media/video/videobuf-dma-contig.c	2009-10-31
> 10:24:40.000000000 +0000
> > @@ -19,6 +19,7 @@
> >  #include <linux/mm.h>
> >  #include <linux/pagemap.h>
> >  #include <linux/dma-mapping.h>
> > +#include <linux/sched.h>
> >  #include <media/videobuf-dma-contig.h>
> >
> >  struct videobuf_dma_contig_memory {
> >
> > --
> > Martin Michlmayr
> > http://www.cyrius.com/
> 
> --
> Martin Michlmayr
> http://www.cyrius.com/
> --
> 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

--
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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux