On Mon, 2017-09-18 at 13:16 +0300, Sergey Matyukevich wrote: > > > > > > - if (timer_pending(&mac->scan_timeout)) > > > > > - del_timer_sync(&mac->scan_timeout); > I don't quite understand the question. Do you mean it makes sense to > call cfg80211_scan_done first and then cancel timer ? No, I'm just not sure why you call timer_pending() first. johannes
