* [PATCH] ARM: ux500: set coherent_dma_mask for dma40
@ 2013-06-13 13:56 Fabio Baltieri
2013-06-17 15:42 ` Linus Walleij
0 siblings, 1 reply; 3+ messages in thread
From: Fabio Baltieri @ 2013-06-13 13:56 UTC (permalink / raw)
To: Linus Walleij, Lee Jones
Cc: Srinidhi Kasagar, Vinod Koul, linux-arm-kernel, linux-kernel,
Fabio Baltieri
Set coherent_dma_mask to DMA_BIT_MASK(32) for dma40 platform_device, as
without this DMA allocations were failing with the error:
dma40 dma40.0: coherent DMA mask is unset
when booting without device-tree.
Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org>
---
Hi Linus, Lee,
I found this while removing the last hack I was keeping in my ASoC tree.
I originally thought that I had to set this on the driver's pdata, but
it turns out it's needed on the DMA controller one instead.
When booting with device-tree enabled the mask seems to be set
automatically.
Would you consider applying this with the other dma40 patches?
Thanks,
Fabio
arch/arm/mach-ux500/devices-db8500.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-ux500/devices-db8500.c b/arch/arm/mach-ux500/devices-db8500.c
index e21ffd8..fa8f7a5 100644
--- a/arch/arm/mach-ux500/devices-db8500.c
+++ b/arch/arm/mach-ux500/devices-db8500.c
@@ -49,6 +49,7 @@ struct stedma40_platform_data dma40_plat_data = {
struct platform_device u8500_dma40_device = {
.dev = {
.platform_data = &dma40_plat_data,
+ .coherent_dma_mask = DMA_BIT_MASK(32),
},
.name = "dma40",
.id = 0,
--
1.8.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: ux500: set coherent_dma_mask for dma40
2013-06-13 13:56 [PATCH] ARM: ux500: set coherent_dma_mask for dma40 Fabio Baltieri
@ 2013-06-17 15:42 ` Linus Walleij
2013-06-18 8:53 ` Fabio Baltieri
0 siblings, 1 reply; 3+ messages in thread
From: Linus Walleij @ 2013-06-17 15:42 UTC (permalink / raw)
To: Fabio Baltieri
Cc: Lee Jones, Srinidhi Kasagar, Vinod Koul, linux-arm-kernel, linux-kernel
On Thu, Jun 13, 2013 at 3:56 PM, Fabio Baltieri
<fabio.baltieri@linaro.org> wrote:
> Set coherent_dma_mask to DMA_BIT_MASK(32) for dma40 platform_device, as
> without this DMA allocations were failing with the error:
>
> dma40 dma40.0: coherent DMA mask is unset
>
> when booting without device-tree.
>
> Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org>
> ---
>
> Hi Linus, Lee,
>
> I found this while removing the last hack I was keeping in my ASoC tree.
> I originally thought that I had to set this on the driver's pdata, but
> it turns out it's needed on the DMA controller one instead.
>
> When booting with device-tree enabled the mask seems to be set
> automatically.
>
> Would you consider applying this with the other dma40 patches?
Those are now upstream in the ARM SoC tree, you'd have to send it
to arm@kernel.org and ask Olof/Arnd to apply it directly to
the next/drivers branch (I think).
I have also queued it on ux500-fixes so it won't be lost.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: ux500: set coherent_dma_mask for dma40
2013-06-17 15:42 ` Linus Walleij
@ 2013-06-18 8:53 ` Fabio Baltieri
0 siblings, 0 replies; 3+ messages in thread
From: Fabio Baltieri @ 2013-06-18 8:53 UTC (permalink / raw)
To: Linus Walleij
Cc: Lee Jones, Srinidhi Kasagar, Vinod Koul, linux-arm-kernel, linux-kernel
On Mon, Jun 17, 2013 at 05:42:48PM +0200, Linus Walleij wrote:
> On Thu, Jun 13, 2013 at 3:56 PM, Fabio Baltieri
> <fabio.baltieri@linaro.org> wrote:
>
> > Set coherent_dma_mask to DMA_BIT_MASK(32) for dma40 platform_device, as
> > without this DMA allocations were failing with the error:
> >
> > dma40 dma40.0: coherent DMA mask is unset
> >
> > when booting without device-tree.
> >
> > Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org>
> > ---
> >
> > Hi Linus, Lee,
> >
> > I found this while removing the last hack I was keeping in my ASoC tree.
> > I originally thought that I had to set this on the driver's pdata, but
> > it turns out it's needed on the DMA controller one instead.
> >
> > When booting with device-tree enabled the mask seems to be set
> > automatically.
> >
> > Would you consider applying this with the other dma40 patches?
>
> Those are now upstream in the ARM SoC tree, you'd have to send it
> to arm@kernel.org and ask Olof/Arnd to apply it directly to
> the next/drivers branch (I think).
Ok I can resend it but I'd like to have Lee's Ack before doing that.
Lee: have you had a chance to look at this patch?
> I have also queued it on ux500-fixes so it won't be lost.
That's nice!
Thanks,
Fabio
--
Fabio Baltieri
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-18 8:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-13 13:56 [PATCH] ARM: ux500: set coherent_dma_mask for dma40 Fabio Baltieri
2013-06-17 15:42 ` Linus Walleij
2013-06-18 8:53 ` Fabio Baltieri
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).