Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1580076 > unrolled thread

[PATCH 01/15] staging: rtl8192u: fixing white space issue

Back to article view | Back to linux.kernel

Contents

  [PATCH 01/15] staging: rtl8192u: fixing white space issue simran singhal <singhalsimran0@gmail.com> - 2017-02-13 22:10 +0100
    Re: [PATCH 01/15] staging: rtl8192u: fixing white space issue Greg KH <gregkh@linuxfoundation.org> - 2017-02-14 18:30 +0100
      Re: [PATCH 01/15] staging: rtl8192u: fixing white space issue SIMRAN SINGHAL <singhalsimran0@gmail.com> - 2017-02-14 18:50 +0100
        Re: [PATCH 01/15] staging: rtl8192u: fixing white space issue Greg KH <gregkh@linuxfoundation.org> - 2017-02-14 19:10 +0100

#1580076 — [PATCH 01/15] staging: rtl8192u: fixing white space issue

Fix the following checkpatch.pl error and warnings:
WARNING: please, no space before tabs

Signed-off-by: simran singhal <singhalsimran0@gmail.com>
---
 drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h b/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h
index 873969c..a54dfad 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h
@@ -30,7 +30,7 @@ typedef struct _TX_TS_RECORD{
 	u16				TxCurSeq;
 	BA_RECORD			TxPendingBARecord;	/*  For BA Originator */
 	BA_RECORD			TxAdmittedBARecord;	/*  For BA Originator */
-/* 	QOS_DL_RECORD		DLRecord; */
+/*	QOS_DL_RECORD	DLRecord; */
 	u8				bAddBaReqInProgress;
 	u8				bAddBaReqDelayed;
 	u8				bUsingBa;
@@ -48,7 +48,7 @@ typedef struct _RX_TS_RECORD {
 	u16				RxLastSeqNum;
 	u8				RxLastFragNum;
 	u8				num;
-/* 	QOS_DL_RECORD		DLRecord; */
+/*	QOS_DL_RECOR	DLRecord; */
 } RX_TS_RECORD, *PRX_TS_RECORD;
 
 
-- 
2.7.4

[toc] | [next] | [standalone]

#1580704

On Tue, Feb 14, 2017 at 02:33:34AM +0530, simran singhal wrote:
> Fix the following checkpatch.pl error and warnings:
> WARNING: please, no space before tabs
> 
> Signed-off-by: simran singhal <singhalsimran0@gmail.com>
> ---
>  drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

You said this is a 15 patch patch series, yet I only received 9 of them.

Please fix and resend all of them, properly threaded (hint, git
send-email will do this for you automatically, or you can do it by hand
with a good email client like mutt), so that I know I get them all and
can review and apply them.

thanks,

greg k-h

[toc] | [prev] | [next] | [standalone]

#1580713

While resending them ...do I have to mention v2.

On Tue, Feb 14, 2017 at 10:54 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Tue, Feb 14, 2017 at 02:33:34AM +0530, simran singhal wrote:
>> Fix the following checkpatch.pl error and warnings:
>> WARNING: please, no space before tabs
>>
>> Signed-off-by: simran singhal <singhalsimran0@gmail.com>
>> ---
>>  drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> You said this is a 15 patch patch series, yet I only received 9 of them.
>
> Please fix and resend all of them, properly threaded (hint, git
> send-email will do this for you automatically, or you can do it by hand
> with a good email client like mutt), so that I know I get them all and
> can review and apply them.
>
> thanks,
>
> greg k-h

[toc] | [prev] | [next] | [standalone]

#1580728

On Tue, Feb 14, 2017 at 11:16:59PM +0530, SIMRAN SINGHAL wrote:
> While resending them ...do I have to mention v2.

Of course, why wouldn't you?  :)

thanks,

greg k-h

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web