diff options
author | Dave Jiang <dave.jiang@intel.com> | 2015-08-20 11:44:14 -0400 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2015-08-21 04:34:24 -0400 |
commit | 64f1d0ffbaaccf2ddaf02d3ebf67bf9044cb4db4 (patch) | |
tree | 67cd5d585e1830e822d7ce93b5a197275e29d7ac /drivers/dma/ioat | |
parent | aaecdebc5855b77e2120d11c750630a3e60ffb10 (diff) |
dmaengine: ioatdma: fix coccinelle warning
Simplifying the end return. This existed in the original code but was
flagged when refactoring of the code made it appear it's new.
coccinelle warnings: (new ones prefixed by >>)
>> drivers/dma/ioat/init.c:1018:1-3: WARNING: end returns can be simpified
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma/ioat')
-rw-r--r-- | drivers/dma/ioat/init.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/dma/ioat/init.c b/drivers/dma/ioat/init.c index 592222105997..60a7c3211e0d 100644 --- a/drivers/dma/ioat/init.c +++ b/drivers/dma/ioat/init.c | |||
@@ -1008,16 +1008,15 @@ out: | |||
1008 | 1008 | ||
1009 | static int ioat3_dma_self_test(struct ioatdma_device *ioat_dma) | 1009 | static int ioat3_dma_self_test(struct ioatdma_device *ioat_dma) |
1010 | { | 1010 | { |
1011 | int rc = ioat_dma_self_test(ioat_dma); | 1011 | int rc; |
1012 | 1012 | ||
1013 | rc = ioat_dma_self_test(ioat_dma); | ||
1013 | if (rc) | 1014 | if (rc) |
1014 | return rc; | 1015 | return rc; |
1015 | 1016 | ||
1016 | rc = ioat_xor_val_self_test(ioat_dma); | 1017 | rc = ioat_xor_val_self_test(ioat_dma); |
1017 | if (rc) | ||
1018 | return rc; | ||
1019 | 1018 | ||
1020 | return 0; | 1019 | return rc; |
1021 | } | 1020 | } |
1022 | 1021 | ||
1023 | static void ioat_intr_quirk(struct ioatdma_device *ioat_dma) | 1022 | static void ioat_intr_quirk(struct ioatdma_device *ioat_dma) |