Re: [PATCH] staging: wlan-ng: remove unused debug macro

On Wed, Jun 17, 2015 at 09:40:42PM -0700, gre...@linuxfoundation.org wrote: > On Tue, Jun 16, 2015 at 05:09:35AM +, Gujulan Elango, Hari Prasath (H.) > wrote: > > On Mon, Jun 15, 2015 at 09:55:15PM -0700, gre...@linuxfoundation.org wrote: > > > On Mon, Jun 15, 201

Re: [PATCH] staging: wlan-ng: remove unused debug macro

On Wed, Jun 17, 2015 at 09:40:42PM -0700, gre...@linuxfoundation.org wrote: > On Tue, Jun 16, 2015 at 05:09:35AM +, Gujulan Elango, Hari Prasath (H.) > wrote: > > On Mon, Jun 15, 2015 at 09:55:15PM -0700, gre...@linuxfoundation.org wrote: > > > On Mon, Jun 15, 201

[PATCH] staging: wlan-ng: Remove ununsed debug print & associated macro

From: Hari Prasath Gujulan Elango This patch removed a undefined macro and debug prints associated with it. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wlan-ng/p80211wep.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/staging/wlan-ng/p80211wep.c

[PATCH] staging: lustre: lnet: selftest: use min_t in place of min

From: Hari Prasath Gujulan Elango This patch silences the checkpatch warning 'min() should probably be min_t' Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/lustre/lnet/selftest/rpc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/stag

[PATCH] staging: sm750fb: convert pr_err to pr_info

From: Hari Prasath Gujulan Elango This patch modifies few debug prints from pr_err() to pr_info() as they fall under that category. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/sm750fb/sm750.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers

[PATCH] staging: dgnc: free memory allocated

From: Hari Prasath Gujulan Elango The memory allocated in dgnc_tty_register() for two objects is not freed anywhere.This patch addresses this by freeing the memory in dgnc_tty_uninit. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/dgnc/dgnc_tty.c | 4 1 file changed, 4

[PATCH] staging: dgnc: fix typo in comments secion

From: Hari Prasath Gujulan Elango This patch fixes a typo in the comments section as warned by checkpatch.pl Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/dgnc/dgnc_driver.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.h

[PATCH] staging: ste_rmi4: remove redundant __func__ from debug prints

From: Hari Prasath Gujulan Elango This patch removes the redundant __func__ from dynamic debug prints as the pr_xxx set of functions can be dynamically controlled to include function name as well. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c

[PATCH 1/2] staging: sm750fb: remove redundant __func__ in debug statement

From: Hari Prasath Gujulan Elango This patch removes the redundant __func__ from dynamic debug prints as the pr_xxx set of functions can be dynamically controlled to include function name as well Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/sm750fb/sm750_accel.c | 2 +- 1

[PATCH 2/2] staging: sm750fb: fix typo in debug statement

From: Hari Prasath Gujulan Elango This patch fixes a typo in the debug statement Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/sm750fb/sm750_accel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/sm750_accel.c b/drivers/staging

[PATCH 01/10] staging: wilc1000: remove ununsed variable and associated line

From: Hari Prasath Gujulan Elango This patch removes a unused variable 'u16RespLen' that is assigned a value that is never used. The line that does the assignment is also removed. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/coreconfigurator.c | 4 --

[PATCH 02/10] staging: wilc1000: remove redundant initialization of variable

From: Hari Prasath Gujulan Elango This patch removes the redundant initialization of the variable 'st' as it is reassigned a new value before its being used anywhere else. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/coreconfigurator.c | 2 +- 1 file

[PATCH 03/10] staging: wilc1000: remove ununsed if..else.. code blocks

From: Hari Prasath Gujulan Elango This patch removes ununsed if..else... code blocks. Its actually some dead code. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/coreconfigurator.c | 22 -- 1 file changed, 22 deletions(-) diff --git a/drivers

[PATCH 04/10] staging: wilc1000: remove ununsed variable & corresponding lines

From: Hari Prasath Gujulan Elango This patch removes a couple of ununsed variable.The lines in which these variables are assigned are also removed as they are not necessary. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/linux_mon.c | 4 1 file changed, 4

[PATCH 06/10] staging: wilc1000: remove redundant assignment of variable

From: Hari Prasath Gujulan Elango This patch the removes the redundant assignement of the variable ret as its being overwritren before being used anywhere. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/wilc_wlan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 05/10] staging: wilc1000: remove unused variable

From: Hari Prasath Gujulan Elango This patch removes a unused variable timeout and the associated code. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/linux_wlan.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers

[PATCH 07/10] staging: wilc1000: remove ununsed variable

From: Hari Prasath Gujulan Elango This patch removes ununsed variable 'ret'. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/wilc_wlan.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drive

[PATCH 08/10] staging: wilc1000: remove ununsed variable & associated code

From: Hari Prasath Gujulan Elango This patch removes the ununsed variable 'priv' at multiple instances and all its associated code where its assigned a value. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 12 1 file c

[PATCH 09/10] staging: wilc1000: remove unused variable msg_len

