* [PATCH] arm64: fix build error if DMA_CMA is enabled
@ 2014-01-24 8:23 Pankaj Dubey
2014-01-24 15:37 ` Catalin Marinas
0 siblings, 1 reply; 3+ messages in thread
From: Pankaj Dubey @ 2014-01-24 8:23 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel
Cc: catalin.marinas, will.deacon, lauraa, Pankaj Dubey, Somraj Mani
arm64/include/asm/dma-contiguous.h is trying to include
<asm-genric/dma-contiguous.h> which does not exist, and thus failing
build for arm64 if we enable CONFIG_DMA_CMA. This patch fixes build
error by removing unwanted header inclusion from arm64's dma-contiguous.h.
Signed-off-by: Pankaj Dubey <pankaj.dubey@samsung.com>
Signed-off-by: Somraj Mani <somraj.mani@samsung.com>
---
arch/arm64/include/asm/dma-contiguous.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/include/asm/dma-contiguous.h b/arch/arm64/include/asm/dma-contiguous.h
index d6aacb6..14c4c0c 100644
--- a/arch/arm64/include/asm/dma-contiguous.h
+++ b/arch/arm64/include/asm/dma-contiguous.h
@@ -18,7 +18,6 @@
#ifdef CONFIG_DMA_CMA
#include <linux/types.h>
-#include <asm-generic/dma-contiguous.h>
static inline void
dma_contiguous_early_fixup(phys_addr_t base, unsigned long size) { }
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: fix build error if DMA_CMA is enabled
2014-01-24 8:23 [PATCH] arm64: fix build error if DMA_CMA is enabled Pankaj Dubey
@ 2014-01-24 15:37 ` Catalin Marinas
2014-01-24 22:13 ` Laura Abbott
0 siblings, 1 reply; 3+ messages in thread
From: Catalin Marinas @ 2014-01-24 15:37 UTC (permalink / raw)
To: Pankaj Dubey
Cc: linux-arm-kernel, linux-kernel, Will Deacon, lauraa, Somraj Mani
On Fri, Jan 24, 2014 at 08:23:08AM +0000, Pankaj Dubey wrote:
> arm64/include/asm/dma-contiguous.h is trying to include
> <asm-genric/dma-contiguous.h> which does not exist, and thus failing
> build for arm64 if we enable CONFIG_DMA_CMA. This patch fixes build
> error by removing unwanted header inclusion from arm64's dma-contiguous.h.
>
> Signed-off-by: Pankaj Dubey <pankaj.dubey@samsung.com>
> Signed-off-by: Somraj Mani <somraj.mani@samsung.com>
> ---
> arch/arm64/include/asm/dma-contiguous.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/include/asm/dma-contiguous.h b/arch/arm64/include/asm/dma-contiguous.h
> index d6aacb6..14c4c0c 100644
> --- a/arch/arm64/include/asm/dma-contiguous.h
> +++ b/arch/arm64/include/asm/dma-contiguous.h
> @@ -18,7 +18,6 @@
> #ifdef CONFIG_DMA_CMA
>
> #include <linux/types.h>
> -#include <asm-generic/dma-contiguous.h>
Thanks for this.
Laura, did you have additional patches adding
asm-generic/dma-contiguous.h?
--
Catalin
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: fix build error if DMA_CMA is enabled
2014-01-24 15:37 ` Catalin Marinas
@ 2014-01-24 22:13 ` Laura Abbott
0 siblings, 0 replies; 3+ messages in thread
From: Laura Abbott @ 2014-01-24 22:13 UTC (permalink / raw)
To: Catalin Marinas, Pankaj Dubey
Cc: linux-arm-kernel, linux-kernel, Will Deacon, Somraj Mani
On 1/24/2014 7:37 AM, Catalin Marinas wrote:
> On Fri, Jan 24, 2014 at 08:23:08AM +0000, Pankaj Dubey wrote:
>> arm64/include/asm/dma-contiguous.h is trying to include
>> <asm-genric/dma-contiguous.h> which does not exist, and thus failing
>> build for arm64 if we enable CONFIG_DMA_CMA. This patch fixes build
>> error by removing unwanted header inclusion from arm64's dma-contiguous.h.
>>
>> Signed-off-by: Pankaj Dubey <pankaj.dubey@samsung.com>
>> Signed-off-by: Somraj Mani <somraj.mani@samsung.com>
>> ---
>> arch/arm64/include/asm/dma-contiguous.h | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/arch/arm64/include/asm/dma-contiguous.h b/arch/arm64/include/asm/dma-contiguous.h
>> index d6aacb6..14c4c0c 100644
>> --- a/arch/arm64/include/asm/dma-contiguous.h
>> +++ b/arch/arm64/include/asm/dma-contiguous.h
>> @@ -18,7 +18,6 @@
>> #ifdef CONFIG_DMA_CMA
>>
>> #include <linux/types.h>
>> -#include <asm-generic/dma-contiguous.h>
>
> Thanks for this.
>
> Laura, did you have additional patches adding
> asm-generic/dma-contiguous.h?
>
no, asm-generic/dma-contiguous.h was an old file which was later
removed. I missed this when rebasing from my older branch to mainline.
You can have
Acked-by: Laura Abbott <lauraa@codeaurora.org>
Thanks,
Laura
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-01-24 22:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-24 8:23 [PATCH] arm64: fix build error if DMA_CMA is enabled Pankaj Dubey
2014-01-24 15:37 ` Catalin Marinas
2014-01-24 22:13 ` Laura Abbott
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).