diff options
| author |   Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-09-05 14:53:46 +0800 |
|---|---|---|
| committer |   David Brown <davidb@codeaurora.org> | 2012-09-13 11:12:50 -0700 |
| commit | 6eebf2de9e315dbad4f5f5d8f9faa249a38f5f09 (patch) | |
| tree | fc2b36f693cbcbaf25d6206afdec221fca4dd474 /arch/arm/mach-msm/dma.c | |
| parent | ARM: msm: Fix sparse warnings due to incorrect type (diff) | |
| download | linux-dev-6eebf2de9e315dbad4f5f5d8f9faa249a38f5f09.tar.xz linux-dev-6eebf2de9e315dbad4f5f5d8f9faa249a38f5f09.zip | |
ARM: msm: dma: use list_move_tail instead of list_del/list_add_tail
Using list_move_tail() instead of list_del() + list_add_tail().
spatch with a semantic match is used to found this problem.
(http://coccinelle.lip6.fr/)
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: David Brown <davidb@codeaurora.org>
Diffstat (limited to 'arch/arm/mach-msm/dma.c')
| -rw-r--r-- | arch/arm/mach-msm/dma.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/arm/mach-msm/dma.c b/arch/arm/mach-msm/dma.c index 02cae5e2951c..354b91d4c3ac 100644 --- a/arch/arm/mach-msm/dma.c +++ b/arch/arm/mach-msm/dma.c @@ -223,8 +223,7 @@ static irqreturn_t msm_datamover_irq_handler(int irq, void *dev_id) PRINT_FLOW("msm_datamover_irq_handler id %d, status %x\n", id, ch_status); if ((ch_status & DMOV_STATUS_CMD_PTR_RDY) && !list_empty(&ready_commands[id])) { cmd = list_entry(ready_commands[id].next, typeof(*cmd), list); - list_del(&cmd->list); - list_add_tail(&cmd->list, &active_commands[id]); + list_move_tail(&cmd->list, &active_commands[id]); if (cmd->execute_func) cmd->execute_func(cmd); PRINT_FLOW("msm_datamover_irq_handler id %d, start command\n", id); |