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 1cUD8x-0001XS-8o for ath10k@lists.infradead.org; Thu, 19 Jan 2017 13:52:50 +0000 MIME-Version: 1.0 Date: Thu, 19 Jan 2017 19:22:25 +0530 From: Tamizh chelvam Subject: Re: [PATCH 2/4] cfg80211: Add new NL80211_CMD_SET_BTCOEX_PRIORITY to support BTCOEX In-Reply-To: <1483958190.17582.15.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> <6f9711ed6f6fe170b915b67a054b985b@codeaurora.org> <1483958190.17582.15.camel@sipsolutions.net> Message-ID: <53738f0ff79f0cf32026b9c700059fb1@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 SGkgSm9oYW5uZXMsCgpTb3JyeSBmb3IgdGhlIGxhdGUgcmVzcG9uc2U6KAoKT24gMjAxNy0wMS0w OSAxNjowNiwgSm9oYW5uZXMgQmVyZyB3cm90ZToKPj4gSXMgaXQgZmluZSB0byBoYXZlIHNvbWV0 aGluZyBsaWtlIHRoaXMKPj4gCj4+IDEpIFdlIGNhbiBoYXZlIHRoaXMgYnRjb2V4X3ByaW9yaXR5 IHZhbHVlIGFzIGEgb3B0aW9uYWwgdmFsdWUgaW4KPj4gYnRjb2V4wqBlbmFibGUgY29tbWFuZCBs aWtlIGJlbG93Cj4+IAo+PiBpdyBwaHlYIGJ0Y29leF9zdGF0ZSA8ZW5hYmxlfCBkaXNhYmxlPiBb cHJpcm9yaXR5KHZlbmRvciBzcGNpZmljCj4+IHZhbHVlKV0KPj4gCj4+IDIpIE9yIHdlIGNhbiBo YXZlIHNlcGVyYXRlIGNvbW1hbmQgZm9yIGJ0Y29leF9wcmlvcml0eSBhcyBiZWxvdwo+PiAKPj4g aXcgcGh5WCBzZXQgYnRjb2V4X3ByaW9yaXR5IDxwcmlvcml0eSAodmVuZG9yIHNwY2lmaWMgdmFs dWUpPgo+PiAKPj4gSG9wZWZ1bGx5IHRoaXMgd2lsbCBnZXQgcmlkIG9mZiBhbGwgdGhlIG5sODAy MTEgYml0cy4KPiAKPiBUaGF0IG1ha2VzIG5vIHNlbnNlLgo+IAo+IElmIHRoZSBiaXRzIGFyZSB2 ZW5kb3Igc3BlY2lmaWMsIHRoZW4gdGhlcmUncyBubyB2YWx1ZSBpbiBoYXZpbmcgdGhpcwo+IGFz IGFuIG5sODAyMTEgY29tbWFuZCAocmF0aGVyIHRoYW4gYSB2ZW5kb3IgY29tbWFuZCkgdG8gc3Rh cnQgd2l0aC4KPiAKPiBZb3UgbmVlZCB0byB1bmRlcnN0YW5kIHRoYXQgSSdtIGRpZmZlcmVudGlh dGluZyBiZXR3ZWVuICpjYXBhYmlsaXR5Kgo+IGJpdHMgYW5kIGFjdHVhbCAqcHJpb3JpdHkgc2V0 dGluZyogYml0cyAtIHBsZWFzZSByZS1yZWFkIHRoZSB0aHJlYWQKPiB3aXRoIHRoYXQgaW4gbWlu ZC4KPiAKVGhvc2UgYXJlIGZvciBwcmlvcml0eSBhbmQgbm90IGZvciBjYXBhYmlsaXR5IHB1cnBv c2UuIEhhcmR3YXJlIGlzIApjYXBhYmxlIG9mCnRyYW5zbWl0dGluZyBhbGwgdGhlIGZyYW1lcyB3 aGVuIHRoZXJlIGlzIGEgYnQgdHJhZmZpYy4gVGhpcyBpcyBqdXN0IApzZXR0aW5nIHByaW9yaXR5 IGZvciB0aGUgd2xhbgpmcmFtZSwgd2hlbiB0aGVyZSBpcyBhIGNvbnRlbnRpb24gd2l0aCBCVCB0 cmFmZmljIHdoaWNoIG9uZSBoYXMgdG8gZ28gCm91dCBmaXJzdC4KT3RoZXIgdGhhbiB0aGF0IGhh cmR3YXJlIGhhcyB0aGUgY2FwYWJpbGl0eSBvZiBzZW5kaW5nIGFsbCB0aGUgZnJhbWVzLgoKVGhp cyBwcmlvcml0eSBmaWVsZCBpcyBvcHRpb25hbCwgaWYgdXNlcnMgZG9lcyBub3Qgd2FudCB0byBz ZXQgYW55IApwcmlvcml0eSB0aGVuIGRyaXZlciB3aWxsIGZvcmNlIHRoZQpkZWZhdWx0IHByaW9y aXRpZXMgd2hlbiBidF9jb2V4IGlzIGVuYWJsZWQKCml3IHBoeVggYnRjb2V4X3N0YXRlIDxlbmFi bGV8IGRpc2FibGU+IFtwcmlyb3JpdHldCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fXwphdGgxMGsgbWFpbGluZyBsaXN0CmF0aDEwa0BsaXN0cy5pbmZyYWRl YWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vYXRoMTBr Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:44216 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752611AbdASOBv (ORCPT ); Thu, 19 Jan 2017 09:01:51 -0500 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Date: Thu, 19 Jan 2017 19:22:25 +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: <1483958190.17582.15.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> <6f9711ed6f6fe170b915b67a054b985b@codeaurora.org> <1483958190.17582.15.camel@sipsolutions.net> Message-ID: <53738f0ff79f0cf32026b9c700059fb1@codeaurora.org> (sfid-20170119_150223_256239_935F4885) Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi Johannes, Sorry for the late response:( On 2017-01-09 16:06, Johannes Berg wrote: >> 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. > > That makes no sense. > > If the bits are vendor specific, then there's no value in having this > as an nl80211 command (rather than a vendor command) to start with. > > You need to understand that I'm differentiating between *capability* > bits and actual *priority setting* bits - please re-read the thread > with that in mind. > Those are for priority and not for capability purpose. Hardware is capable of transmitting all the frames when there is a bt traffic. This is just setting priority for the wlan frame, when there is a contention with BT traffic which one has to go out first. Other than that hardware has the capability of sending all the frames. This priority field is optional, if users does not want to set any priority then driver will force the default priorities when bt_coex is enabled iw phyX btcoex_state [prirority]