Tony Lindgren <t...@atomide.com> wrote:

> 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.rox...@linaro.org>
> Cc: Eyal Reizer <ey...@ti.com>
> Cc: Guy Mishol <g...@ti.com>
> Cc: John Stultz <john.stu...@linaro.org>
> Cc: Ulf Hansson <ulf.hans...@linaro.org>
> Signed-off-by: Tony Lindgren <t...@atomide.com>

Patch applied to wireless-drivers-next.git, thanks.

4633d30b61ac wlcore: clean-up clearing of WL1271_FLAG_IRQ_RUNNING

-- 
https://patchwork.kernel.org/patch/11181621/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

Reply via email to