diff for duplicates of <87mvk4kc9p.fsf@ni.com> diff --git a/a/1.txt b/N1/1.txt index 9490955..ba137e2 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -3,13 +3,13 @@ On Mon, 22 Aug 2016, Kalle Valo <kvalo@codeaurora.org> wrote: > >> On Fr, 2016-08-19 at 13:03 +0300, Kalle Valo wrote: >>> Actually, I see two patches which might be related but not identical: ->>> +>>>=20 >>> ath9k: fix client mode beacon configuration >>> https://patchwork.kernel.org/patch/9247699/ ->>> +>>>=20 >>> ath9k: Fix beacon configuration assertion failure >>> https://patchwork.kernel.org/patch/9281191/ ->>> +>>>=20 >>> Felix (CCed) & Benjamin: please take a look and advice which one I >>> should take. >> @@ -18,7 +18,7 @@ On Mon, 22 Aug 2016, Kalle Valo <kvalo@codeaurora.org> wrote: >> Felix solution looks entirely correct to me, the second solution seems >> slightly wrong because it prevents the call to ath9k_beacon_config from >> happening instead of ensuring the correct parameter value. ->> ath9k_beacon_config needs to be called even if iter_data.beacons?is +>> ath9k_beacon_config needs to be called even if iter_data.beacons=C2=A0is >> false as it disables the interrupts. > > Ok, I'll the patch from Felix then. Thanks. @@ -26,5 +26,5 @@ On Mon, 22 Aug 2016, Kalle Valo <kvalo@codeaurora.org> wrote: Tested Felix patch. Confirm it works great. Thanks! > -> -- +> --=20 > Kalle Valo diff --git a/a/content_digest b/N1/content_digest index 59740b5..3243264 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,9 +3,15 @@ "ref\01471855240.16921.6.camel@open-mesh.com\0" "ref\087r39h3p0f.fsf@purkki.adurom.net\0" "From\0Ioan-Adrian Ratiu <adi@adirat.com>\0" - "Subject\0[ath9k-devel] ath9k: Fix beacon configuration assertion failure\0" - "Date\0Mon, 22 Aug 2016 16:20:30 -0000\0" - "To\0ath9k-devel@lists.ath9k.org\0" + "Subject\0Re: ath9k: Fix beacon configuration assertion failure\0" + "Date\0Mon, 22 Aug 2016 19:18:42 +0300\0" + "To\0Kalle Valo <kvalo@codeaurora.org>" + " Benjamin Berg <benjamin.berg@open-mesh.com>\0" + "Cc\0linux-wireless@vger.kernel.org" + ath9k-devel@qca.qualcomm.com + ath9k-devel@lists.ath9k.org + linux-kernel@vger.kernel.org + " Felix Fietkau <nbd@nbd.name>\0" "\00:1\0" "b\0" "On Mon, 22 Aug 2016, Kalle Valo <kvalo@codeaurora.org> wrote:\n" @@ -13,13 +19,13 @@ ">\n" ">> On Fr, 2016-08-19 at 13:03 +0300, Kalle Valo wrote:\n" ">>> Actually, I see two patches which might be related but not identical:\n" - ">>> \n" + ">>>=20\n" ">>> ath9k: fix client mode beacon configuration\n" ">>> https://patchwork.kernel.org/patch/9247699/\n" - ">>> \n" + ">>>=20\n" ">>> ath9k: Fix beacon configuration assertion failure\n" ">>> https://patchwork.kernel.org/patch/9281191/\n" - ">>> \n" + ">>>=20\n" ">>> Felix (CCed) & Benjamin: please take a look and advice which one I\n" ">>> should take.\n" ">>\n" @@ -28,7 +34,7 @@ ">> Felix solution looks entirely correct to me, the second solution seems\n" ">> slightly wrong because it prevents the call to ath9k_beacon_config from\n" ">> happening instead of ensuring the correct parameter value.\n" - ">> ath9k_beacon_config needs to be called even if iter_data.beacons?is\n" + ">> ath9k_beacon_config needs to be called even if iter_data.beacons=C2=A0is\n" ">> false as it disables the interrupts.\n" ">\n" "> Ok, I'll the patch from Felix then. Thanks.\n" @@ -36,7 +42,7 @@ "Tested Felix patch. Confirm it works great. Thanks!\n" "\n" ">\n" - "> -- \n" + "> --=20\n" > Kalle Valo -826a895aa68b8db4778d0ff0bd1ef74bf8e67a26111ebac3a8678b83020b425d +04a07c2ce0398be117df101ababb9463e018ac0ad7354922830f66efe62e579a
diff --git a/a/1.txt b/N2/1.txt index 9490955..5b25007 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -18,7 +18,7 @@ On Mon, 22 Aug 2016, Kalle Valo <kvalo@codeaurora.org> wrote: >> Felix solution looks entirely correct to me, the second solution seems >> slightly wrong because it prevents the call to ath9k_beacon_config from >> happening instead of ensuring the correct parameter value. ->> ath9k_beacon_config needs to be called even if iter_data.beacons?is +>> ath9k_beacon_config needs to be called even if iter_data.beacons is >> false as it disables the interrupts. > > Ok, I'll the patch from Felix then. Thanks. diff --git a/a/content_digest b/N2/content_digest index 59740b5..e96a6dd 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -3,9 +3,15 @@ "ref\01471855240.16921.6.camel@open-mesh.com\0" "ref\087r39h3p0f.fsf@purkki.adurom.net\0" "From\0Ioan-Adrian Ratiu <adi@adirat.com>\0" - "Subject\0[ath9k-devel] ath9k: Fix beacon configuration assertion failure\0" - "Date\0Mon, 22 Aug 2016 16:20:30 -0000\0" - "To\0ath9k-devel@lists.ath9k.org\0" + "Subject\0Re: ath9k: Fix beacon configuration assertion failure\0" + "Date\0Mon, 22 Aug 2016 19:18:42 +0300\0" + "To\0Kalle Valo <kvalo@codeaurora.org>" + " Benjamin Berg <benjamin.berg@open-mesh.com>\0" + "Cc\0linux-wireless@vger.kernel.org" + ath9k-devel@qca.qualcomm.com + ath9k-devel@venema.h4ckr.net + linux-kernel@vger.kernel.org + " Felix Fietkau <nbd@nbd.name>\0" "\00:1\0" "b\0" "On Mon, 22 Aug 2016, Kalle Valo <kvalo@codeaurora.org> wrote:\n" @@ -28,7 +34,7 @@ ">> Felix solution looks entirely correct to me, the second solution seems\n" ">> slightly wrong because it prevents the call to ath9k_beacon_config from\n" ">> happening instead of ensuring the correct parameter value.\n" - ">> ath9k_beacon_config needs to be called even if iter_data.beacons?is\n" + ">> ath9k_beacon_config needs to be called even if iter_data.beacons\302\240is\n" ">> false as it disables the interrupts.\n" ">\n" "> Ok, I'll the patch from Felix then. Thanks.\n" @@ -39,4 +45,4 @@ "> -- \n" > Kalle Valo -826a895aa68b8db4778d0ff0bd1ef74bf8e67a26111ebac3a8678b83020b425d +9cbcfd473183cdc00e4248cbfa9e653961604d146c8ad4c8ffd89398ee5fac05
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.