diff options
| author |   Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2020-02-26 12:18:40 +0200 |
|---|---|---|
| committer |   Vinod Koul <vkoul@kernel.org> | 2020-03-02 12:48:13 +0530 |
| commit | 5f77dd850c0a32d4d5047d139077718ee7f1a8fe (patch) | |
| tree | 0d6d031d60ad3e810e65f2a9bce633578c6d7eba /include/linux/dmaengine.h | |
| parent | dmaengine: Use negative condition for better readability (diff) | |
| download | wireguard-linux-5f77dd850c0a32d4d5047d139077718ee7f1a8fe.tar.xz wireguard-linux-5f77dd850c0a32d4d5047d139077718ee7f1a8fe.zip | |
dmaengine: Drop redundant 'else' keyword
It's obvious that 'else' keyword is redundant in the code like
if (foo)
return bar;
else if (baz)
...
Drop it for good.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20200226101842.29426-3-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.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h index ae56a91c2a05..1bb5477ef7ec 100644 --- a/include/linux/dmaengine.h +++ b/include/linux/dmaengine.h @@ -1230,9 +1230,9 @@ static inline int dma_maxpq(struct dma_device *dma, enum dma_ctrl_flags flags) { if (dma_dev_has_pq_continue(dma) || !dmaf_continue(flags)) return dma_dev_to_maxpq(dma); - else if (dmaf_p_disabled_continue(flags)) + if (dmaf_p_disabled_continue(flags)) return dma_dev_to_maxpq(dma) - 1; - else if (dmaf_continue(flags)) + if (dmaf_continue(flags)) return dma_dev_to_maxpq(dma) - 3; BUG(); } @@ -1243,7 +1243,7 @@ static inline size_t dmaengine_get_icg(bool inc, bool sgl, size_t icg, if (inc) { if (dir_icg) return dir_icg; - else if (sgl) + if (sgl) return icg; } |