media: staging: rkisp1: stats: replace spin_lock_irqsave with spin_lock_irq
authorDafna Hirschfeld <dafna.hirschfeld@collabora.com>
Fri, 26 Jun 2020 08:51:39 +0000 (10:51 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Sun, 19 Jul 2020 08:47:41 +0000 (10:47 +0200)
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 <dafna.hirschfeld@collabora.com>
Acked-by: Helen Koike <helen.koike@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/rkisp1/rkisp1-stats.c

index b19a6d9cdd4daa4b26f36555098b41b1f4091dce..d4f0df4342e0960bb82e8e1875d91c62ee60019c 100644 (file)
@@ -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);