brcmfmac: improve Raspberry Pi 3 stability
authorÁlvaro Fernández Rojas <noltari@gmail.com>
Tue, 7 Feb 2017 13:45:19 +0000 (14:45 +0100)
committerÁlvaro Fernández Rojas <noltari@gmail.com>
Tue, 7 Feb 2017 14:17:44 +0000 (15:17 +0100)
- Really disable power management (wrong config flags).
- Disable internal roaming engine.

Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
package/kernel/mac80211/patches/862-brcmfmac-Disable-power-management.patch
package/kernel/mac80211/patches/864-brcmfmac-do-not-use-internal-roaming-engine-by-default.patch [new file with mode: 0644]

index 41b0f64f090bc8e5cfbb99cf9a1f66ed2a953247..f0226a2843737c9c73692f9657a62f5ab81843ca 100644 (file)
@@ -18,8 +18,8 @@ Signed-off-by: Phil Elwell <phil@raspberrypi.org>
         * preference in cfg struct to apply this to
         * FW later while initializing the dongle
         */
-+#if defined(CONFIG_BCM2708) || defined(CONFIG_BCM2709)
-+      pr_info("power management disabled\n");
++#if defined(CONFIG_ARCH_BCM2708) || defined(CONFIG_ARCH_BCM2709)
++      brcmf_dbg(INFO, "power management disabled\n");
 +      enabled = false;
 +#endif
        cfg->pwr_save = enabled;
diff --git a/package/kernel/mac80211/patches/864-brcmfmac-do-not-use-internal-roaming-engine-by-default.patch b/package/kernel/mac80211/patches/864-brcmfmac-do-not-use-internal-roaming-engine-by-default.patch
new file mode 100644 (file)
index 0000000..5ae4c11
--- /dev/null
@@ -0,0 +1,23 @@
+brcmfmac: do not use internal roaming engine by default
+
+Some evidence of curing disconnects with this disabled, so make it a default.
+Can be overridden with module parameter roamoff=0
+See: http://projectable.me/optimize-my-pi-wi-fi/
+
+Signed-off-by: Phil Elwell <phil@raspberrypi.org>
+---
+
+--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
++++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
+@@ -69,7 +69,11 @@ static int brcmf_fcmode;
+ module_param_named(fcmode, brcmf_fcmode, int, 0);
+ MODULE_PARM_DESC(fcmode, "Mode of firmware signalled flow control");
++#if defined(CONFIG_ARCH_BCM2708) || defined(CONFIG_ARCH_BCM2709)
++static int brcmf_roamoff = 1;
++#else
+ static int brcmf_roamoff;
++#endif
+ module_param_named(roamoff, brcmf_roamoff, int, S_IRUSR);
+ MODULE_PARM_DESC(roamoff, "Do not use internal roaming engine");