diff options
| author | Dave Jiang <dave.jiang@intel.com> | 2015-08-20 17:44:14 +0200 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2015-08-21 10:34:24 +0200 |
| commit | 64f1d0ffbaaccf2ddaf02d3ebf67bf9044cb4db4 (patch) | |
| tree | 67cd5d585e1830e822d7ce93b5a197275e29d7ac /drivers/dma/ioat | |
| parent | dmaengine: ioatdma: fix zero day warning on incompatible pointer type (diff) | |
| download | linux-64f1d0ffbaaccf2ddaf02d3ebf67bf9044cb4db4.tar.xz linux-64f1d0ffbaaccf2ddaf02d3ebf67bf9044cb4db4.zip | |
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: static int ioat3_dma_self_test(struct ioatdma_device *ioat_dma) { - int rc = ioat_dma_self_test(ioat_dma); + int rc; + rc = ioat_dma_self_test(ioat_dma); if (rc) return rc; rc = ioat_xor_val_self_test(ioat_dma); - if (rc) - return rc; - return 0; + return rc; } static void ioat_intr_quirk(struct ioatdma_device *ioat_dma) |