From 4e611ac5dfe16bde1741ab76b1cd5d7d97bf9f1e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= Date: Thu, 2 Feb 2017 09:21:13 +0100 Subject: mac80211: brcmfmac: backport some old patches from 2016 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Rafał Miłecki --- ...oid-maybe-uninitialized-warning-in-brcmf_.patch | 43 ++++++++++++++++++++++ 1 file changed, 43 insertions(+) create mode 100644 package/kernel/mac80211/patches/356-0001-brcmfmac-avoid-maybe-uninitialized-warning-in-brcmf_.patch (limited to 'package/kernel/mac80211/patches/356-0001-brcmfmac-avoid-maybe-uninitialized-warning-in-brcmf_.patch') diff --git a/package/kernel/mac80211/patches/356-0001-brcmfmac-avoid-maybe-uninitialized-warning-in-brcmf_.patch b/package/kernel/mac80211/patches/356-0001-brcmfmac-avoid-maybe-uninitialized-warning-in-brcmf_.patch new file mode 100644 index 0000000..e7ca4be --- /dev/null +++ b/package/kernel/mac80211/patches/356-0001-brcmfmac-avoid-maybe-uninitialized-warning-in-brcmf_.patch @@ -0,0 +1,43 @@ +From d3532ea6ce4ea501e421d130555e59edc2945f99 Mon Sep 17 00:00:00 2001 +From: Arnd Bergmann +Date: Tue, 18 Oct 2016 00:13:40 +0200 +Subject: [PATCH] brcmfmac: avoid maybe-uninitialized warning in + brcmf_cfg80211_start_ap +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +A bugfix added a sanity check around the assignment and use of the +'is_11d' variable, which looks correct to me, but as the function is +rather complex already, this confuses the compiler to the point where +it can no longer figure out if the variable is always initialized +correctly: + +brcm80211/brcmfmac/cfg80211.c: In function ‘brcmf_cfg80211_start_ap’: +brcm80211/brcmfmac/cfg80211.c:4586:10: error: ‘is_11d’ may be used uninitialized in this function [-Werror=maybe-uninitialized] + +This adds an initialization for the newly introduced case in which +the variable should not really be used, in order to make the warning +go away. + +Fixes: b3589dfe0212 ("brcmfmac: ignore 11d configuration errors") +Cc: Hante Meuleman +Cc: Arend van Spriel +Cc: Kalle Valo +Signed-off-by: Arnd Bergmann +Signed-off-by: Kalle Valo +--- + drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c ++++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c +@@ -4516,7 +4516,7 @@ brcmf_cfg80211_start_ap(struct wiphy *wi + /* store current 11d setting */ + if (brcmf_fil_cmd_int_get(ifp, BRCMF_C_GET_REGULATORY, + &ifp->vif->is_11d)) { +- supports_11d = false; ++ is_11d = supports_11d = false; + } else { + country_ie = brcmf_parse_tlvs((u8 *)settings->beacon.tail, + settings->beacon.tail_len, -- cgit v1.1