ASoC: Intel: Delete an unnecessary check before the function call "sst_dma_free"
authorMarkus Elfring <elfring@users.sourceforge.net>
Sat, 3 Jan 2015 18:03:55 +0000 (19:03 +0100)
committerMark Brown <broonie@kernel.org>
Mon, 5 Jan 2015 19:02:35 +0000 (19:02 +0000)
The sst_dma_free() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/sst-dsp.c

index 86e41084567002ebb999dca591b598a7224690fe..64e94212d2d2b129ebeddf9f1e4e55ac24775826 100644 (file)
@@ -410,8 +410,7 @@ void sst_dsp_free(struct sst_dsp *sst)
        if (sst->ops->free)
                sst->ops->free(sst);
 
-       if (sst->dma)
-               sst_dma_free(sst->dma);
+       sst_dma_free(sst->dma);
 }
 EXPORT_SYMBOL_GPL(sst_dsp_free);