[PATCH] drivers: staging: bcm: removed a developer debug message per the TODO list.

[PATCH] drivers: staging: bcm: removed a developer debug message per the TODO list.

[Chuong Ngo]

Removed a debug message as outlined in the TODO list.
---
 drivers/staging/bcm/Bcmchar.c |    2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/bcm/Bcmchar.c b/drivers/staging/bcm/Bcmchar.c
index f91bc1f..526594b 100644
--- a/drivers/staging/bcm/Bcmchar.c
+++ b/drivers/staging/bcm/Bcmchar.c
@@ -50,8 +50,6 @@ static int bcm_char_release(struct inode *inode, struct file *filp)
 	pTarang = (struct bcm_tarang_data *)filp->private_data;
 
 	if (pTarang == NULL) {
-		BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0,
-				"ptarang is null\n");
 		return 0;
 	}
 
-- 
1.7.10.4

Re: [PATCH] drivers: staging: bcm: removed a developer debug message per the TODO list.

[Dan Carpenter]

On Mon, Oct 21, 2013 at 02:57:24AM -0400, Chuong Ngo wrote:
> Removed a debug message as outlined in the TODO list.

The Signed-off-by line is missing.  Run checkpatch.pl over your patch.
Hopefully it should warn you to remove the curly braced because they
are not needed any more.

Otherwise the patch is ok.  Please fix the two issues and resend.

regards,
dan carpenter