On Mon, 22 Aug 2016, Kalle Valo <kvalo@xxxxxxxxxxxxxx> wrote: > Benjamin Berg <benjamin.berg@xxxxxxxxxxxxx> writes: > >> On Fr, 2016-08-19 at 13:03 +0300, Kalle Valo wrote: >>> Actually, I see two patches which might be related but not identical: >>> >>> ath9k: fix client mode beacon configuration >>> https://patchwork.kernel.org/patch/9247699/ >>> >>> ath9k: Fix beacon configuration assertion failure >>> https://patchwork.kernel.org/patch/9281191/ >>> >>> Felix (CCed) & Benjamin: please take a look and advice which one I >>> should take. >> >> Yes, both patches are designed to fix the same issue in my patch. >> >> Felix solution looks entirely correct to me, the second solution seems >> slightly wrong because it prevents the call to ath9k_beacon_config from >> happening instead of ensuring the correct parameter value. >> ath9k_beacon_config needs to be called even if iter_data.beacons is >> false as it disables the interrupts. > > Ok, I'll the patch from Felix then. Thanks. Tested Felix patch. Confirm it works great. Thanks! > > -- > Kalle Valo
