From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kukjin Kim Subject: RE: [PATCH v2] ARM: Samsung: s5p-fimc: fix dma coherent mask Date: Thu, 09 Sep 2010 20:20:17 +0900 Message-ID: <011101cb5010$fdb1d520$f9157f60$%kim@samsung.com> References: <000301cb4fb7$b4448530$1ccd8f90$%kim@samsung.com> <1283998673-14899-1-git-send-email-m.szyprowski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7BIT Return-path: Received: from mailout2.samsung.com ([203.254.224.25]:55170 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752443Ab0IILUK (ORCPT ); Thu, 9 Sep 2010 07:20:10 -0400 Received: from epmmp2 (mailout2.samsung.com [203.254.224.25]) by mailout2.samsung.com (Sun Java(tm) System Messaging Server 7u3-15.01 64bit (built Feb 12 2010)) with ESMTP id <0L8H00MP58TFMY40@mailout2.samsung.com> for linux-samsung-soc@vger.kernel.org; Thu, 09 Sep 2010 20:20:04 +0900 (KST) Received: from kgenekim ([12.23.103.96]) by mmp2.samsung.com (iPlanet Messaging Server 5.2 Patch 2 (built Jul 14 2004)) with ESMTPA id <0L8H00GNQ8TFUH@mmp2.samsung.com> for linux-samsung-soc@vger.kernel.org; Thu, 09 Sep 2010 20:20:04 +0900 (KST) In-reply-to: <1283998673-14899-1-git-send-email-m.szyprowski@samsung.com> Content-language: ko Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: 'Marek Szyprowski' , linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: kyungmin.park@samsung.com, ben-linux@fluff.org Marek Szyprowski wrote: > > FIMC driver uses DMA_coherent allocator, which requires proper dma mask > to be set. > > Signed-off-by: Marek Szyprowski > Signed-off-by: Kyungmin Park > --- > > Chnages since v1: > - use DMA_BIT_MASK() macro, minor coding style change > > --- > arch/arm/plat-s5p/dev-fimc0.c | 7 +++++++ > arch/arm/plat-s5p/dev-fimc1.c | 7 +++++++ > arch/arm/plat-s5p/dev-fimc2.c | 7 +++++++ > 3 files changed, 21 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/plat-s5p/dev-fimc0.c b/arch/arm/plat-s5p/dev-fimc0.c > index d3f1a9b..f9203fa 100644 > --- a/arch/arm/plat-s5p/dev-fimc0.c > +++ b/arch/arm/plat-s5p/dev-fimc0.c > @@ -10,6 +10,7 @@ > */ > > #include > +#include > #include > #include > #include > @@ -28,9 +29,15 @@ static struct resource s5p_fimc0_resource[] = { > }, > }; > > +static u64 s5p_fimc0_dma_mask = DMA_BIT_MASK(32); > + > struct platform_device s5p_device_fimc0 = { > .name = "s5p-fimc", > .id = 0, > .num_resources = ARRAY_SIZE(s5p_fimc0_resource), > .resource = s5p_fimc0_resource, > + .dev = { > + .dma_mask = &s5p_fimc0_dma_mask, > + .coherent_dma_mask = DMA_BIT_MASK(32), > + }, > }; > diff --git a/arch/arm/plat-s5p/dev-fimc1.c b/arch/arm/plat-s5p/dev-fimc1.c > index 41bd698..7ed91a3 100644 > --- a/arch/arm/plat-s5p/dev-fimc1.c > +++ b/arch/arm/plat-s5p/dev-fimc1.c > @@ -10,6 +10,7 @@ > */ > > #include > +#include > #include > #include > #include > @@ -28,9 +29,15 @@ static struct resource s5p_fimc1_resource[] = { > }, > }; > > +static u64 s5p_fimc1_dma_mask = DMA_BIT_MASK(32); > + > struct platform_device s5p_device_fimc1 = { > .name = "s5p-fimc", > .id = 1, > .num_resources = ARRAY_SIZE(s5p_fimc1_resource), > .resource = s5p_fimc1_resource, > + .dev = { > + .dma_mask = &s5p_fimc1_dma_mask, > + .coherent_dma_mask = DMA_BIT_MASK(32), > + }, > }; > diff --git a/arch/arm/plat-s5p/dev-fimc2.c b/arch/arm/plat-s5p/dev-fimc2.c > index dfddeda..5543d6373 100644 > --- a/arch/arm/plat-s5p/dev-fimc2.c > +++ b/arch/arm/plat-s5p/dev-fimc2.c > @@ -10,6 +10,7 @@ > */ > > #include > +#include > #include > #include > #include > @@ -28,9 +29,15 @@ static struct resource s5p_fimc2_resource[] = { > }, > }; > > +static u64 s5p_fimc2_dma_mask = DMA_BIT_MASK(32); > + > struct platform_device s5p_device_fimc2 = { > .name = "s5p-fimc", > .id = 2, > .num_resources = ARRAY_SIZE(s5p_fimc2_resource), > .resource = s5p_fimc2_resource, > + .dev = { > + .dma_mask = &s5p_fimc2_dma_mask, > + .coherent_dma_mask = DMA_BIT_MASK(32), > + }, > }; > -- > 1.7.2.2 looks good to me..will apply :-) Thanks. Best regards, Kgene. -- Kukjin Kim , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd.