On Thu, Nov 02, 2017 at 04:40:57PM +0100, Arnd Bergmann wrote: > On Thu, Nov 2, 2017 at 4:23 PM, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote: > > Brian has already fixed this, please check that: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=ath-next&id=20665a9076d48e9abd9a2db13d307f58f7ef6647 > > > > But apparently I forgot to merge ath-next to wireless-drivers-next, will > > do that soon. > > Yes, Brian's version is better. I considered the same, but wasn't sure > it was safe. Yes, it's safe. The code is still basically dead, since the mac80211 ops get dropped with !CONFIG_PM (so no one calls the *_hif_{suspend,resume}() functions), but at least we have fewer #ifdef's. Brian