All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
@ 2012-03-30  8:20 Paul Mundt
  2012-04-01 23:57 ` Kuninori Morimoto
                   ` (4 more replies)
  0 siblings, 5 replies; 10+ messages in thread
From: Paul Mundt @ 2012-03-30  8:20 UTC (permalink / raw)
  To: linux-sh

Leading up to the ->device_prep_slave_sg change in
185ecb5f4fd43911c35956d4cc7d94a1da30417f 'dmaengine: add context
parameter to prep_slave_sg and prep_dma_cyclic' a generic wrapper was
added in place to guard against the API change, though the fsi driver
wasn't updated in the process (presumably its dmaengine support hadn't
been merged yet at the time). This trivially switches over to the new
wrapper and gets it building again.

Signed-off-by: Paul Mundt <lethal@linux-sh.org>

---

 sound/soc/sh/fsi.c |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c
index 378cc5b..74ed2df 100644
--- a/sound/soc/sh/fsi.c
+++ b/sound/soc/sh/fsi.c
@@ -1001,11 +1001,10 @@ static void fsi_dma_do_tasklet(unsigned long data)
 	sg_dma_address(&sg) = buf;
 	sg_dma_len(&sg) = len;
 
-	desc = chan->device->device_prep_slave_sg(chan, &sg, 1, dir,
-						  DMA_PREP_INTERRUPT |
-						  DMA_CTRL_ACK);
+	desc = dmaengine_prep_slave_sg(chan, &sg, 1, dir,
+				       DMA_PREP_INTERRUPT | DMA_CTRL_ACK);
 	if (!desc) {
-		dev_err(dai->dev, "device_prep_slave_sg() fail\n");
+		dev_err(dai->dev, "dmaengine_prep_slave_sg() fail\n");
 		return;
 	}
 

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

