[PATCH] staging: sm750fb: ddk750_chip: use consistent spacing

2015-07-14 Thread Sunil Shahu
Remove all checkpatch error by using consistent spacing. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/sm750fb/ddk750_chip.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/staging/sm750fb

[PATCH v2] staging: sm750fb: ddk750_chip: use consistent spacing

2015-07-14 Thread Sunil Shahu
Remove all checkpatch error by using consistent spacing. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/sm750fb/ddk750_chip.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/staging/sm750fb

[PATCH] staging: rtl8188eu: core: rtw_mlme: remove space before ','

2015-07-07 Thread Sunil Shahu
Fix coding style error by removing spaces before ',' as suggested by checkpatch.pl script. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/rtl8188eu/core/rtw_mlme.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8188eu/core

Re: [PATCH] staging: rtl8712: rtl871x_security.c: remove unnecessary variable initialization

2015-07-02 Thread Sunil Shahu
On Fri, 2015-06-26 at 18:51 +0300, Dan Carpenter wrote: It's based on earlier patches on the list. Normally only Greg tests that patches apply and it will for him because he applies them as the order they arrive. But we should probably start putting notes on the patches which earlier

[PATCH] staging: rtl8712: rtl871x_security.c: remove unnecessary variable initialization

2015-06-26 Thread Sunil Shahu
Variable u32 c always re-initialize in for loop. Initialized value of u32 c is not used in function and is redundant, hence removed. Suggested-by: Dan Carpenter dan.carpen...@oracle.com Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/rtl8712/rtl871x_security.c | 2 +- 1 file

Re: [PATCH v2] drivers: staging: rtl8712: remove unnecessary else statement

2015-06-26 Thread Sunil Shahu
Hi Dan, On Mon, 2015-06-22 at 20:34 +0300, Dan Carpenter wrote: + u32 c = 0x1234; ^^ In another follow on patch you can get rid of this. It is nonsense and it isn't used. By follow on patch, did you mean that I should send v3 revision of my patch or send a

[PATCH v2] drivers: staging: rtl8712: remove unnecessary else statement

2015-06-22 Thread Sunil Shahu
else statement after if is unnecessary, hence removed. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/rtl8712/rtl871x_security.c | 35 ++ 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_security.c b

[PATCH] staging: wilc1000: host_interface: add spaces around '='

2015-06-22 Thread Sunil Shahu
Fix coding style error by placing spaces around '=' as suggested by checkpatch.pl script. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c

[PATCH v2] staging: media: lirc: fix coding style error

2015-06-20 Thread Sunil Shahu
Fix code indentation error by replacing tab in place of spaces. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/media/lirc/lirc_sasem.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/lirc/lirc_sasem.c b/drivers/staging/media/lirc

[PATCH] drivers: staging: rtl8712: remove unnecessary else statement

2015-06-19 Thread Sunil Shahu
else statement after if is unnecessary, hence removed. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/rtl8712/rtl871x_security.c | 35 ++ 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_security.c b

[PATCH] drivers: staging: rtl8712: remove unnecessory else statement

2015-06-19 Thread Sunil Shahu
else statement after if is unnecessory, hence removed. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/rtl8712/rtl871x_security.c | 35 ++ 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/drivers/staging/rtl8712/rtl871x_security.c b

[PATCH] staging: media: lirc: fix coding style error

2015-06-19 Thread Sunil Shahu
Fix code indentation error by replacing tab in place of spaces. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/media/lirc/lirc_sasem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/lirc/lirc_sasem.c b/drivers/staging/media/lirc

[PATCH] staging: media: lirc: fix coding style error

2015-06-19 Thread Sunil Shahu
Fix code indentation error by replacing tab in place of spaces. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/media/lirc/lirc_sasem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/lirc/lirc_sasem.c b/drivers/staging/media/lirc

[PATCH] drivers: staging: rtl8712: Resolved kernel coding style warnings

2015-06-18 Thread Sunil Shahu
1) else statement after if is unnecessory, hence removed. 2) Fixed line over 80 characters warning. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/rtl8712/rtl871x_security.c | 39 ++ 1 file changed, 18 insertions(+), 21 deletions(-) diff --git

[PATCH] staging: media: lirc: fix coding style error

2015-06-17 Thread Sunil Shahu
Fix code indentation error by replacing tab in place of spaces. Signed-off-by: Sunil Shahu shsh...@gmail.com --- drivers/staging/media/lirc/lirc_sasem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/lirc/lirc_sasem.c b/drivers/staging/media/lirc