diff options
author | Antti Palosaari <crope@iki.fi> | 2014-08-24 22:26:23 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-09-21 15:42:35 -0400 |
commit | b63ab6b064f73721568a46080d41422ea223ee6e (patch) | |
tree | 47152c1827e1165f31b1e07a989d0b6e9c0840a3 /drivers/media/usb/msi2500 | |
parent | 5fa51cc36ac14e0a2a6fd4f9037bf4372066e894 (diff) |
[media] msi2500: remove unneeded spinlock irq flags initialization
There is no need to init flags before calling spin_lock_irqsave().
spin_lock_irqsave is macro which stores value to 'flags'.
Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/usb/msi2500')
-rw-r--r-- | drivers/media/usb/msi2500/msi2500.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/media/usb/msi2500/msi2500.c b/drivers/media/usb/msi2500/msi2500.c index 647846b5904a..efc761c78f72 100644 --- a/drivers/media/usb/msi2500/msi2500.c +++ b/drivers/media/usb/msi2500/msi2500.c | |||
@@ -160,7 +160,7 @@ struct msi2500_state { | |||
160 | static struct msi2500_frame_buf *msi2500_get_next_fill_buf( | 160 | static struct msi2500_frame_buf *msi2500_get_next_fill_buf( |
161 | struct msi2500_state *s) | 161 | struct msi2500_state *s) |
162 | { | 162 | { |
163 | unsigned long flags = 0; | 163 | unsigned long flags; |
164 | struct msi2500_frame_buf *buf = NULL; | 164 | struct msi2500_frame_buf *buf = NULL; |
165 | 165 | ||
166 | spin_lock_irqsave(&s->queued_bufs_lock, flags); | 166 | spin_lock_irqsave(&s->queued_bufs_lock, flags); |
@@ -558,7 +558,7 @@ static int msi2500_isoc_init(struct msi2500_state *s) | |||
558 | /* Must be called with vb_queue_lock hold */ | 558 | /* Must be called with vb_queue_lock hold */ |
559 | static void msi2500_cleanup_queued_bufs(struct msi2500_state *s) | 559 | static void msi2500_cleanup_queued_bufs(struct msi2500_state *s) |
560 | { | 560 | { |
561 | unsigned long flags = 0; | 561 | unsigned long flags; |
562 | 562 | ||
563 | dev_dbg(s->dev, "\n"); | 563 | dev_dbg(s->dev, "\n"); |
564 | 564 | ||
@@ -634,7 +634,7 @@ static void msi2500_buf_queue(struct vb2_buffer *vb) | |||
634 | struct msi2500_state *s = vb2_get_drv_priv(vb->vb2_queue); | 634 | struct msi2500_state *s = vb2_get_drv_priv(vb->vb2_queue); |
635 | struct msi2500_frame_buf *buf = | 635 | struct msi2500_frame_buf *buf = |
636 | container_of(vb, struct msi2500_frame_buf, vb); | 636 | container_of(vb, struct msi2500_frame_buf, vb); |
637 | unsigned long flags = 0; | 637 | unsigned long flags; |
638 | 638 | ||
639 | /* Check the device has not disconnected between prep and queuing */ | 639 | /* Check the device has not disconnected between prep and queuing */ |
640 | if (unlikely(!s->udev)) { | 640 | if (unlikely(!s->udev)) { |