[PATCH] staging: lustre: ptlrpc: lproc_ptlrpc.c - fix dereferenceing user space buffer

drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c:652:33:got char const *buffer Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c b

[PATCH] staging: lustre: ldlm: ldlm_resource.c - fix dereferenceing user space buffer

char const *buffer Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_resource.c b/drivers/staging/lustre/lustre/ldlm/ldlm_resource.c index 91d028b

[PATCH] staging: lustre: ldlm: ldlm_resource.c - fix dereferenceing user space buffer

:got char const *buffer Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_resource.c b/drivers/staging/lustre/lustre/ldlm

[PATCH] staging: lustre: ptlrpc: lproc_ptlrpc.c - fix dereferenceing user space buffer

*from drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c:652:33:got char const *buffer Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/ptlrpc

[PATCH v3] staging: vt6655: ioctl.c - missing __user annotation

in argument 2 (different address spaces) drivers/staging/vt6655/ioctl.c:78:51:expected void const [noderef] asn:1>*from drivers/staging/vt6655/ioctl.c:78:51:got void *data Signed-off-by: Anil Belur --- v2: - replaced the 'void __user *' casts by adding '__user' annotation to the d

[PATCH v2 1/1] staging: vt6655: ioctl.c - missing __user annotation

From: Anil Belur v2: - private_ioctl() internally calls copy_{to,from}_user() and does not use '__user' which gives out several sparse warnings - this patch adds __user annotation to the data member of struct tagSCmdRequest as suggested by tklau...@distanz.ch - sparse warnings fixed

[PATCH v2 1/1] staging: vt6655: ioctl.c - missing __user annotation

From: Anil Belur ask...@gmail.com v2: - private_ioctl() internally calls copy_{to,from}_user() and does not use '__user' which gives out several sparse warnings - this patch adds __user annotation to the data member of struct tagSCmdRequest as suggested by tklau...@distanz.ch - sparse

[PATCH v3] staging: vt6655: ioctl.c - missing __user annotation

in argument 2 (different address spaces) drivers/staging/vt6655/ioctl.c:78:51:expected void const [noderef] asn:1*from drivers/staging/vt6655/ioctl.c:78:51:got void *data Signed-off-by: Anil Belur ask...@gmail.com --- v2: - replaced the 'void __user *' casts by adding '__user

[PATCH 1/1] drivers: staging: vt6655: ioctl.c - missing __user annotation

From: Anil Belur - private_ioctl() the internally calls copy_{to,from}_user() and does not use '__user' while refrencing user space pointers. - this patch passes __user annotation as a cast, when the pointer is being refernced. - this patch fixes the following sparse errors: drivers/staging

[PATCH 1/1] drivers: staging: vt6655: ioctl.c - missing __user annotation

From: Anil Belur ask...@gmail.com - private_ioctl() the internally calls copy_{to,from}_user() and does not use '__user' while refrencing user space pointers. - this patch passes __user annotation as a cast, when the pointer is being refernced. - this patch fixes the following sparse errors

[PATCH 1/1] drivers: staging: cxt1e1: linux.c - missing __user annotation

From: Anil Belur - Some of the functions internally call copy_{to,from}_user() but does not use '__user'. this patch fixes missing __user annotations. - this patch fixes the following sparse errors: drivers/staging/cxt1e1/linux.c:488:33: warning: incorrect type in argument 2 (different

[PATCH 1/1] drivers: staging: cxt1e1: linux.c - missing __user annotation

From: Anil Belur ask...@gmail.com - Some of the functions internally call copy_{to,from}_user() but does not use '__user'. this patch fixes missing __user annotations. - this patch fixes the following sparse errors: drivers/staging/cxt1e1/linux.c:488:33: warning: incorrect type in argument 2

[PATCH 1/2] staging: rtl8723au: core: rtw_ap.c - removed NULL pointer check before kfree()

From: Anil Belur - as kfree() internally check for NULL, additional check it not required. Signed-off-by: Anil Belur --- drivers/staging/rtl8723au/core/rtw_ap.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging

[PATCH 2/2] staging: rtl8723au: core: rtw_cmd.c - removed NULL pointer check before kfree()

From: Anil Belur - as kfree() internally checks for NULL, additional check it not required. Signed-off-by: Anil Belur --- drivers/staging/rtl8723au/core/rtw_cmd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_cmd.c b/drivers/staging

[PATCH 2/2] staging: nokia_h4p: nokia_core.c - removed IRQF_DISABLED macro

From: Anil Belur - this patch removes the IRQF_DISABLED macro, as this is deprecated/noop. Signed-off-by: Anil Belur --- drivers/staging/nokia_h4p/nokia_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/nokia_h4p/nokia_core.c b/drivers/staging

[PATCH 1/2] staging: media: bcm2048: radio-bcm2048.c - removed IRQF_DISABLED macro

From: Anil Belur - this patch removes IRQF_DISABLED macro, as this is deprecated/noop. Signed-off-by: Anil Belur --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers

[PATCH 1/2] staging: rtl8723au: core: rtw_ap.c - removed NULL pointer check before kfree()

From: Anil Belur ask...@gmail.com - as kfree() internally check for NULL, additional check it not required. Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/rtl8723au/core/rtw_ap.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au

[PATCH 2/2] staging: rtl8723au: core: rtw_cmd.c - removed NULL pointer check before kfree()

From: Anil Belur ask...@gmail.com - as kfree() internally checks for NULL, additional check it not required. Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/rtl8723au/core/rtw_cmd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au

[PATCH 1/2] staging: media: bcm2048: radio-bcm2048.c - removed IRQF_DISABLED macro

From: Anil Belur ask...@gmail.com - this patch removes IRQF_DISABLED macro, as this is deprecated/noop. Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media

[PATCH 2/2] staging: nokia_h4p: nokia_core.c - removed IRQF_DISABLED macro

From: Anil Belur ask...@gmail.com - this patch removes the IRQF_DISABLED macro, as this is deprecated/noop. Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/nokia_h4p/nokia_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/nokia_h4p

[PATCH v2 1/1] staging: lustre: libcfs: nidstrings.c - use ARRAY_SIZE macro

From: Anil Belur - this patch replaces sizeof(a)/sizeof(a[0]) with the ARRAY_SIZE macro to get the number of nidstrings (and netstrfns) - v2: also for netstrfns Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/libcfs/nidstrings.c | 5 ++--- 1 file changed, 2 insertions(+), 3

[PATCH v2 1/1] staging: lustre: libcfs: nidstrings.c - use ARRAY_SIZE macro

From: Anil Belur ask...@gmail.com - this patch replaces sizeof(a)/sizeof(a[0]) with the ARRAY_SIZE macro to get the number of nidstrings (and netstrfns) - v2: also for netstrfns Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/libcfs/nidstrings.c | 5 ++--- 1 file

[PATCH 1/1] staging: speakup: main.c - use time_after()

From: Anil Belur - this patch fixes jiffies comparision with a safer function to prevent any overflows Signed-off-by: Anil Belur --- drivers/staging/speakup/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup

[PATCH 1/1] staging: speakup: main.c - use time_after()

From: Anil Belur ask...@gmail.com - this patch fixes jiffies comparision with a safer function to prevent any overflows Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/speakup/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup

[PATCH 1/1] staging: lustre: libcfs: nidstrings.c - use ARRAY_SIZE macro

From: Anil Belur - this patch replaces sizeof(a)/sizeof(a[0]) with the ARRAY_SIZE macro to get the number of nidstrings Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/libcfs/nidstrings.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging

[PATCH 1/1] staging: speakup: speakup_dectlk.c - use time_before_eq()

From: Anil Belur - this replaces jiffies comparision with safer function using time_after_eq() Signed-off-by: Anil Belur --- drivers/staging/speakup/speakup_dectlk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/speakup_dectlk.c b/drivers

[PATCH 1/1] staging: speakup: speakup_dectlk.c - use time_before_eq()

From: Anil Belur ask...@gmail.com - this replaces jiffies comparision with safer function using time_after_eq() Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/speakup/speakup_dectlk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup

[PATCH 1/1] staging: lustre: libcfs: nidstrings.c - use ARRAY_SIZE macro

From: Anil Belur ask...@gmail.com - this patch replaces sizeof(a)/sizeof(a[0]) with the ARRAY_SIZE macro to get the number of nidstrings Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/libcfs/nidstrings.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions

[PATCH 1/1] staging: media: msi3101: sdr-msi3101.c - replace with time_before_eq()

From: Anil Belur - this fix replaces jiffies interval comparision with safer function to avoid any overflow and wrap around ? Signed-off-by: Anil Belur --- drivers/staging/media/msi3101/sdr-msi3101.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git

[PATCH v2 1/1] staging: lustre: lnet: api-ni.c: fixed sparse warnings

From: Anil Belur - this fixes few sparse warnings for missing static in functions: drivers/staging/lustre/lnet/lnet/api-ni.c:63:6: warning: symbol 'lnet_get_routes' was not declared. Should it be static? drivers/staging/lustre/lnet/lnet/api-ni.c:69:6: warning: symbol 'lnet_get_networks

[PATCH v2 1/1] staging: lustre: lnet: api-ni.c: fixed sparse warnings

From: Anil Belur ask...@gmail.com - this fixes few sparse warnings for missing static in functions: drivers/staging/lustre/lnet/lnet/api-ni.c:63:6: warning: symbol 'lnet_get_routes' was not declared. Should it be static? drivers/staging/lustre/lnet/lnet/api-ni.c:69:6: warning: symbol

[PATCH 1/1] staging: media: msi3101: sdr-msi3101.c - replace with time_before_eq()

From: Anil Belur ask...@gmail.com - this fix replaces jiffies interval comparision with safer function to avoid any overflow and wrap around ? Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/media/msi3101/sdr-msi3101.c | 16 1 file changed, 12 insertions(+), 4

[PATCH 2/4] staging: lustre: llite: llite_capa.c - fixed warning to use recomended headers

From: Anil Belur - this fixes "WARNING: Use #include instead of " Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/llite/llite_capa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_capa.c b/drivers/stag

[PATCH 4/4] staging: lustre: llite: dir.c - fix for coding style issues

From: Anil Belur - this fixes some coding style issues "ERROR: "(foo*)" should be "(foo *)" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/llite/dir.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lust

[PATCH 3/4] staging: lustre: llite: dir.c - fixes indented labels in code

From: Anil Belur - this fixes "WARNING: labels should not be indented" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/llite/dir.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c b/drive

[PATCH 1/4] staging: lustre: llite: llite_capa.c - fixes checkpatch errors - unecessary init of static variables to 0

From: Anil Belur - this commit fixes some "ERROR: do not initialise statics to 0 or NULL" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/llite/llite_capa.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/ll

[PATCH 3/4] staging: lustre: llite: dir.c - fixes indented labels in code

From: Anil Belur ask...@gmail.com - this fixes WARNING: labels should not be indented Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/llite/dir.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite

[PATCH 2/4] staging: lustre: llite: llite_capa.c - fixed warning to use recomended headers linux/uaccess.h

From: Anil Belur ask...@gmail.com - this fixes WARNING: Use #include linux/uaccess.h instead of asm/uaccess.h Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH 4/4] staging: lustre: llite: dir.c - fix for coding style issues

From: Anil Belur ask...@gmail.com - this fixes some coding style issues ERROR: (foo*) should be (foo *) Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/llite/dir.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre

[PATCH 1/4] staging: lustre: llite: llite_capa.c - fixes checkpatch errors - unecessary init of static variables to 0

From: Anil Belur ask...@gmail.com - this commit fixes some ERROR: do not initialise statics to 0 or NULL Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging

[PATCH 1/1] staging: lustre: lustre: libcfs: fail.c - fix sparse warnings EXPORT_SYMBOL

From: Anil Belur - this fixes "WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/libcfs/fail.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lus

[PATCH 1/1] staging: lustre: lustre: libcfs: fail.c - fix sparse warnings EXPORT_SYMBOL

From: Anil Belur ask...@gmail.com - this fixes WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/libcfs/fail.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git

[PATCH 2/3] staging: lustre: lustre: ldlm: ldlm_lib.c - removed unnecessary braces

From: Anil Belur - this change fixes "WARNING: braces {} are not necessary for single statement blocks" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/l

[PATCH 1/3] staging: lustre: ldlm: ldlm_resource.c removed unecessary braces

From: Anil Belur - this fixed the "WARNING: braces {} are not necessary for single statement blocks" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/l

[PATCH 3/3] staging: lustre: ldlm: ldlm_lib.c removed unecessary space after function name

From: Anil Belur - this fixes "WARNING: space prohibited between function name and open parenthesis '(" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drive

[PATCH 1/1] staging: lustre: ldlm: ldlm_resource.c removed unecessary braces

From: Anil Belur - this fixed the "WARNING: braces {} are not necessary for single statement blocks" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/l

[PATCH 1/1] staging: lustre: ldlm: ldlm_resource.c replace obsolete function with kstrtoul()

From: Anil Belur - this change fixes the warning "WARNING: simple_strtoul is obsolete, use kstrtoul instead" - removed the unused var 'end' as this is no longer required for kstrtoul() Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 7 -

[PATCH 2/2] staging: lustre: lustre: fld: lproc_fld.c fixed warning

From: Anil Belur fixed warning "WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/fld/fld_request.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/fld/fld_re

[PATCH 1/2] staging: lustre: lustre: fld: lproc_fld.c fixed warning

From: Anil Belur fixed warning for line over 80 characters by moving the struct init onto a diff line. Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/fld/lproc_fld.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/fld

[PATCH 1/1] staging : lustre : lustre : fid : lproc_fid.c: fixed warning

From: Anil Belur fixed warning for line over 80 characters by moving the initilation onto a diff line. Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/fid/lproc_fid.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/fid

[PATCH 1/1] staging : lustre : lustre : fid : lproc_fid.c: fixed warning

From: Anil Belur ask...@gmail.com fixed warning for line over 80 characters by moving the initilation onto a diff line. Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/fid/lproc_fid.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH 2/2] staging: lustre: lustre: fld: lproc_fld.c fixed warning

