wlcore: clean-up clearing of WL1271_FLAG_IRQ_RUNNING
authorTony Lindgren <tony@atomide.com>
Wed, 9 Oct 2019 16:50:06 +0000 (09:50 -0700)
committerKalle Valo <kvalo@codeaurora.org>
Fri, 11 Oct 2019 08:55:40 +0000 (11:55 +0300)
commit4633d30b61ac1415817b954c3d4d241ca7439a7c
tree9320354c9b87ba85910fdd4710bf5e2dedae960d
parent59f4567d228f7f5686e0f91d30e6aeaca999e2fd
wlcore: clean-up clearing of WL1271_FLAG_IRQ_RUNNING

We set WL1271_FLAG_IRQ_RUNNING in the beginning of wlcore_irq(), but clear
it before interrupt handling is done in wlcore_irq_locked().

Let's move the clearing to the end of wlcore_irq() where it gets set,
and remove the old comments about hardirq. That's no longer the case as
we're using request_threaded_irq().

Note that the WL1271_FLAG_IRQ_RUNNING should never race between the
interrupt handler and wlcore_runtime_resume() as because of autosuspend
timeout we cannot enter idle between wlcore_irq_locked() and the end of
wlcore_irq().

Cc: Anders Roxell <anders.roxell@linaro.org>
Cc: Eyal Reizer <eyalr@ti.com>
Cc: Guy Mishol <guym@ti.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ti/wlcore/main.c