From: Hari Prasath Gujulan Elango This patch removes a ununsed variable msg_len and its associated code. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/wilc_wlan_cfg.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan_cfg.c b

[PATCH 10/10] staging: wilc1000: use BIT macro

From: Hari Prasath Gujulan Elango This patch addresses the checkpatch warning advising the usage of the BIT macro for Bit shift operation. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/wilc_wlan_if.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH] staging: sm750fb: remove unused variables

From: Hari Prasath Gujulan Elango This patch removes unused variables as reported by cppcheck Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/sm750fb/ddk750_chip.c | 3 +-- drivers/staging/sm750fb/ddk750_mode.c | 3 +-- drivers/staging/sm750fb/ddk750_sii164.c | 3 +-- 3

Re: [PATCH 07/10] staging: wilc1000: remove ununsed variable

On Mon, Jun 22, 2015 at 06:12:11PM +0530, Sudip Mukherjee wrote: > On Mon, Jun 22, 2015 at 07:05:04AM +, Gujulan Elango, Hari Prasath (H.) > wrote: > > From: Hari Prasath Gujulan Elango > > > > This patch removes ununsed variable 'ret'. > ret or len?

[PATCH v2 07/10] staging: wilc1000: remove ununsed variable

From: Hari Prasath Gujulan Elango This patch removes ununsed variable 'len' Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/wilc_wlan.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drive

Re: [PATCH] staging: sm750fb: convert pr_err to pr_info

On Monday 22 June 2015 08:46 PM, Dan Carpenter wrote: > On Thu, Jun 18, 2015 at 12:56:54PM +, Gujulan Elango, Hari Prasath (H.) > wrote: >> From: Hari Prasath Gujulan Elango >> >> This patch modifies few debug prints from pr_err() to pr_info() as they >

Re: [PATCH] staging: dgnc: free memory allocated

adding him.Greg has already added this patch.I will ensure not to miss him next time. > On Thu, Jun 18, 2015 at 01:27:28PM +0000, Gujulan Elango, Hari Prasath (H.) > wrote: >> From: Hari Prasath Gujulan Elango >> >> The memory allocated in dgnc_tty_register() for two objects i

Re: [PATCH 2/2] staging: sm750fb: fix typo in debug statement

On Tuesday 23 June 2015 02:07 PM, Dan Carpenter wrote: > On Mon, Jun 22, 2015 at 04:48:01AM +, Gujulan Elango, Hari Prasath (H.) > wrote: >> From: Hari Prasath Gujulan Elango >> >> This patch fixes a typo in the debug statement >> >> Signed

Re: [PATCH] staging: sm750fb: remove unused variables

On Tuesday 23 June 2015 02:16 PM, Dan Carpenter wrote: > On Mon, Jun 22, 2015 at 07:26:25AM +, Gujulan Elango, Hari Prasath (H.) > wrote: >> @@ -122,7 +121,7 @@ void setChipClock(unsigned int frequency) >> * Sometime, the chip cannot set up the exact clock re

Re: [PATCH v2 07/10] staging: wilc1000: remove ununsed variable

On Tue, Jun 23, 2015 at 11:49:24AM +0300, Dan Carpenter wrote: > On Mon, Jun 22, 2015 at 01:13:58PM +, Gujulan Elango, Hari Prasath (H.) > wrote: > > From: Hari Prasath Gujulan Elango > > > > This patch removes ununsed variable 'len' > > > &

Re: [PATCH] staging: sm750fb: convert pr_err to pr_info

On Mon, Jun 22, 2015 at 06:15:06PM +0300, Dan Carpenter wrote: > On Thu, Jun 18, 2015 at 12:56:54PM +, Gujulan Elango, Hari Prasath (H.) > wrote: > > From: Hari Prasath Gujulan Elango > > > > This patch modifies few debug prints from pr_err() to pr_info() as

[PATCHv2] staging: sm750fb: remove redundant __func__ in debug statement

From: Hari Prasath Gujulan Elango This patch removes the redundant __func__ from dynamic debug prints as the pr_xxx set of functions can be dynamically controlled to include function name.Also fix a typo in debug statement. Signed-off-by: Hari Prasath Gujulan Elango --- v2:Address Dan

[PATCH] staging: comedi: use BIT macro

From: Hari Prasath Gujulan Elango Prefer using the BIT macro for shifting operation Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/comedi/comedi.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging

[PATCH] staging: lustre: remove dead code

From: Hari Prasath Gujulan Elango This patch removes commented code. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/lustre/lustre/osc/osc_cache.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/lustre/lustre/osc/osc_cache.c b/drivers/staging/lustre

[PATCH] staging: xgifb: prefer using the BIT macro

From: Hari Prasath Gujulan Elango This patch uses the BIT macro for bit shift operation. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/xgifb/XGI_main_26.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/xgifb/XGI_main_26.c b/drivers

[PATCH] staging: wilc1000: simplify 'memset' of 2D array

From: Hari Prasath Gujulan Elango This patch simplifies the 'memset' done on a static 2D array. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/wilc1000/host_interface.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/stagin

<    1   2   3