From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cQWv9-0001d1-7N for ath10k@lists.infradead.org; Mon, 09 Jan 2017 10:11:20 +0000 MIME-Version: 1.0 Date: Mon, 09 Jan 2017 15:40:57 +0530 From: Tamizh chelvam Subject: Re: [PATCH 2/4] cfg80211: Add new NL80211_CMD_SET_BTCOEX_PRIORITY to support BTCOEX In-Reply-To: <1483623500.4394.18.camel@sipsolutions.net> References: <1478610932-21954-1-git-send-email-c_traja@qti.qualcomm.com> <1478610932-21954-3-git-send-email-c_traja@qti.qualcomm.com> <1480949353.31788.27.camel@sipsolutions.net> <5e5e8971c96293a81e7cb37bcdfbd593@codeaurora.org> <1481645351.20412.34.camel@sipsolutions.net> <134cc8e58ecb804b6dda0137c4c37be8@codeaurora.org> <1481881024.27953.14.camel@sipsolutions.net> <1483354130.4596.5.camel@sipsolutions.net> <1483623500.4394.18.camel@sipsolutions.net> Message-ID: <6f9711ed6f6fe170b915b67a054b985b@codeaurora.org> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: base64 Content-Type: text/plain; charset="utf-8"; Format="flowed" Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Johannes Berg Cc: c_traja@qti.qualcomm.com, linux-wireless@vger.kernel.org, ath10k@lists.infradead.org SGkgSm9oYW5uZXMsCgpUaGFuayB5b3UgZm9yIHRoZSByZXBseS4KCk9uIDIwMTctMDEtMDUgMTk6 MDgsIEpvaGFubmVzIEJlcmcgd3JvdGU6Cj4+ID4gSU9XIC0gd2h5IGhhdmUgYWxsIHRoZXNlIGJp dHMgcmF0aGVyIHRoYW4ganVzdCBvbmU/Cj4+IAo+PiBIYXJkd2FyZSBzdXBwb3J0cyBkYXRhIGFj cm9zcyBhbGwgdGhlIGFjY2VzcyBjYXRlZ29yaWVzLCB0aGlzIGlzCj4+IGp1c3TCoCBtZWFudCBm b3IgcHJpb3JpdGlzaW5nIHRoZSB0cmFmZmljLiBmLmUsIElmIHRoZSBmdy90YXJnZXQgaGFzCj4+ IGJvdGggd2xhbiBhbmQgYnQgdHJhZmZpYyBxdWV1ZWQgYW5kIGlmIFZPIGlzwqAgc2V0IGFzIHBy aW9yaXR5LCB0aGVuCj4+IHdsYW4gVk8gcGFja2V0cyB3aWxsIGJlIHB1c2hlZCBvdXQgb2YgdGhl IHJhZGlvIGZpcnN0IGFuZCB0aGVuIHRoZSBidAo+PiB0cmFmZmljLgo+IAo+IEV4YWN0bHkuIFNv IGFzIGZhciBhcyAqY2FwYWJpbGl0aWVzKiBhcmUgY29uY2VybmVkLCB3aHkgZG8gd2UgbmVlZCBz bwo+IG1hbnkgYml0cz8KPiAKSXMgaXQgZmluZSB0byBoYXZlIHNvbWV0aGluZyBsaWtlIHRoaXMK CjEpIFdlIGNhbiBoYXZlIHRoaXMgYnRjb2V4X3ByaW9yaXR5IHZhbHVlIGFzIGEgb3B0aW9uYWwg dmFsdWUgaW4gYnRjb2V4IAplbmFibGUgY29tbWFuZCBsaWtlIGJlbG93CgppdyBwaHlYIGJ0Y29l eF9zdGF0ZSA8ZW5hYmxlfCBkaXNhYmxlPiBbcHJpcm9yaXR5KHZlbmRvciBzcGNpZmljIHZhbHVl KV0KCjIpIE9yIHdlIGNhbiBoYXZlIHNlcGVyYXRlIGNvbW1hbmQgZm9yIGJ0Y29leF9wcmlvcml0 eSBhcyBiZWxvdwoKaXcgcGh5WCBzZXQgYnRjb2V4X3ByaW9yaXR5IDxwcmlvcml0eSAodmVuZG9y IHNwY2lmaWMgdmFsdWUpPgoKSG9wZWZ1bGx5IHRoaXMgd2lsbCBnZXQgcmlkIG9mZiBhbGwgdGhl IG5sODAyMTEgYml0cy4KCktpbmRseSBzdWdnZXN0IHlvdXIgdGhvdWdodHMuCgpUaGFua3MsClRh bWl6aC4KCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmF0 aDEwayBtYWlsaW5nIGxpc3QKYXRoMTBrQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3Rz LmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9hdGgxMGsK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:47058 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935288AbdAIKK7 (ORCPT ); Mon, 9 Jan 2017 05:10:59 -0500 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Date: Mon, 09 Jan 2017 15:40:57 +0530 From: Tamizh chelvam To: Johannes Berg Cc: c_traja@qti.qualcomm.com, linux-wireless@vger.kernel.org, ath10k@lists.infradead.org Subject: Re: [PATCH 2/4] cfg80211: Add new NL80211_CMD_SET_BTCOEX_PRIORITY to support BTCOEX In-Reply-To: <1483623500.4394.18.camel@sipsolutions.net> References: <1478610932-21954-1-git-send-email-c_traja@qti.qualcomm.com> <1478610932-21954-3-git-send-email-c_traja@qti.qualcomm.com> <1480949353.31788.27.camel@sipsolutions.net> <5e5e8971c96293a81e7cb37bcdfbd593@codeaurora.org> <1481645351.20412.34.camel@sipsolutions.net> <134cc8e58ecb804b6dda0137c4c37be8@codeaurora.org> <1481881024.27953.14.camel@sipsolutions.net> <1483354130.4596.5.camel@sipsolutions.net> <1483623500.4394.18.camel@sipsolutions.net> Message-ID: <6f9711ed6f6fe170b915b67a054b985b@codeaurora.org> (sfid-20170109_111109_678226_B4156143) Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi Johannes, Thank you for the reply. On 2017-01-05 19:08, Johannes Berg wrote: >> > IOW - why have all these bits rather than just one? >> >> Hardware supports data across all the access categories, this is >> just  meant for prioritising the traffic. f.e, If the fw/target has >> both wlan and bt traffic queued and if VO is  set as priority, then >> wlan VO packets will be pushed out of the radio first and then the bt >> traffic. > > Exactly. So as far as *capabilities* are concerned, why do we need so > many bits? > Is it fine to have something like this 1) We can have this btcoex_priority value as a optional value in btcoex enable command like below iw phyX btcoex_state [prirority(vendor spcific value)] 2) Or we can have seperate command for btcoex_priority as below iw phyX set btcoex_priority Hopefully this will get rid off all the nl80211 bits. Kindly suggest your thoughts. Thanks, Tamizh.