Hi, On Wednesday, April 25, 2012 1:13 AM H Hartley Sweeten wrote: > The function vb2_dma_contig_vaddr returns a void * not an integer. > > Quiets the sparse noise: > > warning: Using plain integer as NULL pointer > > 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..7de6843 100644 > --- a/drivers/media/video/videobuf2-dma-contig.c > +++ b/drivers/media/video/videobuf2-dma-contig.c > @@ -85,7 +85,7 @@ static void *vb2_dma_contig_vaddr(void *buf_priv) > { > struct vb2_dc_buf *buf = buf_priv; > if (!buf) > - return 0; > + return NULL; > > return buf->vaddr; > } 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