Skip to content

Commit

Permalink
spi/spi-pl022: Call pl022_dma_remove(pl022) only if enable_dma is true
Browse files Browse the repository at this point in the history
pl022_dma_remove() should be called only if enable_dma is true. There is no
point calling it when pl022_dma_probe() is not called, which again depends on
enable_dma.

Signed-off-by: Viresh Kumar <viresh.kumar@st.com>
Tested-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
  • Loading branch information
Viresh Kumar authored and Grant Likely committed Sep 21, 2011
1 parent 0379b2a commit 3e3ea71
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions drivers/spi/spi-pl022.c
Original file line number Diff line number Diff line change
Expand Up @@ -2237,7 +2237,9 @@ pl022_probe(struct amba_device *adev, const struct amba_id *id)
err_start_queue:
err_init_queue:
destroy_queue(pl022);
pl022_dma_remove(pl022);
if (platform_info->enable_dma)
pl022_dma_remove(pl022);

free_irq(adev->irq[0], pl022);
pm_runtime_disable(&adev->dev);
err_no_irq:
Expand Down Expand Up @@ -2265,7 +2267,9 @@ pl022_remove(struct amba_device *adev)
if (destroy_queue(pl022) != 0)
dev_err(&adev->dev, "queue remove failed\n");
load_ssp_default_config(pl022);
pl022_dma_remove(pl022);
if (pl022->master_info->enable_dma)
pl022_dma_remove(pl022);

free_irq(adev->irq[0], pl022);
clk_disable(pl022->clk);
clk_put(pl022->clk);
Expand Down

0 comments on commit 3e3ea71

Please sign in to comment.