Commit 51c99e04 authored by Sandeep Paulraj's avatar Sandeep Paulraj Committed by Kevin Hilman

DaVinci: EDMA: Fix bug in edma_free_cont_slots API

In the edma_free_cont_slots API, the variable slot was being modified
and then used in the for loop.
This results in incorrect behaviour when the API is used.
Signed-off-by: default avatarSandeep Paulraj <s-paulraj@ti.com>
Signed-off-by: default avatarKevin Hilman <khilman@deeprootsystems.com>
parent 7761ef67
...@@ -813,7 +813,7 @@ EXPORT_SYMBOL(edma_alloc_cont_slots); ...@@ -813,7 +813,7 @@ EXPORT_SYMBOL(edma_alloc_cont_slots);
*/ */
int edma_free_cont_slots(unsigned slot, int count) int edma_free_cont_slots(unsigned slot, int count)
{ {
unsigned ctlr; unsigned ctlr, slot_to_free;
int i; int i;
ctlr = EDMA_CTLR(slot); ctlr = EDMA_CTLR(slot);
...@@ -826,11 +826,11 @@ int edma_free_cont_slots(unsigned slot, int count) ...@@ -826,11 +826,11 @@ int edma_free_cont_slots(unsigned slot, int count)
for (i = slot; i < slot + count; ++i) { for (i = slot; i < slot + count; ++i) {
ctlr = EDMA_CTLR(i); ctlr = EDMA_CTLR(i);
slot = EDMA_CHAN_SLOT(i); slot_to_free = EDMA_CHAN_SLOT(i);
memcpy_toio(edmacc_regs_base[ctlr] + PARM_OFFSET(slot), memcpy_toio(edmacc_regs_base[ctlr] + PARM_OFFSET(slot_to_free),
&dummy_paramset, PARM_SIZE); &dummy_paramset, PARM_SIZE);
clear_bit(slot, edma_info[ctlr]->edma_inuse); clear_bit(slot_to_free, edma_info[ctlr]->edma_inuse);
} }
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment