Re: [PATCH] Staging: bcm: Checkpatch fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



It would be better to see a patch which will remove all of this crap
instead of fixing that...

On 10/23/12, Ceri James <jamesceri@xxxxxxxxxxxxxx> wrote:
> This fixes the following checkpatch issues:
>
> WARNING: line over 80 characters
> +	BCM_DEBUG_PRINT(Adapter,DBG_TYPE_TX, TOKEN_COUNTS, DBG_LVL_ALL,
> "=====>\n");
>
> ERROR: space required after that ',' (ctx:VxV)
> +	BCM_DEBUG_PRINT(Adapter,DBG_TYPE_TX, TOKEN_COUNTS, DBG_LVL_ALL,
> "=====>\n");
>  	                       ^
> WARNING: line over 80 characters
> +		BCM_DEBUG_PRINT(Adapter,DBG_TYPE_TX, TOKEN_COUNTS, DBG_LVL_ALL, "Adapter
> found NULL!\n");
>
> ERROR: space required after that ',' (ctx:VxV)
> +		BCM_DEBUG_PRINT(Adapter,DBG_TYPE_TX, TOKEN_COUNTS, DBG_LVL_ALL, "Adapter
> found NULL!\n");
>  		                       ^
>
> Signed-off-by: Ceri James <jamesceri@xxxxxxxxxxxxxx>
> ---
>  drivers/staging/bcm/LeakyBucket.c |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/bcm/LeakyBucket.c
> b/drivers/staging/bcm/LeakyBucket.c
> index 6e8a327..877cf0b 100644
> --- a/drivers/staging/bcm/LeakyBucket.c
> +++ b/drivers/staging/bcm/LeakyBucket.c
> @@ -21,10 +21,12 @@ static VOID UpdateTokenCount(register struct
> bcm_mini_adapter *Adapter)
>  	INT 	i = 0;
>  	struct timeval tv;
>
> -	BCM_DEBUG_PRINT(Adapter,DBG_TYPE_TX, TOKEN_COUNTS, DBG_LVL_ALL,
> "=====>\n");
> +	BCM_DEBUG_PRINT(Adapter, DBG_TYPE_TX, TOKEN_COUNTS, DBG_LVL_ALL,
> +			"=====>\n");
>  	if(NULL == Adapter)
>  	{
> -		BCM_DEBUG_PRINT(Adapter,DBG_TYPE_TX, TOKEN_COUNTS, DBG_LVL_ALL, "Adapter
> found NULL!\n");
> +		BCM_DEBUG_PRINT(Adapter, DBG_TYPE_TX, TOKEN_COUNTS,
> +				DBG_LVL_ALL, "Adapter found NULL!\n");
>  		return;
>  	}
>
> --
> 1.7.6.5
>
> _______________________________________________
> devel mailing list
> devel@xxxxxxxxxxxxxxxxxxxxxx
> http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
>


-- 
Regards,
Denis
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux