diff for duplicates of <87twa8yr22.fsf@kamboji.qca.qualcomm.com> diff --git a/a/1.txt b/N1/1.txt index a876b21..43b255d 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -2,7 +2,8 @@ Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: > Hello Kalle, > -> On Fri, Nov 25, 2016 at 4:06 PM, Valo, Kalle <kvalo@qca.qualcomm.com> wrote: +> On Fri, Nov 25, 2016 at 4:06 PM, Valo, Kalle <kvalo@qca.qualcomm.com> wro= +te: >> Kalle Valo <kvalo@codeaurora.org> writes: >> >>> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: @@ -22,7 +23,7 @@ Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: >>>> is Big Endian (this is also done by the FreeBSD kernel). >>>> This has a nice advantage: currently there are some out-of-tree hacks >>>> (in OpenWrt and LEDE) where the EEPROM has a Big Endian header on a ->>>> Big Endian system (= no swab16 is performed) but the EEPROM itself +>>>> Big Endian system (=3D no swab16 is performed) but the EEPROM itself >>>> indicates that it's data is Little Endian. Until now the out-of-tree >>>> code simply did a swab16 before passing the data to ath9k, so ath9k >>>> first did the swab16 - this also enabled the format specific swapping. @@ -32,7 +33,7 @@ Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: >>>> robustness and portability of the swapping logic in the original OF >>>> support patch review, see [0]. >>>> ->>>> After a second round of patches (= v1 of this series) neither Arnd +>>>> After a second round of patches (=3D v1 of this series) neither Arnd >>>> Bergmann nor I were really happy with the complexity of the EEPROM >>>> swapping logic. Based on a discussion (see [1] and [2]) we decided >>>> that ath9k should use a defined format (specifying the endianness @@ -61,7 +62,8 @@ Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: >>> I think this looks pretty good. If there's a bug somewhere it should be >>> quite easy to fix so I'm not that worried and would be willing to take >>> these as soon as I have applied the dependency series. IIRC your ->>> devicetree patches will have at least one more review round so that will +>>> devicetree patches will have at least one more review round so that wil= +l >>> take some time still. In the meantime it would be great if LEDE folks >>> could take a look at these and comment (or test). >> @@ -89,5 +91,5 @@ Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: Sounds good to me, thanks for the thorough followup. I'm planning to apply these any day. --- -Kalle Valo +--=20 +Kalle Valo= diff --git a/a/content_digest b/N1/content_digest index 86e760c..cb8cd63 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -4,16 +4,24 @@ "ref\0871sxza9al.fsf@kamboji.qca.qualcomm.com\0" "ref\0CAFBinCC6JWBhZwma=66fBi3_to2SaHOMNDQS23jHNhcc+RUcYQ@mail.gmail.com\0" "From\0Valo, Kalle <kvalo@qca.qualcomm.com>\0" - "Subject\0[ath9k-devel] [PATCH v2 0/7] ath9k: EEPROM swapping improvements\0" + "Subject\0Re: [PATCH v2 0/7] ath9k: EEPROM swapping improvements\0" "Date\0Tue, 13 Dec 2016 12:03:18 +0000\0" - "To\0ath9k-devel@lists.ath9k.org\0" + "To\0Martin Blumenstingl <martin.blumenstingl@googlemail.com>\0" + "Cc\0ath9k-devel <ath9k-devel@qca.qualcomm.com>" + linux-wireless@vger.kernel.org <linux-wireless@vger.kernel.org> + ath9k-devel@lists.ath9k.org <ath9k-devel@lists.ath9k.org> + devicetree@vger.kernel.org <devicetree@vger.kernel.org> + arnd@arndb.de <arnd@arndb.de> + chunkeey@googlemail.com <chunkeey@googlemail.com> + " nbd@nbd.name <nbd@nbd.name>\0" "\00:1\0" "b\0" "Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:\n" "\n" "> Hello Kalle,\n" ">\n" - "> On Fri, Nov 25, 2016 at 4:06 PM, Valo, Kalle <kvalo@qca.qualcomm.com> wrote:\n" + "> On Fri, Nov 25, 2016 at 4:06 PM, Valo, Kalle <kvalo@qca.qualcomm.com> wro=\n" + "te:\n" ">> Kalle Valo <kvalo@codeaurora.org> writes:\n" ">>\n" ">>> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:\n" @@ -33,7 +41,7 @@ ">>>> is Big Endian (this is also done by the FreeBSD kernel).\n" ">>>> This has a nice advantage: currently there are some out-of-tree hacks\n" ">>>> (in OpenWrt and LEDE) where the EEPROM has a Big Endian header on a\n" - ">>>> Big Endian system (= no swab16 is performed) but the EEPROM itself\n" + ">>>> Big Endian system (=3D no swab16 is performed) but the EEPROM itself\n" ">>>> indicates that it's data is Little Endian. Until now the out-of-tree\n" ">>>> code simply did a swab16 before passing the data to ath9k, so ath9k\n" ">>>> first did the swab16 - this also enabled the format specific swapping.\n" @@ -43,7 +51,7 @@ ">>>> robustness and portability of the swapping logic in the original OF\n" ">>>> support patch review, see [0].\n" ">>>>\n" - ">>>> After a second round of patches (= v1 of this series) neither Arnd\n" + ">>>> After a second round of patches (=3D v1 of this series) neither Arnd\n" ">>>> Bergmann nor I were really happy with the complexity of the EEPROM\n" ">>>> swapping logic. Based on a discussion (see [1] and [2]) we decided\n" ">>>> that ath9k should use a defined format (specifying the endianness\n" @@ -72,7 +80,8 @@ ">>> I think this looks pretty good. If there's a bug somewhere it should be\n" ">>> quite easy to fix so I'm not that worried and would be willing to take\n" ">>> these as soon as I have applied the dependency series. IIRC your\n" - ">>> devicetree patches will have at least one more review round so that will\n" + ">>> devicetree patches will have at least one more review round so that wil=\n" + "l\n" ">>> take some time still. In the meantime it would be great if LEDE folks\n" ">>> could take a look at these and comment (or test).\n" ">>\n" @@ -100,7 +109,7 @@ "Sounds good to me, thanks for the thorough followup. I'm planning to\n" "apply these any day.\n" "\n" - "-- \n" - Kalle Valo + "--=20\n" + Kalle Valo= -2f77252dd681d3def2973ad9fed876f4d1025667bc5e2d576607f36ed2987b29 +5127461234bec25eed263ea46d68be7e46ef49be393165dbeed8012673e5a893
diff --git a/a/1.txt b/N2/1.txt index a876b21..59cf86d 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,11 +1,11 @@ -Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: +Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> writes: > Hello Kalle, > -> On Fri, Nov 25, 2016 at 4:06 PM, Valo, Kalle <kvalo@qca.qualcomm.com> wrote: ->> Kalle Valo <kvalo@codeaurora.org> writes: +> On Fri, Nov 25, 2016 at 4:06 PM, Valo, Kalle <kvalo-A+ZNKFmMK5xy9aJCnZT0Uw@public.gmane.org> wrote: +>> Kalle Valo <kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> writes: >> ->>> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: +>>> Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> writes: >>> >>>> There are two types of swapping the EEPROM data in the ath9k driver. >>>> Before this series one type of swapping could not be used without the @@ -90,4 +90,7 @@ Sounds good to me, thanks for the thorough followup. I'm planning to apply these any day. -- -Kalle Valo +Kalle Valo-- +To unsubscribe from this list: send the line "unsubscribe devicetree" in +the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org +More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N2/content_digest index 86e760c..923230b 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -3,20 +3,28 @@ "ref\087insxg0yc.fsf@kamboji.qca.qualcomm.com\0" "ref\0871sxza9al.fsf@kamboji.qca.qualcomm.com\0" "ref\0CAFBinCC6JWBhZwma=66fBi3_to2SaHOMNDQS23jHNhcc+RUcYQ@mail.gmail.com\0" - "From\0Valo, Kalle <kvalo@qca.qualcomm.com>\0" - "Subject\0[ath9k-devel] [PATCH v2 0/7] ath9k: EEPROM swapping improvements\0" + "ref\0CAFBinCC6JWBhZwma=66fBi3_to2SaHOMNDQS23jHNhcc+RUcYQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org\0" + "From\0Valo, Kalle <kvalo-A+ZNKFmMK5xy9aJCnZT0Uw@public.gmane.org>\0" + "Subject\0Re: [PATCH v2 0/7] ath9k: EEPROM swapping improvements\0" "Date\0Tue, 13 Dec 2016 12:03:18 +0000\0" - "To\0ath9k-devel@lists.ath9k.org\0" + "To\0Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>\0" + "Cc\0ath9k-devel <ath9k-devel-A+ZNKFmMK5xy9aJCnZT0Uw@public.gmane.org>" + linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> + ath9k-devel-xDcbHBWguxHbcTqmT+pZeQ@public.gmane.org <ath9k-devel-xDcbHBWguxHbcTqmT+pZeQ@public.gmane.org> + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> + arnd-r2nGTMty4D4@public.gmane.org <arnd-r2nGTMty4D4@public.gmane.org> + chunkeey-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org <chunkeey-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> + " nbd-Vt+b4OUoWG0@public.gmane.org <nbd-Vt+b4OUoWG0@public.gmane.org>\0" "\00:1\0" "b\0" - "Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:\n" + "Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> writes:\n" "\n" "> Hello Kalle,\n" ">\n" - "> On Fri, Nov 25, 2016 at 4:06 PM, Valo, Kalle <kvalo@qca.qualcomm.com> wrote:\n" - ">> Kalle Valo <kvalo@codeaurora.org> writes:\n" + "> On Fri, Nov 25, 2016 at 4:06 PM, Valo, Kalle <kvalo-A+ZNKFmMK5xy9aJCnZT0Uw@public.gmane.org> wrote:\n" + ">> Kalle Valo <kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> writes:\n" ">>\n" - ">>> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:\n" + ">>> Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> writes:\n" ">>>\n" ">>>> There are two types of swapping the EEPROM data in the ath9k driver.\n" ">>>> Before this series one type of swapping could not be used without the\n" @@ -101,6 +109,9 @@ "apply these any day.\n" "\n" "-- \n" - Kalle Valo + "Kalle Valo--\n" + "To unsubscribe from this list: send the line \"unsubscribe devicetree\" in\n" + "the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\n" + More majordomo info at http://vger.kernel.org/majordomo-info.html -2f77252dd681d3def2973ad9fed876f4d1025667bc5e2d576607f36ed2987b29 +4426f6b4d01f67d1a9f12d678778ca545b8a690e41268bd52b28c1d1cf4fc4a3
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.