mirror of
https://github.com/hak5/wifipineapple-openwrt.git
synced 2025-10-29 16:57:19 +00:00
109 lines
3.5 KiB
Diff
109 lines
3.5 KiB
Diff
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
|
||
Date: Tue, 7 Jun 2016 21:10:18 +0200
|
||
Subject: [PATCH] brcmfmac: slightly simplify building interface combinations
|
||
MIME-Version: 1.0
|
||
Content-Type: text/plain; charset=UTF-8
|
||
Content-Transfer-Encoding: 8bit
|
||
|
||
This change reorders some operations in brcmf_setup_ifmodes in hope to
|
||
make it simpler:
|
||
1) It allocates arrays right before filling them. This way it's easier
|
||
to follow requested array length as it's immediately followed by
|
||
code filling it. It's easier to check e.g. why we need 4 entries for
|
||
P2P. Other than that it deduplicates some checks (e.g. for P2P).
|
||
2) It reorders code to first prepare limits and then define a new combo.
|
||
Previously this was mixed (e.g. we were setting num of channels
|
||
before preparing limits).
|
||
3) It modifies mbss code to use i variable just like other combos do.
|
||
|
||
Signed-off-by: RafaÅ MiÅecki <zajec5@gmail.com>
|
||
Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
|
||
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
|
||
---
|
||
|
||
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
|
||
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
|
||
@@ -6208,29 +6208,15 @@ static int brcmf_setup_ifmodes(struct wi
|
||
if (!combo)
|
||
goto err;
|
||
|
||
- c0_limits = kcalloc(p2p ? 3 : 2, sizeof(*c0_limits), GFP_KERNEL);
|
||
- if (!c0_limits)
|
||
- goto err;
|
||
-
|
||
- if (p2p) {
|
||
- p2p_limits = kcalloc(4, sizeof(*p2p_limits), GFP_KERNEL);
|
||
- if (!p2p_limits)
|
||
- goto err;
|
||
- }
|
||
-
|
||
- if (mbss) {
|
||
- mbss_limits = kcalloc(1, sizeof(*mbss_limits), GFP_KERNEL);
|
||
- if (!mbss_limits)
|
||
- goto err;
|
||
- }
|
||
-
|
||
wiphy->interface_modes = BIT(NL80211_IFTYPE_STATION) |
|
||
BIT(NL80211_IFTYPE_ADHOC) |
|
||
BIT(NL80211_IFTYPE_AP);
|
||
|
||
c = 0;
|
||
i = 0;
|
||
- combo[c].num_different_channels = 1;
|
||
+ c0_limits = kcalloc(p2p ? 3 : 2, sizeof(*c0_limits), GFP_KERNEL);
|
||
+ if (!c0_limits)
|
||
+ goto err;
|
||
c0_limits[i].max = 1;
|
||
c0_limits[i++].types = BIT(NL80211_IFTYPE_STATION);
|
||
if (p2p) {
|
||
@@ -6248,6 +6234,7 @@ static int brcmf_setup_ifmodes(struct wi
|
||
c0_limits[i].max = 1;
|
||
c0_limits[i++].types = BIT(NL80211_IFTYPE_AP);
|
||
}
|
||
+ combo[c].num_different_channels = 1;
|
||
combo[c].max_interfaces = i;
|
||
combo[c].n_limits = i;
|
||
combo[c].limits = c0_limits;
|
||
@@ -6255,7 +6242,9 @@ static int brcmf_setup_ifmodes(struct wi
|
||
if (p2p) {
|
||
c++;
|
||
i = 0;
|
||
- combo[c].num_different_channels = 1;
|
||
+ p2p_limits = kcalloc(4, sizeof(*p2p_limits), GFP_KERNEL);
|
||
+ if (!p2p_limits)
|
||
+ goto err;
|
||
p2p_limits[i].max = 1;
|
||
p2p_limits[i++].types = BIT(NL80211_IFTYPE_STATION);
|
||
p2p_limits[i].max = 1;
|
||
@@ -6264,6 +6253,7 @@ static int brcmf_setup_ifmodes(struct wi
|
||
p2p_limits[i++].types = BIT(NL80211_IFTYPE_P2P_CLIENT);
|
||
p2p_limits[i].max = 1;
|
||
p2p_limits[i++].types = BIT(NL80211_IFTYPE_P2P_DEVICE);
|
||
+ combo[c].num_different_channels = 1;
|
||
combo[c].max_interfaces = i;
|
||
combo[c].n_limits = i;
|
||
combo[c].limits = p2p_limits;
|
||
@@ -6271,14 +6261,19 @@ static int brcmf_setup_ifmodes(struct wi
|
||
|
||
if (mbss) {
|
||
c++;
|
||
+ i = 0;
|
||
+ mbss_limits = kcalloc(1, sizeof(*mbss_limits), GFP_KERNEL);
|
||
+ if (!mbss_limits)
|
||
+ goto err;
|
||
+ mbss_limits[i].max = 4;
|
||
+ mbss_limits[i++].types = BIT(NL80211_IFTYPE_AP);
|
||
combo[c].beacon_int_infra_match = true;
|
||
combo[c].num_different_channels = 1;
|
||
- mbss_limits[0].max = 4;
|
||
- mbss_limits[0].types = BIT(NL80211_IFTYPE_AP);
|
||
combo[c].max_interfaces = 4;
|
||
- combo[c].n_limits = 1;
|
||
+ combo[c].n_limits = i;
|
||
combo[c].limits = mbss_limits;
|
||
}
|
||
+
|
||
wiphy->n_iface_combinations = n_combos;
|
||
wiphy->iface_combinations = combo;
|
||
return 0;
|