linux-sh.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM
@ 2010-03-19  4:47 Magnus Damm
  2010-03-19  7:24 ` Guennadi Liakhovetski
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Magnus Damm @ 2010-03-19  4:47 UTC (permalink / raw)
  To: linux-sh

From: Magnus Damm <damm@opensource.se>

Enable the shdma dmaengine driver on SH-Mobile ARM.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 drivers/dma/Kconfig        |    2 +-
 drivers/dma/shdma.c        |    8 ++++----
 include/linux/serial_sci.h |    2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

--- 0001/drivers/dma/Kconfig
+++ work/drivers/dma/Kconfig	2010-03-18 23:38:59.000000000 +0900
@@ -128,7 +128,7 @@ config TXX9_DMAC
 
 config SH_DMAE
 	tristate "Renesas SuperH DMAC support"
-	depends on SUPERH && SH_DMA
+	depends on (SUPERH && SH_DMA) || (ARM && ARCH_SHMOBILE)
 	depends on !SH_DMA_API
 	select DMA_ENGINE
 	help
--- 0008/drivers/dma/shdma.c
+++ work/drivers/dma/shdma.c	2010-03-19 12:39:33.000000000 +0900
@@ -793,7 +793,7 @@ static irqreturn_t sh_dmae_interrupt(int
 	return ret;
 }
 
-#if defined(CONFIG_CPU_SH4)
+#if defined(CONFIG_CPU_SH4) || defined(CONFIG_ARCH_SHMOBILE)
 static irqreturn_t sh_dmae_err(int irq, void *data)
 {
 	struct sh_dmae_device *shdev = (struct sh_dmae_device *)data;
@@ -1034,7 +1034,7 @@ static int __init sh_dmae_probe(struct p
 	/* Default transfer size of 32 bytes requires 32-byte alignment */
 	shdev->common.copy_align = LOG2_DEFAULT_XFER_SIZE;
 
-#if defined(CONFIG_CPU_SH4)
+#if defined(CONFIG_CPU_SH4) || defined(CONFIG_ARCH_SHMOBILE)
 	chanirq_res = platform_get_resource(pdev, IORESOURCE_IRQ, 1);
 
 	if (!chanirq_res)
@@ -1059,7 +1059,7 @@ static int __init sh_dmae_probe(struct p
 
 #else
 	chanirq_res = errirq_res;
-#endif /* CONFIG_CPU_SH4 */
+#endif /* CONFIG_CPU_SH4 || CONFIG_ARCH_SHMOBILE */
 
 	if (chanirq_res->start = chanirq_res->end &&
 	    !platform_get_resource(pdev, IORESOURCE_IRQ, 1)) {
@@ -1106,7 +1106,7 @@ static int __init sh_dmae_probe(struct p
 chan_probe_err:
 	sh_dmae_chan_remove(shdev);
 eirqres:
-#if defined(CONFIG_CPU_SH4)
+#if defined(CONFIG_CPU_SH4) || defined(CONFIG_ARCH_SHMOBILE)
 	free_irq(errirq, shdev);
 eirq_err:
 #endif
--- 0005/include/linux/serial_sci.h
+++ work/include/linux/serial_sci.h	2010-03-18 23:38:59.000000000 +0900
@@ -3,7 +3,7 @@
 
 #include <linux/serial_core.h>
 #ifdef CONFIG_SERIAL_SH_SCI_DMA
-#include <asm/dmaengine.h>
+#include <linux/sh_dma.h>
 #endif
 
 /*

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM
  2010-03-19  4:47 [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM Magnus Damm
@ 2010-03-19  7:24 ` Guennadi Liakhovetski
  2010-03-19  8:15 ` Magnus Damm
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Guennadi Liakhovetski @ 2010-03-19  7:24 UTC (permalink / raw)
  To: linux-sh

On Fri, 19 Mar 2010, Magnus Damm wrote:

> From: Magnus Damm <damm@opensource.se>
> 
> Enable the shdma dmaengine driver on SH-Mobile ARM.
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>
> ---

[snip]

> --- 0005/include/linux/serial_sci.h
> +++ work/include/linux/serial_sci.h	2010-03-18 23:38:59.000000000 +0900
> @@ -3,7 +3,7 @@
>  
>  #include <linux/serial_core.h>
>  #ifdef CONFIG_SERIAL_SH_SCI_DMA
> -#include <asm/dmaengine.h>
> +#include <linux/sh_dma.h>

Shouldn't this be done in patch "04/05"?

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM
  2010-03-19  4:47 [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM Magnus Damm
  2010-03-19  7:24 ` Guennadi Liakhovetski
@ 2010-03-19  8:15 ` Magnus Damm
  2010-03-19  8:15 ` Paul Mundt
  2010-04-07  7:34 ` Paul Mundt
  3 siblings, 0 replies; 5+ messages in thread
From: Magnus Damm @ 2010-03-19  8:15 UTC (permalink / raw)
  To: linux-sh

Hey Guennadi,

On Fri, Mar 19, 2010 at 4:24 PM, Guennadi Liakhovetski
<g.liakhovetski@gmx.de> wrote:
> On Fri, 19 Mar 2010, Magnus Damm wrote:
>> --- 0005/include/linux/serial_sci.h
>> +++ work/include/linux/serial_sci.h   2010-03-18 23:38:59.000000000 +0900
>> @@ -3,7 +3,7 @@
>>
>>  #include <linux/serial_core.h>
>>  #ifdef CONFIG_SERIAL_SH_SCI_DMA
>> -#include <asm/dmaengine.h>
>> +#include <linux/sh_dma.h>
>
> Shouldn't this be done in patch "04/05"?

It could, but nothing requires this change in 04/05. So the final
point where this is needed is 05/05.

/ magnus

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM
  2010-03-19  4:47 [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM Magnus Damm
  2010-03-19  7:24 ` Guennadi Liakhovetski
  2010-03-19  8:15 ` Magnus Damm
@ 2010-03-19  8:15 ` Paul Mundt
  2010-04-07  7:34 ` Paul Mundt
  3 siblings, 0 replies; 5+ messages in thread
From: Paul Mundt @ 2010-03-19  8:15 UTC (permalink / raw)
  To: linux-sh

On Fri, Mar 19, 2010 at 05:15:42PM +0900, Magnus Damm wrote:
> Hey Guennadi,
> 
> On Fri, Mar 19, 2010 at 4:24 PM, Guennadi Liakhovetski
> <g.liakhovetski@gmx.de> wrote:
> > On Fri, 19 Mar 2010, Magnus Damm wrote:
> >> --- 0005/include/linux/serial_sci.h
> >> +++ work/include/linux/serial_sci.h ? 2010-03-18 23:38:59.000000000 +0900
> >> @@ -3,7 +3,7 @@
> >>
> >> ?#include <linux/serial_core.h>
> >> ?#ifdef CONFIG_SERIAL_SH_SCI_DMA
> >> -#include <asm/dmaengine.h>
> >> +#include <linux/sh_dma.h>
> >
> > Shouldn't this be done in patch "04/05"?
> 
> It could, but nothing requires this change in 04/05. So the final
> point where this is needed is 05/05.
> 
The ifdefs are also superfluous at this point, but those can be killed
off with an incremental patch.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM
  2010-03-19  4:47 [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM Magnus Damm
                   ` (2 preceding siblings ...)
  2010-03-19  8:15 ` Paul Mundt
@ 2010-04-07  7:34 ` Paul Mundt
  3 siblings, 0 replies; 5+ messages in thread
From: Paul Mundt @ 2010-04-07  7:34 UTC (permalink / raw)
  To: linux-sh

On Fri, Mar 19, 2010 at 01:47:19PM +0900, Magnus Damm wrote:
> Enable the shdma dmaengine driver on SH-Mobile ARM.

Applied to the genesis tree, on top of the sh/dmaengine topic branch.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2010-04-07  7:34 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-19  4:47 [PATCH 05/05] dmaengine: shdma: Enable on SH-Mobile ARM Magnus Damm
2010-03-19  7:24 ` Guennadi Liakhovetski
2010-03-19  8:15 ` Magnus Damm
2010-03-19  8:15 ` Paul Mundt
2010-04-07  7:34 ` Paul Mundt

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).