summaryrefslogtreecommitdiff
path: root/include/linux/dmaengine.h
diff options
context:
space:
mode:
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>2020-02-26 12:18:41 +0200
committerVinod Koul <vkoul@kernel.org>2020-03-02 12:48:14 +0530
commit1873300afa6147a1882aeba1e8bc9a13c5487571 (patch)
tree151e7652af2fb34873232f1e18ad967e54fe7b20 /include/linux/dmaengine.h
parent5f77dd850c0a32d4d5047d139077718ee7f1a8fe (diff)
downloadlwn-1873300afa6147a1882aeba1e8bc9a13c5487571.tar.gz
lwn-1873300afa6147a1882aeba1e8bc9a13c5487571.zip
dmaengine: consistently return string literal from switch-case
There is no need to have 'break;' statement in the default case followed by return certain string literal when all other cases have returned the string literals. So, refactor it accordingly. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Reviewed-by: Peter Ujfalusi <peter.ujfalusi@ti.com> Link: https://lore.kernel.org/r/20200226101842.29426-4-andriy.shevchenko@linux.intel.com Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'include/linux/dmaengine.h')
-rw-r--r--include/linux/dmaengine.h4
1 files changed, 1 insertions, 3 deletions
diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h
index 1bb5477ef7ec..d3672f065a64 100644
--- a/include/linux/dmaengine.h
+++ b/include/linux/dmaengine.h
@@ -1560,9 +1560,7 @@ dmaengine_get_direction_text(enum dma_transfer_direction dir)
case DMA_DEV_TO_DEV:
return "DEV_TO_DEV";
default:
- break;
+ return "invalid";
}
-
- return "invalid";
}
#endif /* DMAENGINE_H */