Hi, On Wednesday, April 25, 2012 1:08 AM H Hartley Sweeten wrote: > Include the header to pickup the definitions of the exported symbols. > > Quiets the following sparse warnings: > > warning: symbol 'vb2_dma_contig_memops' was not declared. Should it be static? > warning: symbol 'vb2_dma_contig_init_ctx' was not declared. Should it be static? > warning: symbol 'vb2_dma_contig_cleanup_ctx' was not declared. Should it be static? > > Signed-off-by: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx> > Cc: Pawel Osciak <pawel@xxxxxxxxxx> > Cc: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > Cc: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> Acked-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > --- > > diff --git a/drivers/media/video/videobuf2-dma-contig.c b/drivers/media/video/videobuf2-dma- > contig.c > index f17ad98..a1bee6c 100644 > --- a/drivers/media/video/videobuf2-dma-contig.c > +++ b/drivers/media/video/videobuf2-dma-contig.c > @@ -15,6 +15,7 @@ > #include <linux/dma-mapping.h> > > #include <media/videobuf2-core.h> > +#include <media/videobuf2-dma-contig.h> > #include <media/videobuf2-memops.h> > > struct vb2_dc_conf { Best regards -- Marek Szyprowski Samsung Poland R&D Center -- 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