Re: [PATCH 02/20] staging: rtl8192u: r8192U_core: fix consistent spacing code style error

2015-08-17 Thread Sudip Mukherjee
On Tue, Aug 18, 2015 at 02:02:49AM -0400, Raphaël Beamonte wrote: > 2015-08-18 1:44 GMT-04:00 Sudip Mukherjee : > > I think that will be better, since you are sending the patch for > > consistent spacing. > > Actually, going through the bunch of patches, it seems that the > whitespace neatening on

Re: [PATCH 02/20] staging: rtl8192u: r8192U_core: fix consistent spacing code style error

2015-08-17 Thread Raphaël Beamonte
2015-08-18 1:44 GMT-04:00 Sudip Mukherjee : > I think that will be better, since you are sending the patch for > consistent spacing. Actually, going through the bunch of patches, it seems that the whitespace neatening one is taking care of that. That one patch was mostly about taking care of the c

Re: [PATCH 02/20] staging: rtl8192u: r8192U_core: fix consistent spacing code style error

2015-08-17 Thread Sudip Mukherjee
On Tue, Aug 18, 2015 at 01:38:21AM -0400, Raphaël Beamonte wrote: > 2015-08-18 1:02 GMT-04:00 Sudip Mukherjee : > > since you are modifying these lines for consistent spacing, it will be > > better to have: > > (priv->ShortRetryLimit << TCR_SRL_OFFSET) | // Short retry limit > > > This shoul

Re: [PATCH 02/20] staging: rtl8192u: r8192U_core: fix consistent spacing code style error

2015-08-17 Thread Raphaël Beamonte
2015-08-18 1:02 GMT-04:00 Sudip Mukherjee : > since you are modifying these lines for consistent spacing, it will be > better to have: > (priv->ShortRetryLimit << TCR_SRL_OFFSET) | // Short retry limit > This should be: > priv->eeprom_CustomerID = eprom_read(dev, (EEPROM_Customer_ID>>1)) >>

Re: [PATCH 02/20] staging: rtl8192u: r8192U_core: fix consistent spacing code style error

2015-08-17 Thread Sudip Mukherjee
On Sat, Aug 15, 2015 at 09:33:58PM -0400, Raphaël Beamonte wrote: > Fix multiple occurences of the need consistent spacing code style error > > Signed-off-by: Raphaël Beamonte > --- > drivers/staging/rtl8192u/r8192U_core.c | 28 ++-- > 1 file changed, 14 insertions(+), 14

[PATCH 02/20] staging: rtl8192u: r8192U_core: fix consistent spacing code style error

2015-08-15 Thread Raphaël Beamonte
Fix multiple occurences of the need consistent spacing code style error Signed-off-by: Raphaël Beamonte --- drivers/staging/rtl8192u/r8192U_core.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/