linux-kernel - Re: [PATCH] Staging: rtl8187se: Fix whitespace issues

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20110902080457.GE2430@shale.localdomain>
Date:	Fri, 2 Sep 2011 11:04:57 +0300
From:	Dan Carpenter <error27@...il.com>
To:	Gerard Snitselaar <dev@...tselaar.org>
Cc:	gregkh@...e.de, devel@...verdev.osuosl.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Staging: rtl8187se: Fix whitespace issues

On Fri, Sep 02, 2011 at 12:51:53AM -0700, Gerard Snitselaar wrote:
>  #define MAX_LD_SLOT_NUM 10
> -#define KEEP_ALIVE_INTERVAL 				20 // in seconds.
> +#define KEEP_ALIVE_INTERVAL				20 // in seconds.
>  #define CHECK_FOR_HANG_PERIOD			2 //be equal to watchdog check time
>  #define DEFAULT_KEEP_ALIVE_LEVEL			1
>  #define DEFAULT_SLOT_NUM					2

It would be better if the 1 and 2 were lined up.

> @@ -591,14 +591,14 @@ typedef struct r8180_priv
>  	int	   ForcedDataRate; // Force Data Rate. 0: Auto, 0x02: 1M ~ 0x6C: 54M.)
>  	u32     NumTxUnicast; //YJ,add,080828,for keep alive
>  	u8      keepAliveLevel; //YJ,add,080828,for KeepAlive
> -	unsigned long 	NumTxOkTotal;
> +	unsigned long	NumTxOkTotal;
>  	u16                                 LastRetryCnt;
>          u16                                     LastRetryRate;
>          unsigned long       LastTxokCnt;
>          unsigned long           LastRxokCnt;
>          u16                                     CurrRetryCnt;
>          unsigned long           LastTxOKBytes;
> -	unsigned long 		    NumTxOkBytesTotal;
> +	unsigned long		    NumTxOkBytesTotal;
>          u8                          LastFailTxRate;
>          long                        LastFailTxRateSS;
>          u8                          FailTxRateCount;

These are all higgledy-piggledy.

> @@ -579,7 +579,7 @@ StaRateAdaptive87SE(
>  	u32			TxThroughput;
>  	long		CurrSignalStrength;
>  	bool		bUpdateInitialGain = false;
> -    	u8			u1bOfdm=0, u1bCck = 0;
> +	u8			u1bOfdm=0, u1bCck = 0;
                                      ^^^
Spaces around the '=' char.

> @@ -846,7 +846,7 @@ StaRateAdaptive87SE(
>  			bTryDown = true;
>  			priv->TryDownCountLowData += TryDownTh;
>  		}
> -  		else if ( (CurrRetryRate<20) && (priv->LastRetryRate<21)) //TO DO: need to consider (RSSI)
> +		else if ( (CurrRetryRate<20) && (priv->LastRetryRate<21)) //TO DO: need to consider (RSSI)
                         ^             ^^^                         ^^^
Delete extra space, add spaces around the '<' chars.

> --- a/drivers/staging/rtl8187se/r8185b_init.c
> +++ b/drivers/staging/rtl8187se/r8185b_init.c
> @@ -172,7 +172,7 @@ if (offset == PhyAddr)	{
>  		*/
>  /*		NdisAcquireSpinLock( &(pDevice->IoSpinLock) );	*/
>  
> -		for (idx = 0; idx < 30; idx++)	{ 
> +		for (idx = 0; idx < 30; idx++)	{
                                               ^
Delete the second space here.

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/