wlcore: Use spin_trylock in wlcore_irq() to see if we need to queue tx
authorTony Lindgren <tony@atomide.com>
Thu, 2 Jul 2020 16:29:50 +0000 (09:29 -0700)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 15 Jul 2020 09:12:30 +0000 (12:12 +0300)
commit35fba0f0fd762a8b87d403ae3c723e0061c4aa25
tree856af36414701eeaea243be770051cd2be71c4d5
parentf0325e38ab39c2e270770b72c79795772ac3b49e
wlcore: Use spin_trylock in wlcore_irq() to see if we need to queue tx

We currently have a collection of flags and locking between the
threaded irq and tx work:

- wl->flags bitops
- wl->mutex
- wl->wl_lock spinlock

The bitops flags do not need a spinlock around them, and we only need
the spinlock to see if we need to queue tx work or not. And wlcore_irq()
holds the mutex.

To simplify the locking, we can use spin_trylock and always queue tx
work unless we know there's nothing to do.

Let's also update the comment a bit while at it.

Signed-off-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200702162951.45392-4-tony@atomide.com
drivers/net/wireless/ti/wlcore/main.c