From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wg0-f47.google.com ([74.125.82.47]:43458 "EHLO mail-wg0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751347AbaGXJkk convert rfc822-to-8bit (ORCPT ); Thu, 24 Jul 2014 05:40:40 -0400 Received: by mail-wg0-f47.google.com with SMTP id b13so2404688wgh.6 for ; Thu, 24 Jul 2014 02:40:39 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <201407241136.08090.sw@simonwunderlich.de> References: <87mwc0ro8i.fsf@kamboji.qca.qualcomm.com> <1406136773-19957-1-git-send-email-sw@simonwunderlich.de> <871ttbrtp8.fsf@kamboji.qca.qualcomm.com> <201407241136.08090.sw@simonwunderlich.de> Date: Thu, 24 Jul 2014 11:40:39 +0200 Message-ID: (sfid-20140724_114043_998530_1D1B15F7) Subject: Re: [PATCHv3-ath-resend 2/2] ath10k: add spectral scan feature From: Michal Kazior To: Simon Wunderlich Cc: Kalle Valo , "ath10k@lists.infradead.org" , Mathias Kretschmer , "Giori, Kathy" , linux-wireless , Sven Eckelmann Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 24 July 2014 11:36, Simon Wunderlich wrote: >> > @@ -800,8 +800,6 @@ int ath10k_core_start(struct ath10k *ar) >> > >> > else >> > >> > ar->free_vdev_map = (1 << TARGET_NUM_VDEVS) - 1; >> > >> > - INIT_LIST_HEAD(&ar->arvifs); >> > - >> > >> > if (!test_bit(ATH10K_FLAG_FIRST_BOOT_DONE, &ar->dev_flags)) { >> > >> > ath10k_info("%s (0x%08x, 0x%08x) fw %s api %d htt %d.%d\n", >> > >> > ar->hw_params.name, >> > >> > @@ -1084,6 +1082,12 @@ struct ath10k *ath10k_core_create(void *hif_priv, >> > struct device *dev, >> > >> > INIT_WORK(&ar->register_work, ath10k_core_register_work); >> > INIT_WORK(&ar->restart_work, ath10k_core_restart); >> > >> > + INIT_LIST_HEAD(&ar->arvifs); >> >> I'm a bit suspicious about this, why is it needed? > > ath10k_spectral_init() clears the spectral_enabled bit for each vif, and > therefore needs the list to be initialized ... it is also called on firmware > crash by ath10k_halt, where checking the list is more useful. You don't really need to clear arvif->spectral_enabled on crash. arvif's are re-inserted by mac80211 via ath10k_add_interface which uses memset() to clear everything before adding to ar->arvifs. Michał From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-we0-x22d.google.com ([2a00:1450:400c:c03::22d]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1XAFWN-0005Rq-19 for ath10k@lists.infradead.org; Thu, 24 Jul 2014 09:41:07 +0000 Received: by mail-we0-f173.google.com with SMTP id q58so2496425wes.32 for ; Thu, 24 Jul 2014 02:40:39 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <201407241136.08090.sw@simonwunderlich.de> References: <87mwc0ro8i.fsf@kamboji.qca.qualcomm.com> <1406136773-19957-1-git-send-email-sw@simonwunderlich.de> <871ttbrtp8.fsf@kamboji.qca.qualcomm.com> <201407241136.08090.sw@simonwunderlich.de> Date: Thu, 24 Jul 2014 11:40:39 +0200 Message-ID: Subject: Re: [PATCHv3-ath-resend 2/2] ath10k: add spectral scan feature From: Michal Kazior List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Simon Wunderlich Cc: Mathias Kretschmer , linux-wireless , "ath10k@lists.infradead.org" , Kalle Valo , "Giori, Kathy" , Sven Eckelmann T24gMjQgSnVseSAyMDE0IDExOjM2LCBTaW1vbiBXdW5kZXJsaWNoIDxzd0BzaW1vbnd1bmRlcmxp Y2guZGU+IHdyb3RlOgo+PiA+IEBAIC04MDAsOCArODAwLDYgQEAgaW50IGF0aDEwa19jb3JlX3N0 YXJ0KHN0cnVjdCBhdGgxMGsgKmFyKQo+PiA+Cj4+ID4gICAgIGVsc2UKPj4gPgo+PiA+ICAgICAg ICAgICAgIGFyLT5mcmVlX3ZkZXZfbWFwID0gKDEgPDwgVEFSR0VUX05VTV9WREVWUykgLSAxOwo+ PiA+Cj4+ID4gLSAgIElOSVRfTElTVF9IRUFEKCZhci0+YXJ2aWZzKTsKPj4gPiAtCj4+ID4KPj4g PiAgICAgaWYgKCF0ZXN0X2JpdChBVEgxMEtfRkxBR19GSVJTVF9CT09UX0RPTkUsICZhci0+ZGV2 X2ZsYWdzKSkgewo+PiA+Cj4+ID4gICAgICAgICAgICAgYXRoMTBrX2luZm8oIiVzICgweCUwOHgs IDB4JTA4eCkgZncgJXMgYXBpICVkIGh0dCAlZC4lZFxuIiwKPj4gPgo+PiA+ICAgICAgICAgICAg ICAgICAgICAgICAgIGFyLT5od19wYXJhbXMubmFtZSwKPj4gPgo+PiA+IEBAIC0xMDg0LDYgKzEw ODIsMTIgQEAgc3RydWN0IGF0aDEwayAqYXRoMTBrX2NvcmVfY3JlYXRlKHZvaWQgKmhpZl9wcml2 LAo+PiA+IHN0cnVjdCBkZXZpY2UgKmRldiwKPj4gPgo+PiA+ICAgICBJTklUX1dPUksoJmFyLT5y ZWdpc3Rlcl93b3JrLCBhdGgxMGtfY29yZV9yZWdpc3Rlcl93b3JrKTsKPj4gPiAgICAgSU5JVF9X T1JLKCZhci0+cmVzdGFydF93b3JrLCBhdGgxMGtfY29yZV9yZXN0YXJ0KTsKPj4gPgo+PiA+ICsg ICBJTklUX0xJU1RfSEVBRCgmYXItPmFydmlmcyk7Cj4+Cj4+IEknbSBhIGJpdCBzdXNwaWNpb3Vz IGFib3V0IHRoaXMsIHdoeSBpcyBpdCBuZWVkZWQ/Cj4KPiBhdGgxMGtfc3BlY3RyYWxfaW5pdCgp IGNsZWFycyB0aGUgc3BlY3RyYWxfZW5hYmxlZCBiaXQgZm9yIGVhY2ggdmlmLCBhbmQKPiB0aGVy ZWZvcmUgbmVlZHMgdGhlIGxpc3QgdG8gYmUgaW5pdGlhbGl6ZWQgLi4uIGl0IGlzIGFsc28gY2Fs bGVkIG9uIGZpcm13YXJlCj4gY3Jhc2ggYnkgYXRoMTBrX2hhbHQsIHdoZXJlIGNoZWNraW5nIHRo ZSBsaXN0IGlzIG1vcmUgdXNlZnVsLgoKWW91IGRvbid0IHJlYWxseSBuZWVkIHRvIGNsZWFyIGFy dmlmLT5zcGVjdHJhbF9lbmFibGVkIG9uIGNyYXNoLgphcnZpZidzIGFyZSByZS1pbnNlcnRlZCBi eSBtYWM4MDIxMSB2aWEgYXRoMTBrX2FkZF9pbnRlcmZhY2Ugd2hpY2gKdXNlcyBtZW1zZXQoKSB0 byBjbGVhciBldmVyeXRoaW5nIGJlZm9yZSBhZGRpbmcgdG8gYXItPmFydmlmcy4KCgpNaWNoYcWC CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwphdGgxMGsg bWFpbGluZyBsaXN0CmF0aDEwa0BsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZy YWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vYXRoMTBrCg==