dmaengine: st_fdma: drop useless LIST_HEAD
authorJulia Lawall <Julia.Lawall@lip6.fr>
Sun, 23 Dec 2018 08:57:09 +0000 (09:57 +0100)
committerVinod Koul <vkoul@kernel.org>
Mon, 7 Jan 2019 04:19:26 +0000 (09:49 +0530)
Drop LIST_HEAD where the variable it declares is never used.

The declarations were introduced with the file, but the declared
variables were not used.

The semantic patch that fixes this problem is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@
identifier x;
@@
- LIST_HEAD(x);
  ... when != x
// </smpl>

Fixes: 6b4cd727eaf1 ("dmaengine: st_fdma: Add STMicroelectronics FDMA engine driver support")
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/dma/st_fdma.c

index 07c20aa2e955a432c007c838484bb23dd75365d7..954f7cf62bb105e3cec7b87239339a52e4d7db49 100644 (file)
@@ -294,8 +294,6 @@ static void st_fdma_free_chan_res(struct dma_chan *chan)
        struct rproc *rproc = fchan->fdev->slim_rproc->rproc;
        unsigned long flags;
 
-       LIST_HEAD(head);
-
        dev_dbg(fchan->fdev->dev, "%s: freeing chan:%d\n",
                __func__, fchan->vchan.chan.chan_id);
 
@@ -626,7 +624,6 @@ static void st_fdma_issue_pending(struct dma_chan *chan)
 static int st_fdma_pause(struct dma_chan *chan)
 {
        unsigned long flags;
-       LIST_HEAD(head);
        struct st_fdma_chan *fchan = to_st_fdma_chan(chan);
        int ch_id = fchan->vchan.chan.chan_id;
        unsigned long cmd = FDMA_CMD_PAUSE(ch_id);