diff options
author | Arnd Bergmann <arnd@arndb.de> | 2015-01-13 22:17:03 +0100 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2015-01-18 20:01:36 +0530 |
commit | 4d76bbed2d8d9f7bf8bca31e64ef977e015a86fa (patch) | |
tree | d329aa739e52cdbcfbb350557632635f3d2647f6 /include/linux | |
parent | 848e10bb521eca333f4f912cf70efbd5b0f73c32 (diff) | |
download | op-kernel-dev-4d76bbed2d8d9f7bf8bca31e64ef977e015a86fa.zip op-kernel-dev-4d76bbed2d8d9f7bf8bca31e64ef977e015a86fa.tar.gz |
dmaengine: coh901318: fix function return types build warnings
A recent patch that removed coh901318_control() replaced it
with a number of pointers to existing functions, but those
unfortunately have the wrong return type and need to be
changed to return an 'int' with an error value rather than
a 'void' to avoid these build warnings:
drivers/dma/coh901318.c:2697:32: warning: assignment from incompatible pointer type
base->dma_slave.device_config = coh901318_dma_set_runtimeconfig;
^
drivers/dma/coh901318.c:2698:31: warning: assignment from incompatible pointer type
base->dma_slave.device_pause = coh901318_pause;
^
drivers/dma/coh901318.c:2699:32: warning: assignment from incompatible pointer type
base->dma_slave.device_resume = coh901318_resume
The coh901318_base_init function has the correct return type
already, but needs to be marked 'static' to avoid a sparse
warning about a missing declaration.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 6782af118b6c ("dmaengine: coh901318: Split device_control")
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'include/linux')
0 files changed, 0 insertions, 0 deletions