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.jamalmohiuddin@xxxxxxxxx> > --- <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 devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel