On Tue, Feb 03, 2015 at 08:54:45PM +0530, Mohammad Jamal wrote:
> This patch adds a static keyword to lprocfs_lmv_init_vars and
> lprocfs_lmv_module_vars to suppress the sparse warnings about
> static declaration
have you build tested your patch?

After your patch:
drivers/staging/lustre/lustre/lmv/lproc_lmv.c:218:31: warning: 
‘lmv_proc_target_fops’ defined but not used [-Wunused-variable]
drivers/staging/lustre/lustre/lmv/lproc_lmv.c:226:13: warning: 
‘lprocfs_lmv_init_vars’ defined but not used [-Wunused-function]

WARNING: "lprocfs_lmv_init_vars" [drivers/staging/lustre/lustre/lmv/lmv.ko] 
undefined!
WARNING: "lmv_proc_target_fops" [drivers/staging/lustre/lustre/lmv/lmv.ko] 
undefined!

regards
sudip

> 
> Signed-off-by: Mohammad Jamal <md.jamalmohiud...@gmail.com>
> ---
<snip>
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to