[PATCH] sound: open brace should be on the previous line

From: Naveen Kumar Parna Resolved open brace { should be on the previous line checkpatch.pl error. While addressing this error, also corrected the affected code for below mentioned checkpatch errors. ERROR: spaces required around that '<' (ctx:VxV) ERROR: spaces required around that '==' (ctx:Vx

[PATCH] bsr: "foo * bar" should be "foo *bar"

From: Naveen Kumar Parna Fixed the checkpatch error. Used "foo *bar" instead of "foo * bar" Signed-off-by: Naveen Kumar Parna --- drivers/char/bsr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/bsr.c b/drivers/char/bsr.c index a6cef548e01e..d16ba62d03a0 1006

[PATCH] char: misc: Move EXPORT_SYMBOL immediately next to the functions/varibles

From: Naveen Kumar Parna According to checkpatch: EXPORT_SYMBOL(foo); should immediately follow its function/variable. This patch fixes the following checkpatch.pl issues in drivers/char/misc.c: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Naveen K

[PATCH] char: misc: Move EXPORT_SYMBOL immediately next to the functions/varibles

From: Naveen Kumar Parna According to checkpatch: EXPORT_SYMBOL(foo); should immediately follow its function/variable. This patch fixes the following checkpatch.pl issues in drivers/char/misc.c: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Naveen K

[PATCH v3 1/1] bsr: do not use assignment in if condition

From: Naveen Kumar Parna checkpatch.pl does not like assignment in if condition Signed-off-by: Naveen Kumar Parna --- Changes in v3: The first patch has an extra space in if statement, so fixed it in v2 but forgot add what changed from the previous version. In v3 added the complete change hist

[PATCH v2 1/1] bsr: do not use assignment in if condition

From: Naveen Kumar Parna checkpatch.pl does not like assignment in if condition Signed-off-by: Naveen Kumar Parna --- drivers/char/bsr.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/char/bsr.c b/drivers/char/bsr.c index d16ba62d03a0..935d4b300340 100644 --- a/d

[PATCH 3/3] char: misc: Move EXPORT_SYMBOL immediately next to the functions/varibles

From: Naveen Kumar Parna According to checkpatch: EXPORT_SYMBOL(foo); should immediately follow its function/variable. This patch fixes the following checkpatch.pl issues in drivers/char/misc.c: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Naveen K

[PATCH 2/2] bsr: "foo * bar" should be "foo *bar"

From: Naveen Kumar Parna Fixed the checkpatch error. Used "foo *bar" instead of "foo * bar" Signed-off-by: Naveen Kumar Parna --- drivers/char/bsr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/bsr.c b/drivers/char/bsr.c index 2b00748b83d2..35d456716969 1006

[PATCH] bsr: do not use assignment in if condition

From: Naveen Kumar Parna checkpatch.pl does not like assignment in if condition Signed-off-by: Naveen Kumar Parna --- drivers/char/bsr.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/char/bsr.c b/drivers/char/bsr.c index a6cef548e01e..2b00748b83d2 100644 --- a/d

[PATCH 2/2] mbcs: add .owner to mbcs struct file_operations

From: Naveen Kumar Parna Without ".owner = THIS_MODULE" it is possible to crash the kernel by unloading the mbcs module while someone works with the file. Fix this by initializing the ‘struct file_operations' ->owner with THIS_MODULE. Signed-off-by: Naveen Kumar Parna --- drivers/char/mbcs.c |

[PATCH] efirtc: remove unnecessary code efi_rtc_open & efi_rtc_close

From: Naveen Kumar Parna There is no advantage to keep 'struct file_operations.open & .close' API's. So removed the unnecessary code efi_rtc_open & efi_rtc_close. Signed-off-by: Naveen Kumar Parna --- drivers/char/efirtc.c | 23 --- 1 file changed, 23 deletions(-) diff --g