From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932565AbcHVQZl (ORCPT ); Mon, 22 Aug 2016 12:25:41 -0400 Received: from mail-wm0-f41.google.com ([74.125.82.41]:38485 "EHLO mail-wm0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932220AbcHVQZj (ORCPT ); Mon, 22 Aug 2016 12:25:39 -0400 From: Ioan-Adrian Ratiu To: Kalle Valo , Benjamin Berg Cc: linux-wireless@vger.kernel.org, ath9k-devel@qca.qualcomm.com, ath9k-devel@venema.h4ckr.net, linux-kernel@vger.kernel.org, Felix Fietkau Subject: Re: ath9k: Fix beacon configuration assertion failure In-Reply-To: <87r39h3p0f.fsf@purkki.adurom.net> References: <87fuq1yt1x.fsf@kamboji.qca.qualcomm.com> <1471855240.16921.6.camel@open-mesh.com> <87r39h3p0f.fsf@purkki.adurom.net> User-Agent: Notmuch/0.22.1 (https://notmuchmail.org) Emacs/25.1.50.1 (x86_64-unknown-linux-gnu) Date: Mon, 22 Aug 2016 19:18:42 +0300 Message-ID: <87mvk4kc9p.fsf@ni.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id u7MGPji4016387 On Mon, 22 Aug 2016, Kalle Valo wrote: > Benjamin Berg 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