From: Anil Belur ask...@gmail.com fixed warning WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/fld/fld_request.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre

[PATCH 1/2] staging: lustre: lustre: fld: lproc_fld.c fixed warning

From: Anil Belur ask...@gmail.com fixed warning for line over 80 characters by moving the struct init onto a diff line. Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/fld/lproc_fld.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers

[PATCH 1/1] staging: lustre: ldlm: ldlm_resource.c replace obsolete function with kstrtoul()

From: Anil Belur ask...@gmail.com - this change fixes the warning WARNING: simple_strtoul is obsolete, use kstrtoul instead - removed the unused var 'end' as this is no longer required for kstrtoul() Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/ldlm

[PATCH 1/1] staging: lustre: ldlm: ldlm_resource.c removed unecessary braces

From: Anil Belur ask...@gmail.com - this fixed the WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

[PATCH 2/3] staging: lustre: lustre: ldlm: ldlm_lib.c - removed unnecessary braces

From: Anil Belur ask...@gmail.com - this change fixes WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH 1/3] staging: lustre: ldlm: ldlm_resource.c removed unecessary braces

From: Anil Belur ask...@gmail.com - this fixed the WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

[PATCH 3/3] staging: lustre: ldlm: ldlm_lib.c removed unecessary space after function name

From: Anil Belur ask...@gmail.com - this fixes WARNING: space prohibited between function name and open parenthesis '( Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 24 1 file changed, 12 insertions(+), 12 deletions

