From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756808Ab2DZOIv (ORCPT ); Thu, 26 Apr 2012 10:08:51 -0400 Received: from mailout1.w1.samsung.com ([210.118.77.11]:57019 "EHLO mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752418Ab2DZOIu (ORCPT ); Thu, 26 Apr 2012 10:08:50 -0400 Date: Thu, 26 Apr 2012 16:08:47 +0200 From: Marek Szyprowski Subject: RE: [PATCH] media: videobuf2-dma-contig: include header for exported symbols In-reply-to: <201204241608.13305.hartleys@visionengravers.com> To: "'H Hartley Sweeten'" , "'Linux Kernel'" Cc: linux-media@vger.kernel.org, pawel@osciak.com, kyungmin.park@samsung.com, mchehab@infradead.org Message-id: <004c01cd23b6$19342440$4b9c6cc0$%szyprowski@samsung.com> Organization: SPRC MIME-version: 1.0 X-Mailer: Microsoft Office Outlook 12.0 Content-type: text/plain; charset=utf-8 Content-language: pl Content-transfer-encoding: 7BIT Thread-index: Ac0ibyfqFx+OA7KXQ6C/OfQkk2nF5QBRt2ew References: <201204241608.13305.hartleys@visionengravers.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > Cc: Pawel Osciak > Cc: Marek Szyprowski > Cc: Kyungmin Park > Cc: Mauro Carvalho Chehab Acked-by: Marek Szyprowski > --- > > 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 > > #include > +#include > #include > > struct vb2_dc_conf { Best regards -- Marek Szyprowski Samsung Poland R&D Center