Re: [lustre-devel] [PATCH] staging: lustre: Fixed the parenthesis

2016-02-16 Thread shalin mehta
Hello, Should I send this patch again due the spelling mistake in the patch description? Thanks, Shalin On Mon, Feb 15, 2016 at 6:51 PM, Drokin, Oleg wrote: > > On Feb 14, 2016, at 10:37 PM, Shalin Mehta wrote: > >> The parentehsis are fixed in the macro for the ldlm lock to s

[PATCH] staging: lustre: Fixed the parenthesis

2016-02-14 Thread Shalin Mehta
The parentehsis are fixed in the macro for the ldlm lock to set and clear the flags. Signed-off-by: Shalin Mehta --- drivers/staging/lustre/lustre/include/lustre_dlm_flags.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include

[PATCH] staging: lustre: Fix the warning messages about casting without __user macro

2014-12-09 Thread Shalin Mehta
From: Shalin Mehta This issue is showed up while compiling with sparse. The iov_base in struct iovec struct explicitly declares that the assigned value should be user space pointer with __user macro. Where as here, the __user macro isn't used while casting. Signed-off-by: Shalin

[PATCH] staging: speakup: str initialization replaced with NULL where it was initialized with int

2013-10-03 Thread Shalin Mehta
Fixed warnings in all of three files where the string was initilized with an integer instead of NULL Signed-off-by: Shalin Mehta --- drivers/staging/speakup/main.c | 2 +- drivers/staging/speakup/speakup_audptr.c | 2 +- drivers/staging/speakup/varhandlers.c| 2 +- 3 files

[PATCH] Staging: bcm: PHSModule.c: Matching the function definition with function declaration

2013-09-17 Thread Shalin Mehta
From: Shalin Mehta The function implementations of the PHSModule didn't match with the declaration. The functions are static in the declaration but in the implemntation they are non-static Signed-off-by: Shalin Mehta --- NOTE: Submitting the same patch again. Added NOTE at wrong locatio

[PATCH] Staging: bcm: PHSModule.c: Matching the function definition with function declaration

2013-09-17 Thread Shalin Mehta
From: Shalin Mehta NOTE: Sending it again. I got mail delivery failure due to typo in emai ID gre...@linuxfoundation.org The function implementations of the PHSModule didn't match with the declaration. The functions are static in the declaration but in the implemntation they are non-s

[PATCH] Staging: bcm: PHSModule.c: Matching the function definition with function declaration

2013-09-17 Thread Shalin Mehta
From: Shalin Mehta The function implementations of the PHSModule didn't match with the declaration. The functions are static in the declaration but in the implemntation they are non-static Signed-off-by: Shalin Mehta --- drivers/staging/bcm/PHSModule.c | 12 ++-- 1 file chang