Commit 1d05a0bd authored by Yunbo Yu's avatar Yunbo Yu Committed by Vinod Koul

dmaengine: plx_dma: Move spin_lock_bh() to spin_lock()

It is unnecessary to call spin_lock_bh() if you are already in a tasklet.
Signed-off-by: default avatarYunbo Yu <yuyunbo519@gmail.com>
Reviewed-by: default avatarLogan Gunthorpe <logang@deltatee.com>
Link: https://lore.kernel.org/r/20220418142021.1241558-1-yuyunbo519@gmail.comSigned-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent 766b540d
...@@ -137,7 +137,7 @@ static void plx_dma_process_desc(struct plx_dma_dev *plxdev) ...@@ -137,7 +137,7 @@ static void plx_dma_process_desc(struct plx_dma_dev *plxdev)
struct plx_dma_desc *desc; struct plx_dma_desc *desc;
u32 flags; u32 flags;
spin_lock_bh(&plxdev->ring_lock); spin_lock(&plxdev->ring_lock);
while (plxdev->tail != plxdev->head) { while (plxdev->tail != plxdev->head) {
desc = plx_dma_get_desc(plxdev, plxdev->tail); desc = plx_dma_get_desc(plxdev, plxdev->tail);
...@@ -165,7 +165,7 @@ static void plx_dma_process_desc(struct plx_dma_dev *plxdev) ...@@ -165,7 +165,7 @@ static void plx_dma_process_desc(struct plx_dma_dev *plxdev)
plxdev->tail++; plxdev->tail++;
} }
spin_unlock_bh(&plxdev->ring_lock); spin_unlock(&plxdev->ring_lock);
} }
static void plx_dma_abort_desc(struct plx_dma_dev *plxdev) static void plx_dma_abort_desc(struct plx_dma_dev *plxdev)
......
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