dmaengine: mmp-tdma: fix terminate_all return code
authorArnd Bergmann <arnd@arndb.de>
Tue, 13 Jan 2015 13:31:46 +0000 (14:31 +0100)
committerVinod Koul <vinod.koul@intel.com>
Tue, 13 Jan 2015 15:50:10 +0000 (21:20 +0530)
In a recent cleanup, the mmp_tdma_terminate_all function was
introduced but does not set a proper return value. Almost
no slave driver uses that return value, but if one does, the
result will be undefined, which the compiler warns about:

dma/mmp_tdma.c: In function 'mmp_tdma_terminate_all':
dma/mmp_tdma.c:474:1: warning: no return statement in function returning non-void [-Wreturn-type]

This changes the driver to return zero, like most other
drivers do.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: f43a6fd400ba6 ("dmaengine: mmp-tdma: Split device_control")
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
drivers/dma/mmp_tdma.c

index 91fb241102f7f4e862ec521c64359e657373d406..70c2fa9963cd4d942afa0b8a99f42c4d2ede784b 100644 (file)
@@ -471,6 +471,8 @@ static int mmp_tdma_terminate_all(struct dma_chan *chan)
        mmp_tdma_disable_chan(chan);
        /* disable interrupt */
        mmp_tdma_enable_irq(tdmac, false);
+
+       return 0;
 }
 
 static int mmp_tdma_config(struct dma_chan *chan,