[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 '=='

[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

[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

[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

[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

[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 ---

[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

[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

[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 ---

[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