diff for duplicates of <87k25duqct.fsf@kamboji.qca.qualcomm.com> diff --git a/a/1.txt b/N1/1.txt index 6d1804b..367c21e 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -12,7 +12,8 @@ Adrian Chadd <adrian@freebsd.org> writes: Oops, I accidentaly replied to the wrong version of the patch. This is a perfect example why adding version to the subject is so important: -https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#patch_version_missing +https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatc= +hes#patch_version_missing But anyway, this is what I wrote in v1: @@ -24,37 +25,43 @@ drivers/net/wireless/ath/ath10k/debug.h:209: return is not a function, parentheses are not required drivers/net/wireless/ath/ath10k/debug.h:210: return is not a function, parentheses are not required -drivers/net/wireless/ath/ath10k/debug.h:214: Alignment should match open parenthesis -drivers/net/wireless/ath/ath10k/debug.h:218: Alignment should match open parenthesis +drivers/net/wireless/ath/ath10k/debug.h:214: Alignment should match open pa= +renthesis +drivers/net/wireless/ath/ath10k/debug.h:218: Alignment should match open pa= +renthesis drivers/net/wireless/ath/ath10k/debug.c:2430: code indent should use tabs where possible -drivers/net/wireless/ath/ath10k/debug.c:2430: please, no spaces at the start of a line +drivers/net/wireless/ath/ath10k/debug.c:2430: please, no spaces at the star= +t of a line drivers/net/wireless/ath/ath10k/debug.c:2431: code indent should use tabs where possible -drivers/net/wireless/ath/ath10k/debug.c:2431: please, no spaces at the start of a line +drivers/net/wireless/ath/ath10k/debug.c:2431: please, no spaces at the star= +t of a line drivers/net/wireless/ath/ath10k/debug.c:2464: code indent should use tabs where possible -drivers/net/wireless/ath/ath10k/debug.c:2464: please, no spaces at the start of a line +drivers/net/wireless/ath/ath10k/debug.c:2464: please, no spaces at the star= +t of a line drivers/net/wireless/ath/ath10k/debug.c:2465: code indent should use tabs where possible -drivers/net/wireless/ath/ath10k/debug.c:2465: please, no spaces at the start of a line -drivers/net/wireless/ath/ath10k/debug.c:2493: Please don't use multiple blank lines +drivers/net/wireless/ath/ath10k/debug.c:2465: please, no spaces at the star= +t of a line +drivers/net/wireless/ath/ath10k/debug.c:2493: Please don't use multiple bla= +nk lines drivers/net/wireless/ath/ath10k/debug.c:2525: Symbolic permissions 'S_IRUSR' are not preferred. Consider using octal permissions '0400'. drivers/net/wireless/ath/ath10k/debug.c:2527: Symbolic permissions 'S_IRUSR' are not preferred. Consider using octal permissions '0400'. -drivers/net/wireless/ath/ath10k/debug.c:2620: Alignment should match open parenthesis -drivers/net/wireless/ath/ath10k/debug.c:2640: Alignment should match open parenthesis +drivers/net/wireless/ath/ath10k/debug.c:2620: Alignment should match open p= +arenthesis +drivers/net/wireless/ath/ath10k/debug.c:2640: Alignment should match open p= +arenthesis I fixed those in the pending branch: -https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=bd8c3bdce70adc201037b2eb7eda0a83911ef375 +https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=3Dp= +ending&id=3Dbd8c3bdce70adc201037b2eb7eda0a83911ef375 I'll look at this more closely later. --- -Kalle Valo -_______________________________________________ -ath10k mailing list -ath10k@lists.infradead.org -http://lists.infradead.org/mailman/listinfo/ath10k +--=20 +Kalle Valo= diff --git a/a/content_digest b/N1/content_digest index d2ace57..b684414 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,8 +3,8 @@ "Subject\0Re: [PATCH] ath10k: add configurable debugging.\0" "Date\0Fri, 19 May 2017 09:51:17 +0000\0" "To\0Adrian Chadd <adrian@freebsd.org>\0" - "Cc\0linux-wireless@vger.kernel.org <linux-wireless@vger.kernel.org>" - " ath10k@lists.infradead.org <ath10k@lists.infradead.org>\0" + "Cc\0ath10k@lists.infradead.org <ath10k@lists.infradead.org>" + " linux-wireless@vger.kernel.org <linux-wireless@vger.kernel.org>\0" "\00:1\0" "b\0" "Adrian Chadd <adrian@freebsd.org> writes:\n" @@ -21,7 +21,8 @@ "Oops, I accidentaly replied to the wrong version of the patch. This is a\n" "perfect example why adding version to the subject is so important:\n" "\n" - "https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#patch_version_missing\n" + "https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatc=\n" + "hes#patch_version_missing\n" "\n" "But anyway, this is what I wrote in v1:\n" "\n" @@ -33,39 +34,45 @@ "parentheses are not required\n" "drivers/net/wireless/ath/ath10k/debug.h:210: return is not a function,\n" "parentheses are not required\n" - "drivers/net/wireless/ath/ath10k/debug.h:214: Alignment should match open parenthesis\n" - "drivers/net/wireless/ath/ath10k/debug.h:218: Alignment should match open parenthesis\n" + "drivers/net/wireless/ath/ath10k/debug.h:214: Alignment should match open pa=\n" + "renthesis\n" + "drivers/net/wireless/ath/ath10k/debug.h:218: Alignment should match open pa=\n" + "renthesis\n" "drivers/net/wireless/ath/ath10k/debug.c:2430: code indent should use\n" "tabs where possible\n" - "drivers/net/wireless/ath/ath10k/debug.c:2430: please, no spaces at the start of a line\n" + "drivers/net/wireless/ath/ath10k/debug.c:2430: please, no spaces at the star=\n" + "t of a line\n" "drivers/net/wireless/ath/ath10k/debug.c:2431: code indent should use\n" "tabs where possible\n" - "drivers/net/wireless/ath/ath10k/debug.c:2431: please, no spaces at the start of a line\n" + "drivers/net/wireless/ath/ath10k/debug.c:2431: please, no spaces at the star=\n" + "t of a line\n" "drivers/net/wireless/ath/ath10k/debug.c:2464: code indent should use\n" "tabs where possible\n" - "drivers/net/wireless/ath/ath10k/debug.c:2464: please, no spaces at the start of a line\n" + "drivers/net/wireless/ath/ath10k/debug.c:2464: please, no spaces at the star=\n" + "t of a line\n" "drivers/net/wireless/ath/ath10k/debug.c:2465: code indent should use\n" "tabs where possible\n" - "drivers/net/wireless/ath/ath10k/debug.c:2465: please, no spaces at the start of a line\n" - "drivers/net/wireless/ath/ath10k/debug.c:2493: Please don't use multiple blank lines\n" + "drivers/net/wireless/ath/ath10k/debug.c:2465: please, no spaces at the star=\n" + "t of a line\n" + "drivers/net/wireless/ath/ath10k/debug.c:2493: Please don't use multiple bla=\n" + "nk lines\n" "drivers/net/wireless/ath/ath10k/debug.c:2525: Symbolic permissions\n" "'S_IRUSR' are not preferred. Consider using octal permissions '0400'.\n" "drivers/net/wireless/ath/ath10k/debug.c:2527: Symbolic permissions\n" "'S_IRUSR' are not preferred. Consider using octal permissions '0400'.\n" - "drivers/net/wireless/ath/ath10k/debug.c:2620: Alignment should match open parenthesis\n" - "drivers/net/wireless/ath/ath10k/debug.c:2640: Alignment should match open parenthesis\n" + "drivers/net/wireless/ath/ath10k/debug.c:2620: Alignment should match open p=\n" + "arenthesis\n" + "drivers/net/wireless/ath/ath10k/debug.c:2640: Alignment should match open p=\n" + "arenthesis\n" "\n" "I fixed those in the pending branch:\n" "\n" - "https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=bd8c3bdce70adc201037b2eb7eda0a83911ef375\n" + "https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=3Dp=\n" + "ending&id=3Dbd8c3bdce70adc201037b2eb7eda0a83911ef375\n" "\n" "I'll look at this more closely later.\n" "\n" - "-- \n" - "Kalle Valo\n" - "_______________________________________________\n" - "ath10k mailing list\n" - "ath10k@lists.infradead.org\n" - http://lists.infradead.org/mailman/listinfo/ath10k + "--=20\n" + Kalle Valo= -5a15938ebcb48db6d6fd68a41866de0a971ae25d386cc259125b3f41380e4057 +030cc83f29c72bd0dc16df15225252f72958c5daa2b3dd3f0da2b5a04e832040
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.