From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wi0-f177.google.com ([209.85.212.177]:39714 "EHLO mail-wi0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751349AbbA3NGJ convert rfc822-to-8bit (ORCPT ); Fri, 30 Jan 2015 08:06:09 -0500 Received: by mail-wi0-f177.google.com with SMTP id r20so2549301wiv.4 for ; Fri, 30 Jan 2015 05:06:08 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <1422622694.1919.17.camel@sipsolutions.net> References: <1422607287-12289-1-git-send-email-michal.kazior@tieto.com> <1422621086-28970-1-git-send-email-michal.kazior@tieto.com> <1422621086-28970-3-git-send-email-michal.kazior@tieto.com> <1422622694.1919.17.camel@sipsolutions.net> Date: Fri, 30 Jan 2015 14:06:08 +0100 Message-ID: (sfid-20150130_140621_385051_2BA65BD6) Subject: Re: [PATCH v2 2/7] ath10k: implement chanctx API From: Michal Kazior To: Johannes Berg Cc: "ath10k@lists.infradead.org" , linux-wireless Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 30 January 2015 at 13:58, Johannes Berg wrote: > >> +++ b/drivers/net/wireless/ath/ath10k/core.h > >> @@ -277,6 +282,7 @@ struct ath10k_vif { >> dma_addr_t beacon_paddr; >> >> struct ath10k *ar; >> + struct ath10k_chanctx *arctx; > > you also have vif->chanctx_conf pointer, which you could use > > ultimately it's a software architecture decision - personally I wouldn't > want to duplicate all that data and the lists etc. in the driver when > mac80211 has them, but that's just my opinion. :) I'm a bit reluctant to freely use mac80211 owned structures out of mac80211_ops callback context, i.e. from a tasklet fearing inconsistencies. Or am I worrying unnecessarily? If so I guess I'll redo this and use mac80211's structures directly :-) Michał From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wg0-x22d.google.com ([2a00:1450:400c:c00::22d]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1YHBHK-0005ra-8f for ath10k@lists.infradead.org; Fri, 30 Jan 2015 13:06:30 +0000 Received: by mail-wg0-f45.google.com with SMTP id x12so26762607wgg.4 for ; Fri, 30 Jan 2015 05:06:08 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <1422622694.1919.17.camel@sipsolutions.net> References: <1422607287-12289-1-git-send-email-michal.kazior@tieto.com> <1422621086-28970-1-git-send-email-michal.kazior@tieto.com> <1422621086-28970-3-git-send-email-michal.kazior@tieto.com> <1422622694.1919.17.camel@sipsolutions.net> Date: Fri, 30 Jan 2015 14:06:08 +0100 Message-ID: Subject: Re: [PATCH v2 2/7] ath10k: implement chanctx API 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: Johannes Berg Cc: linux-wireless , "ath10k@lists.infradead.org" T24gMzAgSmFudWFyeSAyMDE1IGF0IDEzOjU4LCBKb2hhbm5lcyBCZXJnIDxqb2hhbm5lc0BzaXBz b2x1dGlvbnMubmV0PiB3cm90ZToKPgo+PiArKysgYi9kcml2ZXJzL25ldC93aXJlbGVzcy9hdGgv YXRoMTBrL2NvcmUuaAo+Cj4+IEBAIC0yNzcsNiArMjgyLDcgQEAgc3RydWN0IGF0aDEwa192aWYg ewo+PiAgICAgICBkbWFfYWRkcl90IGJlYWNvbl9wYWRkcjsKPj4KPj4gICAgICAgc3RydWN0IGF0 aDEwayAqYXI7Cj4+ICsgICAgIHN0cnVjdCBhdGgxMGtfY2hhbmN0eCAqYXJjdHg7Cj4KPiB5b3Ug YWxzbyBoYXZlIHZpZi0+Y2hhbmN0eF9jb25mIHBvaW50ZXIsIHdoaWNoIHlvdSBjb3VsZCB1c2UK Pgo+IHVsdGltYXRlbHkgaXQncyBhIHNvZnR3YXJlIGFyY2hpdGVjdHVyZSBkZWNpc2lvbiAtIHBl cnNvbmFsbHkgSSB3b3VsZG4ndAo+IHdhbnQgdG8gZHVwbGljYXRlIGFsbCB0aGF0IGRhdGEgYW5k IHRoZSBsaXN0cyBldGMuIGluIHRoZSBkcml2ZXIgd2hlbgo+IG1hYzgwMjExIGhhcyB0aGVtLCBi dXQgdGhhdCdzIGp1c3QgbXkgb3Bpbmlvbi4gOikKCkknbSBhIGJpdCByZWx1Y3RhbnQgdG8gZnJl ZWx5IHVzZSBtYWM4MDIxMSBvd25lZCBzdHJ1Y3R1cmVzIG91dCBvZgptYWM4MDIxMV9vcHMgY2Fs bGJhY2sgY29udGV4dCwgaS5lLiBmcm9tIGEgdGFza2xldCBmZWFyaW5nCmluY29uc2lzdGVuY2ll cy4gT3IgYW0gSSB3b3JyeWluZyB1bm5lY2Vzc2FyaWx5PyBJZiBzbyBJIGd1ZXNzIEknbGwKcmVk byB0aGlzIGFuZCB1c2UgbWFjODAyMTEncyBzdHJ1Y3R1cmVzIGRpcmVjdGx5IDotKQoKCk1pY2hh xYIKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmF0aDEw ayBtYWlsaW5nIGxpc3QKYXRoMTBrQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmlu ZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9hdGgxMGsK