diff for duplicates of <87funj5pr8.fsf@kamboji.qca.qualcomm.com> diff --git a/a/1.txt b/N1/1.txt index 25db954..243630d 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -2,17 +2,15 @@ Arnd Bergmann <arnd@arndb.de> writes: > On Wednesday, October 26, 2016 9:49:58 AM CEST Kalle Valo wrote: >> Arnd Bergmann <arnd@arndb.de> writes: ->>=20 +>> >> > 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 =E2=80=98brcmf_cfg80211_sta= -rt_ap=E2=80=99: ->> > brcm80211/brcmfmac/cfg80211.c:4586:10: error: =E2=80=98is_11d=E2=80=99= - may be used uninitialized in this function [-Werror=3Dmaybe-uninitialized] +>> > 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 @@ -23,7 +21,7 @@ rt_ap=E2=80=99: >> > Cc: Arend van Spriel <arend.vanspriel@broadcom.com> >> > Cc: Kalle Valo <kvalo@codeaurora.org> >> > Signed-off-by: Arnd Bergmann <arnd@arndb.de> ->>=20 +>> >> Via which tree are you planning to submit this? Should I take it? > > I'd prefer if you can take it and forward it along with your other @@ -32,5 +30,5 @@ rt_ap=E2=80=99: Ok, I'll take it. I'm planning to push this to 4.9. ---=20 +-- Kalle Valo diff --git a/a/content_digest b/N1/content_digest index 54415e1..e4a4f21 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -23,17 +23,15 @@ "\n" "> On Wednesday, October 26, 2016 9:49:58 AM CEST Kalle Valo wrote:\n" ">> Arnd Bergmann <arnd@arndb.de> writes:\n" - ">>=20\n" + ">> \n" ">> > A bugfix added a sanity check around the assignment and use of the\n" ">> > 'is_11d' variable, which looks correct to me, but as the function is\n" ">> > rather complex already, this confuses the compiler to the point where\n" ">> > it can no longer figure out if the variable is always initialized\n" ">> > correctly:\n" ">> >\n" - ">> > brcm80211/brcmfmac/cfg80211.c: In function =E2=80=98brcmf_cfg80211_sta=\n" - "rt_ap=E2=80=99:\n" - ">> > brcm80211/brcmfmac/cfg80211.c:4586:10: error: =E2=80=98is_11d=E2=80=99=\n" - " may be used uninitialized in this function [-Werror=3Dmaybe-uninitialized]\n" + ">> > brcm80211/brcmfmac/cfg80211.c: In function \342\200\230brcmf_cfg80211_start_ap\342\200\231:\n" + ">> > brcm80211/brcmfmac/cfg80211.c:4586:10: error: \342\200\230is_11d\342\200\231 may be used uninitialized in this function [-Werror=maybe-uninitialized]\n" ">> >\n" ">> > This adds an initialization for the newly introduced case in which\n" ">> > the variable should not really be used, in order to make the warning\n" @@ -44,7 +42,7 @@ ">> > Cc: Arend van Spriel <arend.vanspriel@broadcom.com>\n" ">> > Cc: Kalle Valo <kvalo@codeaurora.org>\n" ">> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>\n" - ">>=20\n" + ">> \n" ">> Via which tree are you planning to submit this? Should I take it?\n" ">\n" "> I'd prefer if you can take it and forward it along with your other\n" @@ -53,7 +51,7 @@ "\n" "Ok, I'll take it. I'm planning to push this to 4.9.\n" "\n" - "--=20\n" + "-- \n" Kalle Valo -767c802acc0e5da58ede02bd0454051a72ee665a190aa0532aa2898d12398306 +eb0d3f0336855ed203f9fbc7fc1cb9a80d24dbb0b0c4ad7b62062e549dac64d3
diff --git a/a/1.txt b/N2/1.txt index 25db954..a6b4277 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,29 +1,27 @@ -Arnd Bergmann <arnd@arndb.de> writes: +Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> writes: > On Wednesday, October 26, 2016 9:49:58 AM CEST Kalle Valo wrote: ->> Arnd Bergmann <arnd@arndb.de> writes: ->>=20 +>> Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> writes: +>> >> > 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 =E2=80=98brcmf_cfg80211_sta= -rt_ap=E2=80=99: ->> > brcm80211/brcmfmac/cfg80211.c:4586:10: error: =E2=80=98is_11d=E2=80=99= - may be used uninitialized in this function [-Werror=3Dmaybe-uninitialized] +>> > 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 <hante.meuleman@broadcom.com> ->> > Cc: Arend van Spriel <arend.vanspriel@broadcom.com> ->> > Cc: Kalle Valo <kvalo@codeaurora.org> ->> > Signed-off-by: Arnd Bergmann <arnd@arndb.de> ->>=20 +>> > Cc: Hante Meuleman <hante.meuleman-dY08KVG/lbpWk0Htik3J/w@public.gmane.org> +>> > Cc: Arend van Spriel <arend.vanspriel-dY08KVG/lbpWk0Htik3J/w@public.gmane.org> +>> > Cc: Kalle Valo <kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> +>> > Signed-off-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> +>> >> Via which tree are you planning to submit this? Should I take it? > > I'd prefer if you can take it and forward it along with your other @@ -32,5 +30,5 @@ rt_ap=E2=80=99: Ok, I'll take it. I'm planning to push this to 4.9. ---=20 +-- Kalle Valo diff --git a/a/content_digest b/N2/content_digest index 54415e1..f9fc7af 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -2,49 +2,47 @@ "ref\020161017221355.1861551-7-arnd@arndb.de\0" "ref\087zilr61ux.fsf@kamboji.qca.qualcomm.com\0" "ref\03406231.8mt2808XDi@wuerfel\0" - "From\0Kalle Valo <kvalo@codeaurora.org>\0" + "From\0Kalle Valo <kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>\0" "Subject\0Re: [PATCH 19/28] brcmfmac: avoid maybe-uninitialized warning in brcmf_cfg80211_start_ap\0" "Date\0Wed, 26 Oct 2016 14:11:23 +0300\0" - "To\0Arnd Bergmann <arnd@arndb.de>\0" - "Cc\0Arend van Spriel <arend.vanspriel@broadcom.com>" - Linus Torvalds <torvalds@linux-foundation.org> - linux-kernel@vger.kernel.org - Hante Meuleman <hante.meuleman@broadcom.com> - Franky Lin <franky.lin@broadcom.com> - Pieter-Paul Giesberts <pieter-paul.giesberts@broadcom.com> - Franky (Zhenhui) Lin <frankyl@broadcom.com> - " Rafa\305\202 Mi\305\202ecki <rafal@milecki.pl>" - linux-wireless@vger.kernel.org - brcm80211-dev-list.pdl@broadcom.com - " netdev@vger.kernel.org\0" + "To\0Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>\0" + "Cc\0Arend van Spriel <arend.vanspriel-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>" + Linus Torvalds <torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org> + linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + Hante Meuleman <hante.meuleman-dY08KVG/lbpWk0Htik3J/w@public.gmane.org> + Franky Lin <franky.lin-dY08KVG/lbpWk0Htik3J/w@public.gmane.org> + Pieter-Paul Giesberts <pieter-paul.giesberts-dY08KVG/lbpWk0Htik3J/w@public.gmane.org> + Franky (Zhenhui) Lin <frankyl-dY08KVG/lbpWk0Htik3J/w@public.gmane.org> + " Rafa\305\202 Mi\305\202ecki <rafal-g1n6cQUeyibVItvQsEIGlw@public.gmane.org>" + linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + brcm80211-dev-list.pdl-dY08KVG/lbpWk0Htik3J/w@public.gmane.org + " netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\0" "\00:1\0" "b\0" - "Arnd Bergmann <arnd@arndb.de> writes:\n" + "Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> writes:\n" "\n" "> On Wednesday, October 26, 2016 9:49:58 AM CEST Kalle Valo wrote:\n" - ">> Arnd Bergmann <arnd@arndb.de> writes:\n" - ">>=20\n" + ">> Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> writes:\n" + ">> \n" ">> > A bugfix added a sanity check around the assignment and use of the\n" ">> > 'is_11d' variable, which looks correct to me, but as the function is\n" ">> > rather complex already, this confuses the compiler to the point where\n" ">> > it can no longer figure out if the variable is always initialized\n" ">> > correctly:\n" ">> >\n" - ">> > brcm80211/brcmfmac/cfg80211.c: In function =E2=80=98brcmf_cfg80211_sta=\n" - "rt_ap=E2=80=99:\n" - ">> > brcm80211/brcmfmac/cfg80211.c:4586:10: error: =E2=80=98is_11d=E2=80=99=\n" - " may be used uninitialized in this function [-Werror=3Dmaybe-uninitialized]\n" + ">> > brcm80211/brcmfmac/cfg80211.c: In function \342\200\230brcmf_cfg80211_start_ap\342\200\231:\n" + ">> > brcm80211/brcmfmac/cfg80211.c:4586:10: error: \342\200\230is_11d\342\200\231 may be used uninitialized in this function [-Werror=maybe-uninitialized]\n" ">> >\n" ">> > This adds an initialization for the newly introduced case in which\n" ">> > the variable should not really be used, in order to make the warning\n" ">> > go away.\n" ">> >\n" ">> > Fixes: b3589dfe0212 (\"brcmfmac: ignore 11d configuration errors\")\n" - ">> > Cc: Hante Meuleman <hante.meuleman@broadcom.com>\n" - ">> > Cc: Arend van Spriel <arend.vanspriel@broadcom.com>\n" - ">> > Cc: Kalle Valo <kvalo@codeaurora.org>\n" - ">> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>\n" - ">>=20\n" + ">> > Cc: Hante Meuleman <hante.meuleman-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>\n" + ">> > Cc: Arend van Spriel <arend.vanspriel-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>\n" + ">> > Cc: Kalle Valo <kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>\n" + ">> > Signed-off-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>\n" + ">> \n" ">> Via which tree are you planning to submit this? Should I take it?\n" ">\n" "> I'd prefer if you can take it and forward it along with your other\n" @@ -53,7 +51,7 @@ "\n" "Ok, I'll take it. I'm planning to push this to 4.9.\n" "\n" - "--=20\n" + "-- \n" Kalle Valo -767c802acc0e5da58ede02bd0454051a72ee665a190aa0532aa2898d12398306 +1bca25a0648d761f5b76e933961f096ed536ab73fe61be649edfbe7820c52037
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.