diff options
author | Barry Song <21cnbao@gmail.com> | 2018-08-17 09:00:26 -0400 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2018-09-03 06:53:00 -0400 |
commit | 1297b647c58c5d1d576654acf93c3dabb615256a (patch) | |
tree | b95e2b52f521d1bfcabec44b1119b3d2a3896b5d | |
parent | 5b394b2ddf0347bef56e50c69a58773c94343ff3 (diff) |
dmaengine: fsldma: move spin_lock_bh to spin_lock in tasklet
as you are already in a tasklet, it is unnecessary to call spin_lock_bh.
Signed-off-by: Barry Song <21cnbao@gmail.com>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
-rw-r--r-- | drivers/dma/fsldma.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/dma/fsldma.c b/drivers/dma/fsldma.c index 1117b5123a6f..9d360a3fbae3 100644 --- a/drivers/dma/fsldma.c +++ b/drivers/dma/fsldma.c | |||
@@ -987,7 +987,7 @@ static void dma_do_tasklet(unsigned long data) | |||
987 | 987 | ||
988 | chan_dbg(chan, "tasklet entry\n"); | 988 | chan_dbg(chan, "tasklet entry\n"); |
989 | 989 | ||
990 | spin_lock_bh(&chan->desc_lock); | 990 | spin_lock(&chan->desc_lock); |
991 | 991 | ||
992 | /* the hardware is now idle and ready for more */ | 992 | /* the hardware is now idle and ready for more */ |
993 | chan->idle = true; | 993 | chan->idle = true; |
@@ -995,7 +995,7 @@ static void dma_do_tasklet(unsigned long data) | |||
995 | /* Run all cleanup for descriptors which have been completed */ | 995 | /* Run all cleanup for descriptors which have been completed */ |
996 | fsldma_cleanup_descriptors(chan); | 996 | fsldma_cleanup_descriptors(chan); |
997 | 997 | ||
998 | spin_unlock_bh(&chan->desc_lock); | 998 | spin_unlock(&chan->desc_lock); |
999 | 999 | ||
1000 | chan_dbg(chan, "tasklet exit\n"); | 1000 | chan_dbg(chan, "tasklet exit\n"); |
1001 | } | 1001 | } |