diff for duplicates of <87fuannse6.fsf@kamboji.qca.qualcomm.com> diff --git a/a/1.txt b/N1/1.txt index 7559082..b01b232 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -8,14 +8,16 @@ Christos Gkekas <chris.gekas@gmail.com> writes: > drivers/net/wireless/ath/ath10k/spectral.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > -> diff --git a/drivers/net/wireless/ath/ath10k/spectral.c b/drivers/net/wireless/ath/ath10k/spectral.c +> diff --git a/drivers/net/wireless/ath/ath10k/spectral.c b/drivers/net/wir= +eless/ath/ath10k/spectral.c > index dd9cc09..1867937 100644 > --- a/drivers/net/wireless/ath/ath10k/spectral.c > +++ b/drivers/net/wireless/ath/ath10k/spectral.c -> @@ -403,10 +403,7 @@ static ssize_t write_file_spectral_count(struct file *file, +> @@ -403,10 +403,7 @@ static ssize_t write_file_spectral_count(struct file= + *file, > return -EFAULT; -> -> buf[len] = '\0'; +> =20 +> buf[len] =3D '\0'; > - if (kstrtoul(buf, 0, &val)) > - return -EINVAL; > - @@ -27,9 +29,5 @@ Removing the check for negative is correct but I don't think you are simplifying anything, on the contrary it's harder to read. Please keep the two if statements separate. --- -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 b068c57..dd6240a 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,10 +3,10 @@ "Subject\0Re: [PATCH] ath10k: spectral: Simplify error checking\0" "Date\0Fri, 13 Oct 2017 12:28:50 +0000\0" "To\0Christos Gkekas <chris.gekas@gmail.com>\0" - "Cc\0netdev@vger.kernel.org <netdev@vger.kernel.org>" + "Cc\0ath10k@lists.infradead.org <ath10k@lists.infradead.org>" linux-wireless@vger.kernel.org <linux-wireless@vger.kernel.org> - linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> - " ath10k@lists.infradead.org <ath10k@lists.infradead.org>\0" + netdev@vger.kernel.org <netdev@vger.kernel.org> + " linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>\0" "\00:1\0" "b\0" "Christos Gkekas <chris.gekas@gmail.com> writes:\n" @@ -19,14 +19,16 @@ "> drivers/net/wireless/ath/ath10k/spectral.c | 5 +----\n" "> 1 file changed, 1 insertion(+), 4 deletions(-)\n" ">\n" - "> diff --git a/drivers/net/wireless/ath/ath10k/spectral.c b/drivers/net/wireless/ath/ath10k/spectral.c\n" + "> diff --git a/drivers/net/wireless/ath/ath10k/spectral.c b/drivers/net/wir=\n" + "eless/ath/ath10k/spectral.c\n" "> index dd9cc09..1867937 100644\n" "> --- a/drivers/net/wireless/ath/ath10k/spectral.c\n" "> +++ b/drivers/net/wireless/ath/ath10k/spectral.c\n" - "> @@ -403,10 +403,7 @@ static ssize_t write_file_spectral_count(struct file *file,\n" + "> @@ -403,10 +403,7 @@ static ssize_t write_file_spectral_count(struct file=\n" + " *file,\n" "> \t\treturn -EFAULT;\n" - "> \n" - "> \tbuf[len] = '\\0';\n" + "> =20\n" + "> \tbuf[len] =3D '\\0';\n" "> -\tif (kstrtoul(buf, 0, &val))\n" "> -\t\treturn -EINVAL;\n" "> -\n" @@ -38,11 +40,7 @@ "simplifying anything, on the contrary it's harder to read. Please keep\n" "the two if statements separate.\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= -4fd3d0d93c5dce4d31cd43ab520991d4170e22e03e9f2891b480801c0f5c911c +1017f45ce75169dba7ed2ca7aba2f48d22193ddc6ad3bf9464d8e865f54eadeb
diff --git a/a/1.txt b/N2/1.txt index 7559082..00b39bb 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -29,7 +29,3 @@ the two if statements separate. -- Kalle Valo -_______________________________________________ -ath10k mailing list -ath10k@lists.infradead.org -http://lists.infradead.org/mailman/listinfo/ath10k diff --git a/a/content_digest b/N2/content_digest index b068c57..ebd32ed 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -3,10 +3,10 @@ "Subject\0Re: [PATCH] ath10k: spectral: Simplify error checking\0" "Date\0Fri, 13 Oct 2017 12:28:50 +0000\0" "To\0Christos Gkekas <chris.gekas@gmail.com>\0" - "Cc\0netdev@vger.kernel.org <netdev@vger.kernel.org>" + "Cc\0ath10k@lists.infradead.org <ath10k@lists.infradead.org>" linux-wireless@vger.kernel.org <linux-wireless@vger.kernel.org> - linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> - " ath10k@lists.infradead.org <ath10k@lists.infradead.org>\0" + netdev@vger.kernel.org <netdev@vger.kernel.org> + " linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>\0" "\00:1\0" "b\0" "Christos Gkekas <chris.gekas@gmail.com> writes:\n" @@ -39,10 +39,6 @@ "the two if statements separate.\n" "\n" "-- \n" - "Kalle Valo\n" - "_______________________________________________\n" - "ath10k mailing list\n" - "ath10k@lists.infradead.org\n" - http://lists.infradead.org/mailman/listinfo/ath10k + Kalle Valo -4fd3d0d93c5dce4d31cd43ab520991d4170e22e03e9f2891b480801c0f5c911c +48db00c59e408f61b46fd4644aacc87f2b828339f9151770e1d74b0964b33f30
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.