diff options
author | tpugliese <thomas.pugliese@gmail.com> | 2013-05-20 16:09:05 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-05-21 14:48:03 -0400 |
commit | 3138887bd8d18173f3c2baf1e43621c49090cd27 (patch) | |
tree | d4290319d1cabec3813854fb7ba67f772161a1a6 /drivers/uwb | |
parent | c0c6e079497f850759e64f28f39a5a2d350dc230 (diff) |
usb: uwb: use correct locking calls for rc data
The rc object's members can be accessed from both a timer and URB
callback. Use spin_*_irq instead of spin_*_bh.
Signed-off-by: Thomas Pugliese <thomas.pugliese@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/uwb')
-rw-r--r-- | drivers/uwb/drp.c | 4 | ||||
-rw-r--r-- | drivers/uwb/rsv.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/drivers/uwb/drp.c b/drivers/uwb/drp.c index 3fbcf789dfaa..16ada8341c46 100644 --- a/drivers/uwb/drp.c +++ b/drivers/uwb/drp.c | |||
@@ -67,14 +67,14 @@ static void uwb_rc_set_drp_cmd_done(struct uwb_rc *rc, void *arg, | |||
67 | } else | 67 | } else |
68 | dev_err(&rc->uwb_dev.dev, "SET-DRP-IE: timeout\n"); | 68 | dev_err(&rc->uwb_dev.dev, "SET-DRP-IE: timeout\n"); |
69 | 69 | ||
70 | spin_lock_bh(&rc->rsvs_lock); | 70 | spin_lock_irq(&rc->rsvs_lock); |
71 | if (rc->set_drp_ie_pending > 1) { | 71 | if (rc->set_drp_ie_pending > 1) { |
72 | rc->set_drp_ie_pending = 0; | 72 | rc->set_drp_ie_pending = 0; |
73 | uwb_rsv_queue_update(rc); | 73 | uwb_rsv_queue_update(rc); |
74 | } else { | 74 | } else { |
75 | rc->set_drp_ie_pending = 0; | 75 | rc->set_drp_ie_pending = 0; |
76 | } | 76 | } |
77 | spin_unlock_bh(&rc->rsvs_lock); | 77 | spin_unlock_irq(&rc->rsvs_lock); |
78 | } | 78 | } |
79 | 79 | ||
80 | /** | 80 | /** |
diff --git a/drivers/uwb/rsv.c b/drivers/uwb/rsv.c index f4ae05f78c42..738e8a8cb811 100644 --- a/drivers/uwb/rsv.c +++ b/drivers/uwb/rsv.c | |||
@@ -872,7 +872,7 @@ void uwb_rsv_queue_update(struct uwb_rc *rc) | |||
872 | */ | 872 | */ |
873 | void uwb_rsv_sched_update(struct uwb_rc *rc) | 873 | void uwb_rsv_sched_update(struct uwb_rc *rc) |
874 | { | 874 | { |
875 | spin_lock_bh(&rc->rsvs_lock); | 875 | spin_lock_irq(&rc->rsvs_lock); |
876 | if (!delayed_work_pending(&rc->rsv_update_work)) { | 876 | if (!delayed_work_pending(&rc->rsv_update_work)) { |
877 | if (rc->set_drp_ie_pending > 0) { | 877 | if (rc->set_drp_ie_pending > 0) { |
878 | rc->set_drp_ie_pending++; | 878 | rc->set_drp_ie_pending++; |
@@ -881,7 +881,7 @@ void uwb_rsv_sched_update(struct uwb_rc *rc) | |||
881 | uwb_rsv_queue_update(rc); | 881 | uwb_rsv_queue_update(rc); |
882 | } | 882 | } |
883 | unlock: | 883 | unlock: |
884 | spin_unlock_bh(&rc->rsvs_lock); | 884 | spin_unlock_irq(&rc->rsvs_lock); |
885 | } | 885 | } |
886 | 886 | ||
887 | /* | 887 | /* |