diff options
| author |   Alexander Aring <alex.aring@gmail.com> | 2014-03-15 09:29:04 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2014-03-17 16:10:25 -0400 |
| commit | 7e8146189a6c2ba9445b8d848847f2520c6cb028 (patch) | |
| tree | 5d8dbd805e99228fbb2030978e3e2dd84e48d677 /drivers/net/ieee802154 | |
| parent | at86rf230: fix unexpected state change (diff) | |
| download | linux-dev-7e8146189a6c2ba9445b8d848847f2520c6cb028.tar.xz linux-dev-7e8146189a6c2ba9445b8d848847f2520c6cb028.zip | |
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) status &= ~IRQ_TRX_UR; /* FIXME: possibly handle ???*/ if (status & IRQ_TRX_END) { - spin_lock_irqsave(&lp->lock, flags); status &= ~IRQ_TRX_END; + spin_lock_irqsave(&lp->lock, flags); if (lp->is_tx) { lp->is_tx = 0; spin_unlock_irqrestore(&lp->lock, flags); |