[PATCH 2/2] staging: bcm: Bcmchar.c - Checkpatch fixes

Dan Carpenter dan.carpenter at oracle.com
Sat Oct 27 18:23:26 UTC 2012


On Sat, Oct 27, 2012 at 04:24:41PM +0100, Ceri James wrote:
> @@ -1299,7 +1324,10 @@ cntrlEnd:
>  			return STATUS_FAILURE;
>  
>  		if (stNVMReadWrite.uiOffset > Adapter->uiNVMDSDSize - stNVMReadWrite.uiNumBytes) {
> -			/* BCM_DEBUG_PRINT(Adapter,DBG_TYPE_PRINTK, 0, 0,"Can't allow access beyond NVM Size: 0x%x 0x%x\n", stNVMReadWrite.uiOffset, stNVMReadWrite.uiNumBytes); */
> +			/* BCM_DEBUG_PRINT(
> +			 * 	Adapter, DBG_TYPE_PRINTK, 0, 0,
> +			 * 	"Can't allow access beyond NVM Size: 0x%x 0x%x\n",
> +			 * 	stNVMReadWrite.uiOffset, stNVMReadWrite.uiNumBytes); */

I wouldn't have commented except that Greg already did.  The closing
*/ should go on the next line.  Or if you want you could just delete
the this actually.

regards,
dan carpenter

>  			return STATUS_FAILURE;
>  		}
>  




More information about the devel mailing list