* Re: [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
  2012-03-30  8:20 [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout Paul Mundt
@ 2012-04-01 23:57 ` Kuninori Morimoto
  2012-04-16  6:23 ` Kuninori Morimoto
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 10+ messages in thread
From: Kuninori Morimoto @ 2012-04-01 23:57 UTC (permalink / raw)
  To: linux-sh


Hi

> Leading up to the ->device_prep_slave_sg change in
> 185ecb5f4fd43911c35956d4cc7d94a1da30417f 'dmaengine: add context
> parameter to prep_slave_sg and prep_dma_cyclic' a generic wrapper was
> added in place to guard against the API change, though the fsi driver
> wasn't updated in the process (presumably its dmaengine support hadn't
> been merged yet at the time). This trivially switches over to the new
> wrapper and gets it building again.
> 
> Signed-off-by: Paul Mundt <lethal@linux-sh.org>
> 
> ---

Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Best regards
---
Kuninori Morimoto

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

* Re: [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
  2012-03-30  8:20 [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout Paul Mundt
  2012-04-01 23:57 ` Kuninori Morimoto
@ 2012-04-16  6:23 ` Kuninori Morimoto
  2012-04-16  8:27 ` Mark Brown
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 10+ messages in thread
From: Kuninori Morimoto @ 2012-04-16  6:23 UTC (permalink / raw)
  To: linux-sh


Hi Paul, Mark

> > Leading up to the ->device_prep_slave_sg change in
> > 185ecb5f4fd43911c35956d4cc7d94a1da30417f 'dmaengine: add context
> > parameter to prep_slave_sg and prep_dma_cyclic' a generic wrapper was
> > added in place to guard against the API change, though the fsi driver
> > wasn't updated in the process (presumably its dmaengine support hadn't
> > been merged yet at the time). This trivially switches over to the new
> > wrapper and gets it building again.
> > 
> > Signed-off-by: Paul Mundt <lethal@linux-sh.org>
> > 
> > ---
> 
> Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

This patch is required for current linux kernel, but it Cc:ed to linux-sh ML only.
Who control this? Paul? Mark?

Best regards
---
Kuninori Morimoto

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

* Re: [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
  2012-03-30  8:20 [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout Paul Mundt
  2012-04-01 23:57 ` Kuninori Morimoto
  2012-04-16  6:23 ` Kuninori Morimoto
@ 2012-04-16  8:27 ` Mark Brown
  2012-04-16 12:10 ` Paul Mundt
  2012-04-16 12:17 ` Mark Brown
  4 siblings, 0 replies; 10+ messages in thread
From: Mark Brown @ 2012-04-16  8:27 UTC (permalink / raw)
  To: linux-sh

[-- Attachment #1: Type: text/plain, Size: 232 bytes --]

On Sun, Apr 15, 2012 at 11:23:58PM -0700, Kuninori Morimoto wrote:

> This patch is required for current linux kernel, but it Cc:ed to linux-sh ML only.
> Who control this? Paul? Mark?

Not me at the minute, I don't have the patch.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

* Re: [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
  2012-03-30  8:20 [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout Paul Mundt
                   ` (2 preceding siblings ...)
  2012-04-16  8:27 ` Mark Brown
@ 2012-04-16 12:10 ` Paul Mundt
  2012-04-16 12:17 ` Mark Brown
  4 siblings, 0 replies; 10+ messages in thread
From: Paul Mundt @ 2012-04-16 12:10 UTC (permalink / raw)
  To: linux-sh

On Mon, Apr 16, 2012 at 09:27:02AM +0100, Mark Brown wrote:
> On Sun, Apr 15, 2012 at 11:23:58PM -0700, Kuninori Morimoto wrote:
> 
> > This patch is required for current linux kernel, but it Cc:ed to linux-sh ML only.
> > Who control this? Paul? Mark?
> 
> Not me at the minute, I don't have the patch.

You were Cc'ed on it?

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

* Re: [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
  2012-03-30  8:20 [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout Paul Mundt
                   ` (3 preceding siblings ...)
  2012-04-16 12:10 ` Paul Mundt
@ 2012-04-16 12:17 ` Mark Brown
  2012-04-18  2:13     ` kuninori.morimoto.gx
  4 siblings, 1 reply; 10+ messages in thread
From: Mark Brown @ 2012-04-16 12:17 UTC (permalink / raw)
  To: linux-sh

[-- Attachment #1: Type: text/plain, Size: 643 bytes --]

On Mon, Apr 16, 2012 at 09:10:45PM +0900, Paul Mundt wrote:
> On Mon, Apr 16, 2012 at 09:27:02AM +0100, Mark Brown wrote:
> > On Sun, Apr 15, 2012 at 11:23:58PM -0700, Kuninori Morimoto wrote:

> > > This patch is required for current linux kernel, but it Cc:ed to linux-sh ML only.
> > > Who control this? Paul? Mark?

> > Not me at the minute, I don't have the patch.

> You were Cc'ed on it?

That was in the past.  I currently don't have it.  Probably I didn't do
anything with it as it wasn't sent as an ALSA patch (no CC to either the
list or Liam) so it will have looked like you were just CCing for info.
If you can resend I'll apply.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

* [PATCH forward] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
  2012-04-16 12:17 ` Mark Brown
@ 2012-04-18  2:13     ` kuninori.morimoto.gx
  0 siblings, 0 replies; 10+ messages in thread
From: kuninori.morimoto.gx @ 2012-04-18  2:13 UTC (permalink / raw)
  To: Mark Brown
  Cc: Paul Mundt, Linux-ALSA, Liam Girdwood, Kuninori Morimoto, linux-sh


From: Paul Mundt <lethal@linux-sh.org>

Leading up to the ->device_prep_slave_sg change in
185ecb5f4fd43911c35956d4cc7d94a1da30417f 'dmaengine: add context
parameter to prep_slave_sg and prep_dma_cyclic' a generic wrapper was
added in place to guard against the API change, though the fsi driver
wasn't updated in the process (presumably its dmaengine support hadn't
been merged yet at the time). This trivially switches over to the new
wrapper and gets it building again.

Signed-off-by: Paul Mundt <lethal@linux-sh.org>
Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
Nothing happened on ALSA ML.
So I forward this patch.
I added my Acked-by on it.

 sound/soc/sh/fsi.c |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c
index 378cc5b..74ed2df 100644
--- a/sound/soc/sh/fsi.c
+++ b/sound/soc/sh/fsi.c
@@ -1001,11 +1001,10 @@ static void fsi_dma_do_tasklet(unsigned long data)
 	sg_dma_address(&sg) = buf;
 	sg_dma_len(&sg) = len;
 
-	desc = chan->device->device_prep_slave_sg(chan, &sg, 1, dir,
-						  DMA_PREP_INTERRUPT |
-						  DMA_CTRL_ACK);
+	desc = dmaengine_prep_slave_sg(chan, &sg, 1, dir,
+				       DMA_PREP_INTERRUPT | DMA_CTRL_ACK);
 	if (!desc) {
-		dev_err(dai->dev, "device_prep_slave_sg() fail\n");
+		dev_err(dai->dev, "dmaengine_prep_slave_sg() fail\n");
 		return;
 	}
 

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

* [PATCH forward] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
@ 2012-04-18  2:13     ` kuninori.morimoto.gx
  0 siblings, 0 replies; 10+ messages in thread
From: kuninori.morimoto.gx @ 2012-04-18  2:13 UTC (permalink / raw)
  To: Mark Brown
  Cc: Paul Mundt, Linux-ALSA, Liam Girdwood, Kuninori Morimoto, linux-sh


From: Paul Mundt <lethal@linux-sh.org>

Leading up to the ->device_prep_slave_sg change in
185ecb5f4fd43911c35956d4cc7d94a1da30417f 'dmaengine: add context
parameter to prep_slave_sg and prep_dma_cyclic' a generic wrapper was
added in place to guard against the API change, though the fsi driver
wasn't updated in the process (presumably its dmaengine support hadn't
been merged yet at the time). This trivially switches over to the new
wrapper and gets it building again.

Signed-off-by: Paul Mundt <lethal@linux-sh.org>
Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
Nothing happened on ALSA ML.
So I forward this patch.
I added my Acked-by on it.

 sound/soc/sh/fsi.c |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c
index 378cc5b..74ed2df 100644
--- a/sound/soc/sh/fsi.c
+++ b/sound/soc/sh/fsi.c
@@ -1001,11 +1001,10 @@ static void fsi_dma_do_tasklet(unsigned long data)
 	sg_dma_address(&sg) = buf;
 	sg_dma_len(&sg) = len;
 
-	desc = chan->device->device_prep_slave_sg(chan, &sg, 1, dir,
-						  DMA_PREP_INTERRUPT |
-						  DMA_CTRL_ACK);
+	desc = dmaengine_prep_slave_sg(chan, &sg, 1, dir,
+				       DMA_PREP_INTERRUPT | DMA_CTRL_ACK);
 	if (!desc) {
-		dev_err(dai->dev, "device_prep_slave_sg() fail\n");
+		dev_err(dai->dev, "dmaengine_prep_slave_sg() fail\n");
 		return;
 	}
 

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

* Re: [PATCH forward] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
  2012-04-18  2:13     ` kuninori.morimoto.gx
@ 2012-04-18  9:06       ` Mark Brown
  -1 siblings, 0 replies; 10+ messages in thread
From: Mark Brown @ 2012-04-18  9:06 UTC (permalink / raw)
  To: kuninori.morimoto.gx
  Cc: Paul Mundt, Linux-ALSA, Liam Girdwood, Kuninori Morimoto, linux-sh

[-- Attachment #1: Type: text/plain, Size: 602 bytes --]

On Tue, Apr 17, 2012 at 07:13:04PM -0700, kuninori.morimoto.gx@renesas.com wrote:
> 
> Signed-off-by: Paul Mundt <lethal@linux-sh.org>
> Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
> Nothing happened on ALSA ML.
> So I forward this patch.
> I added my Acked-by on it.

I've applied this since I can see Paul's original version in the archive
but when forwarding patches for someone else you should sign off rather
than ack the patch - signoff is mostly a statement saying "I know this
is OK legally" and so everyone who forwards the patch is supposed to add
one.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

* Re: [PATCH forward] ASoC: fsi: update for dmaengine prep_slave_sg fallout.
@ 2012-04-18  9:06       ` Mark Brown
  0 siblings, 0 replies; 10+ messages in thread
From: Mark Brown @ 2012-04-18  9:06 UTC (permalink / raw)
  To: kuninori.morimoto.gx
  Cc: Paul Mundt, Linux-ALSA, Liam Girdwood, Kuninori Morimoto, linux-sh

[-- Attachment #1: Type: text/plain, Size: 602 bytes --]

On Tue, Apr 17, 2012 at 07:13:04PM -0700, kuninori.morimoto.gx@renesas.com wrote:
> 
> Signed-off-by: Paul Mundt <lethal@linux-sh.org>
> Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
> Nothing happened on ALSA ML.
> So I forward this patch.
> I added my Acked-by on it.

I've applied this since I can see Paul's original version in the archive
but when forwarding patches for someone else you should sign off rather
than ack the patch - signoff is mostly a statement saying "I know this
is OK legally" and so everyone who forwards the patch is supposed to add
one.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

end of thread, other threads:[~2012-04-18  9:06 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-03-30  8:20 [PATCH] ASoC: fsi: update for dmaengine prep_slave_sg fallout Paul Mundt
2012-04-01 23:57 ` Kuninori Morimoto
2012-04-16  6:23 ` Kuninori Morimoto
2012-04-16  8:27 ` Mark Brown
2012-04-16 12:10 ` Paul Mundt
2012-04-16 12:17 ` Mark Brown
2012-04-18  2:13   ` [PATCH forward] " kuninori.morimoto.gx
2012-04-18  2:13     ` kuninori.morimoto.gx
2012-04-18  9:06     ` Mark Brown
2012-04-18  9:06       ` Mark Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.