diff options
author | Alexander Aring <alex.aring@gmail.com> | 2014-03-15 04:29:04 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-03-17 16:10:25 -0400 |
commit | 7e8146189a6c2ba9445b8d848847f2520c6cb028 (patch) | |
tree | 5d8dbd805e99228fbb2030978e3e2dd84e48d677 /drivers/net/ieee802154 | |
parent | 7332fcb82a15730f4b0bfa65db074c505c0ffc1a (diff) |
at86rf230: move locking state in xmit
There is no need to lock the clearing of IRQ_TRX_END in status.
Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ieee802154')
-rw-r--r-- | drivers/net/ieee802154/at86rf230.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ieee802154/at86rf230.c b/drivers/net/ieee802154/at86rf230.c index ad296bc86e69..030bf3995e7e 100644 --- a/drivers/net/ieee802154/at86rf230.c +++ b/drivers/net/ieee802154/at86rf230.c | |||
@@ -914,8 +914,8 @@ static void at86rf230_irqwork(struct work_struct *work) | |||
914 | status &= ~IRQ_TRX_UR; /* FIXME: possibly handle ???*/ | 914 | status &= ~IRQ_TRX_UR; /* FIXME: possibly handle ???*/ |
915 | 915 | ||
916 | if (status & IRQ_TRX_END) { | 916 | if (status & IRQ_TRX_END) { |
917 | spin_lock_irqsave(&lp->lock, flags); | ||
918 | status &= ~IRQ_TRX_END; | 917 | status &= ~IRQ_TRX_END; |
918 | spin_lock_irqsave(&lp->lock, flags); | ||
919 | if (lp->is_tx) { | 919 | if (lp->is_tx) { |
920 | lp->is_tx = 0; | 920 | lp->is_tx = 0; |
921 | spin_unlock_irqrestore(&lp->lock, flags); | 921 | spin_unlock_irqrestore(&lp->lock, flags); |