[PATCH v7 5/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b/drivers/staging

[PATCH v7 6/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: ERROR: inline keyword should sit between storage class and type Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b

[PATCH v7 7/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: line over 80 characters, used a new variable 'size_index' to store the offset. Replaced "unsigned long" with "loff_t" type for 'cur_index'. Removed the parenthesis around the second part of the if statement following the '||' as its not r

[PATCH v7 4/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: Missing a blank line after declarations Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b/drivers/staging/lustre/lustre

[PATCH v7 8/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: suspect code indent for conditional statements (32, 32) Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b

[PATCH v7 1/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur Fixed "ERROR: need consistent spacing around '+' (ctx:WxV)" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/glimpse.c b/drive

[PATCH v7 2/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur fixed "WARNING: labels should not be indented" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/glimpse.c b/drivers/staging/lus

[PATCH v7 3/8] staging: lustre: lclient: lcommon_misc.c fixing coding style issues

From: Anil Belur fixed warning: * WARNING: min() should probably be min_t(__u32, desc.ld_tgt_count, LOV_MAX_STRIPE_COUNT) Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_misc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH v7] staging: lustre: lclient: fixing coding style issues

reworked patch 8/8 7/8 and 3/8, fixed changes recomemded by "andreas.dil...@intel.com" -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read

[PATCH v7 2/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur ask...@gmail.com fixed WARNING: labels should not be indented Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/glimpse.c b

[PATCH v7] staging: lustre: lclient: fixing coding style issues

reworked patch 8/8 7/8 and 3/8, fixed changes recomemded by andreas.dil...@intel.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the

[PATCH v7 3/8] staging: lustre: lclient: lcommon_misc.c fixing coding style issues

From: Anil Belur ask...@gmail.com fixed warning: * WARNING: min() should probably be min_t(__u32, desc.ld_tgt_count, LOV_MAX_STRIPE_COUNT) Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/lcommon_misc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v7 1/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur ask...@gmail.com Fixed ERROR: need consistent spacing around '+' (ctx:WxV) Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient

[PATCH v7 8/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur ask...@gmail.com fixed: WARNING: suspect code indent for conditional statements (32, 32) Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre

[PATCH v7 4/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur ask...@gmail.com fixed: WARNING: Missing a blank line after declarations Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c

[PATCH v7 7/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur ask...@gmail.com fixed: WARNING: line over 80 characters, used a new variable 'size_index' to store the offset. Replaced unsigned long with loff_t type for 'cur_index'. Removed the parenthesis around the second part of the if statement following the '||' as its not required

[PATCH v7 6/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur ask...@gmail.com fixed: ERROR: inline keyword should sit between storage class and type Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre

[PATCH v7 5/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur ask...@gmail.com fixed: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient

[PATCH v6 4/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: Missing a blank line after declarations Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b/drivers/staging/lustre/lustre

[PATCH v6 3/8] staging: lustre: lclient: lcommon_misc.c fixing coding style issues

From: Anil Belur fixed warning: * WARNING: min() should probably be min_t(__u32, desc.ld_tgt_count, LOV_MAX_STRIPE_COUNT) Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_misc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH v6 8/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: suspect code indent for conditional statements (32, 32) Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c

[PATCH v6 2/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur fixed "WARNING: labels should not be indented" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/glimpse.c b/drivers/staging/lus

[PATCH v6 7/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: line over 80 characters, used a new variable 'size_index' to store the offset. Replaceed "unsigned long" with "loff_t" type for 'cur_index' Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 5 +++-

[PATCH v6 6/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: ERROR: inline keyword should sit between storage class and type Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b

[PATCH v6 5/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b/drivers/staging

[PATCH v6 1/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur Fixed "ERROR: need consistent spacing around '+' (ctx:WxV)" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/glimpse.c b/drive

[PATCH v6] staging: lustre: lclient: fixing coding style issues

Revised PATCH 7/8 and updated the comment, as suggested by "oleg.dro...@intel.com" -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the

[PATCH v5 5/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b/drivers/staging

[PATCH v5] staging: lustre: lclient: fixing coding style issues

Reworked on PATCH 8/8 as per comments provided by "oleg.dro...@intel.com". This includes changes variable name to 'size_index' and type to loff_t instead of 'unsigned long' -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

[PATCH v5 3/8] staging: lustre: lclient: lcommon_misc.c fixing coding style issues

From: Anil Belur fixed warning: * WARNING: min() should probably be min_t(__u32, desc.ld_tgt_count, LOV_MAX_STRIPE_COUNT) Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_misc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH v5 2/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur fixed "WARNING: labels should not be indented" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/glimpse.c b/drivers/staging/lus

[PATCH v5 4/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: Missing a blank line after declarations Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b/drivers/staging/lustre/lustre

[PATCH v5 7/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: line over 80 characters, used a new variable 'size_index' to store the offset. Replace "unsigned long" with "u64" type for 'cur_index' Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 5 +++-- 1 file c

[PATCH v5 6/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: ERROR: inline keyword should sit between storage class and type Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b

[PATCH v5 8/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: suspect code indent for conditional statements (32, 32) Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c

[PATCH v5 1/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur Fixed "ERROR: need consistent spacing around '+' (ctx:WxV)" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/glimpse.c b/drive

[PATCH v4 5/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b/drivers/staging

[PATCH v4 8/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: suspect code indent for conditional statements (32, 32) Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c b

[PATCH v4 7/8] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

From: Anil Belur fixed: WARNING: line over 80 characters, used a new variable 'check' to store the offset. Replace "unsigned long" with "u64" type fir 'cur_index' Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 5 +++-- 1 file changed

[PATCH v4] staging: lustre: lclient: fixing coding style issues

Resending patches as per changes recommend by "oleg.dro...@intel.com" for 1/8 and 7/8. PATCH 1/8: Now have removed the '+' to make the code more straight forward. The patch v3 7/7 is now split into two separate patches as both are related diff warning or error. -- To unsubscribe from this list:

[PATCH v4 1/8] staging: lustre: lclient: glimpse.c fixing coding style issues

From: Anil Belur Fixed "ERROR: need consistent spacing around '+' (ctx:WxV)" Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/glimpse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/lclient/glimpse.c b/drive

[PATCH v4 3/8] staging: lustre: lclient: lcommon_misc.c fixing coding style issues

From: Anil Belur fixed warning: * WARNING: min() should probably be min_t(__u32, desc.ld_tgt_count, LOV_MAX_STRIPE_COUNT) Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_misc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

  1   2   >