ASoC: rsnd: dma.c: spin lock is no longer needed in IRQ handler
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Wed, 29 Nov 2017 03:08:35 +0000 (03:08 +0000)
committerMark Brown <broonie@kernel.org>
Wed, 29 Nov 2017 10:16:48 +0000 (10:16 +0000)
DMA handler had needed to calculate pointer before, but it doesn't
need now. Thus, we can remove unnecessary spin lock from DMAC handler.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sh/rcar/dma.c

index 4d750bd..41de234 100644 (file)
@@ -71,25 +71,7 @@ static struct rsnd_mod mem = {
 static void __rsnd_dmaen_complete(struct rsnd_mod *mod,
                                  struct rsnd_dai_stream *io)
 {
-       struct rsnd_priv *priv = rsnd_mod_to_priv(mod);
-       bool elapsed = false;
-       unsigned long flags;
-
-       /*
-        * Renesas sound Gen1 needs 1 DMAC,
-        * Gen2 needs 2 DMAC.
-        * In Gen2 case, it are Audio-DMAC, and Audio-DMAC-peri-peri.
-        * But, Audio-DMAC-peri-peri doesn't have interrupt,
-        * and this driver is assuming that here.
-        */
-       spin_lock_irqsave(&priv->lock, flags);
-
        if (rsnd_io_is_working(io))
-               elapsed = true;
-
-       spin_unlock_irqrestore(&priv->lock, flags);
-
-       if (elapsed)
                rsnd_dai_period_elapsed(io);
 }