diff options
| author | Dafna Hirschfeld <[email protected]> | 2020-06-26 10:51:39 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2020-07-19 10:47:41 +0200 |
| commit | 46c308a964fe6515845eacb8765a2c2e2b8c1570 (patch) | |
| tree | e9845913fa064c3cd5e22296e059e4012b80c173 | |
| parent | 24385f9c6ee4e1e118046333b49637e5d337d943 (diff) | |
media: staging: rkisp1: stats: replace spin_lock_irqsave with spin_lock_irq
The function 'rkisp1_stats_vb2_stop_streaming' runs in user context
therefore it is enough to use spin_lock_irq
Signed-off-by: Dafna Hirschfeld <[email protected]>
Acked-by: Helen Koike <[email protected]>
Signed-off-by: Hans Verkuil <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
| -rw-r--r-- | drivers/staging/media/rkisp1/rkisp1-stats.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/staging/media/rkisp1/rkisp1-stats.c b/drivers/staging/media/rkisp1/rkisp1-stats.c index b19a6d9cdd4d..d4f0df4342e0 100644 --- a/drivers/staging/media/rkisp1/rkisp1-stats.c +++ b/drivers/staging/media/rkisp1/rkisp1-stats.c @@ -152,13 +152,12 @@ static void rkisp1_stats_vb2_stop_streaming(struct vb2_queue *vq) { struct rkisp1_stats *stats = vq->drv_priv; struct rkisp1_buffer *buf; - unsigned long flags; unsigned int i; /* Make sure no new work queued in isr before draining wq */ - spin_lock_irqsave(&stats->irq_lock, flags); + spin_lock_irq(&stats->irq_lock); stats->is_streaming = false; - spin_unlock_irqrestore(&stats->irq_lock, flags); + spin_unlock_irq(&stats->irq_lock); drain_workqueue(stats->readout_wq); |