diff options
| author | Sachin Kamat <sachin.kamat@linaro.org> | 2013-09-02 10:24:20 +0200 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2013-09-09 19:26:04 +0200 |
| commit | a577659f42f745b71cbc56c80fb1e801ec7e6348 (patch) | |
| tree | 9d32260e8644af9fdb0c77b7542510e09b7e2466 /drivers/dma/mv_xor.c | |
| parent | ioatdma: silence GCC warnings (diff) | |
| download | linux-a577659f42f745b71cbc56c80fb1e801ec7e6348.tar.xz linux-a577659f42f745b71cbc56c80fb1e801ec7e6348.zip | |
dma: mv_xor: Fix incorrect error path
Return directly if memory allocation fails. There is no need
of dma_free_coherent().
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Cc: Saeed Bishara <saeed@marvell.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/dma/mv_xor.c')
| -rw-r--r-- | drivers/dma/mv_xor.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c index d332b9e3f9ce..0ec086d2b6a0 100644 --- a/drivers/dma/mv_xor.c +++ b/drivers/dma/mv_xor.c @@ -1036,10 +1036,8 @@ mv_xor_channel_add(struct mv_xor_device *xordev, struct dma_device *dma_dev; mv_chan = devm_kzalloc(&pdev->dev, sizeof(*mv_chan), GFP_KERNEL); - if (!mv_chan) { - ret = -ENOMEM; - goto err_free_dma; - } + if (!mv_chan) + return ERR_PTR(-ENOMEM); mv_chan->idx = idx; mv_chan->irq = irq; |