diff options
author | Viresh Kumar <viresh.kumar@st.com> | 2011-08-05 06:02:45 -0400 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2011-08-25 10:05:07 -0400 |
commit | 57001a606f845ce2eda21a0f23e6aab20ee0cb04 (patch) | |
tree | e66120a192e0a98b0c2d0ca0255331edeb6db0d2 /drivers/dma | |
parent | 0a2356572b1910cc977f4ccf3c9ee1ecab08327a (diff) |
dmaengine/amba-pl08x: Call pl08x_free_txd() instead of calling kfree() directly
pl08x_prep_channel_resources() is calling kfree() directly for txd(). To
maintain consistency in code call pl08x_free_txd() instead.
Signed-off-by: Viresh Kumar <viresh.kumar@st.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma')
-rw-r--r-- | drivers/dma/amba-pl08x.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c index a59c3c47286c..849eab85514b 100644 --- a/drivers/dma/amba-pl08x.c +++ b/drivers/dma/amba-pl08x.c | |||
@@ -1174,7 +1174,9 @@ static int pl08x_prep_channel_resources(struct pl08x_dma_chan *plchan, | |||
1174 | 1174 | ||
1175 | num_llis = pl08x_fill_llis_for_desc(pl08x, txd); | 1175 | num_llis = pl08x_fill_llis_for_desc(pl08x, txd); |
1176 | if (!num_llis) { | 1176 | if (!num_llis) { |
1177 | kfree(txd); | 1177 | spin_lock_irqsave(&plchan->lock, flags); |
1178 | pl08x_free_txd(pl08x, txd); | ||
1179 | spin_unlock_irqrestore(&plchan->lock, flags); | ||
1178 | return -EINVAL; | 1180 | return -EINVAL; |
1179 | } | 1181 | } |
1180 | 1182 | ||