Re: [PATCH 1/1] drivers/staging: Fixed sparse error directive in argument list

2014-10-11 Thread Greg KH
On Sat, Oct 11, 2014 at 08:13:42PM +0100, Filipe Gonçalves wrote: This patch fixes a sparse warning on layout.c (ptlrpc) that was caused by having preprocessor directives in the arguments to a macro. Signed-off-by: Filipe Gonçalves fil...@codinghighway.com ---

Re: [PATCH 1/1] drivers/staging: Fixed sparse error directive in argument list

2014-10-11 Thread Drokin, Oleg
Hello! No, it's not the way to test the kernel version, it's the way to test internal lustre version. Either way maintaining compatibility with Lustre 1.8 and 2.0 servers should not be important anymore, so it's fine to drop this check indeed. Bye, Oleg On Oct 11, 2014, at 5:06

Re: [PATCH 1/1] drivers/staging: Fixed sparse error directive in argument list

2014-10-11 Thread Filipe Gonçalves
Hi, Oops.. sorry for the mess then. I am still trying to get to know the codebase. I submitted a new patch where I removed the check. Thanks for your patience :) Best, Filipe On Sat, Oct 11, 2014 at 10:49 PM, Drokin, Oleg oleg.dro...@intel.com wrote: Hello! No, it's not the way to test