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