Re: [PATCH] ath9k: turn on btcoex_enable as default
> On 10 Feb 2018, at 10:05 PM, Felix Fietkau wrote: > > On 2018-02-10 14:56, Kai Heng Feng wrote: >> >>> On 9 Feb 2018, at 3:16 PM, Kalle Valo wrote: >>> Sure, but we have to make sure that we don't create regressions on >>> existing systems. For example, did you test this with any system which >>> don't support btcoex? (just asking, haven't tested this myself) >> >> No not really, but I will definitely test it. >> The only module I have that uses ath9k is Dell’s DW1707. >> How do I check if it support btcoex or not? > I just reviewed the code again, and I am sure that we cannot merge this > patch. Enabling the btcoex parameter makes the driver enable a whole > bunch of code starting timers, listening to some GPIOs, etc. > > On non-btcoex systems, some of those GPIOs might be floating or even > connected to different things, which could cause a lot of undefined > behavior. > > This is simply too big a risk, so there absolutely needs to be a > whitelist for systems that need this, otherwise it has to remain > disabled by default. So what information can we use to whitelist btcoex chips? Can we get btcoex support status at ath9k probing? Kai-Heng > > - Felix
Re: [PATCH] ath9k: turn on btcoex_enable as default
On 2018-02-10 14:56, Kai Heng Feng wrote: > >> On 9 Feb 2018, at 3:16 PM, Kalle Valo wrote: >> Sure, but we have to make sure that we don't create regressions on >> existing systems. For example, did you test this with any system which >> don't support btcoex? (just asking, haven't tested this myself) > > No not really, but I will definitely test it. > The only module I have that uses ath9k is Dell’s DW1707. > How do I check if it support btcoex or not? I just reviewed the code again, and I am sure that we cannot merge this patch. Enabling the btcoex parameter makes the driver enable a whole bunch of code starting timers, listening to some GPIOs, etc. On non-btcoex systems, some of those GPIOs might be floating or even connected to different things, which could cause a lot of undefined behavior. This is simply too big a risk, so there absolutely needs to be a whitelist for systems that need this, otherwise it has to remain disabled by default. - Felix
Re: [PATCH] ath9k: turn on btcoex_enable as default
> On 9 Feb 2018, at 3:16 PM, Kalle Valo wrote: > Sure, but we have to make sure that we don't create regressions on > existing systems. For example, did you test this with any system which > don't support btcoex? (just asking, haven't tested this myself) No not really, but I will definitely test it. The only module I have that uses ath9k is Dell’s DW1707. How do I check if it support btcoex or not? (I resend the mail because my last mail get changed to HTML by my mail client) Kai-Heng > > -- > Kalle Valo
Re: [PATCH] ath9k: turn on btcoex_enable as default
Kai Heng Feng writes: > Hi Felix, > >> On Feb 8, 2018, at 7:02 PM, Felix Fietkau wrote: >> >> On 2018-02-08 06:28, Kai-Heng Feng wrote: >>> Without btcoex_enable, WiFi activies make both WiFi and Bluetooth >>> unstable if there's a bluetooth connection. >>> >>> Enable this option when bt_ant_diversity is disabled. >>> >>> BugLink: https://bugs.launchpad.net/bugs/1746164 >>> Signed-off-by: Kai-Heng Feng >> I think this might cause regressions on devices that don't have >> bluetooth. This probably either needs more EEPROM checks, or something >> to selectively enable it only on affected platforms. > > I think it’s better not to use dmi_match. This issue should affect > more ath9k. And bluetooth peripherals are more than ever now, so it > would be great to use BT out of the box. Sure, but we have to make sure that we don't create regressions on existing systems. For example, did you test this with any system which don't support btcoex? (just asking, haven't tested this myself) -- Kalle Valo
Re: [PATCH] ath9k: turn on btcoex_enable as default
Hi Felix, On Feb 8, 2018, at 7:02 PM, Felix Fietkau wrote: On 2018-02-08 06:28, Kai-Heng Feng wrote: Without btcoex_enable, WiFi activies make both WiFi and Bluetooth unstable if there's a bluetooth connection. Enable this option when bt_ant_diversity is disabled. BugLink: https://bugs.launchpad.net/bugs/1746164 Signed-off-by: Kai-Heng Feng I think this might cause regressions on devices that don't have bluetooth. This probably either needs more EEPROM checks, or something to selectively enable it only on affected platforms. I think it’s better not to use dmi_match. This issue should affect more ath9k. And bluetooth peripherals are more than ever now, so it would be great to use BT out of the box. Can you take a look at the bug link, maybe there are other things caused the erratic behavior that I didn’t notice? Kai-Heng - Felix
Re: [PATCH] ath9k: turn on btcoex_enable as default
On 2018-02-08 06:28, Kai-Heng Feng wrote: > Without btcoex_enable, WiFi activies make both WiFi and Bluetooth > unstable if there's a bluetooth connection. > > Enable this option when bt_ant_diversity is disabled. > > BugLink: https://bugs.launchpad.net/bugs/1746164 > Signed-off-by: Kai-Heng Feng I think this might cause regressions on devices that don't have bluetooth. This probably either needs more EEPROM checks, or something to selectively enable it only on affected platforms. - Felix