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

2016-02-19 Thread Dan Carpenter
On Tue, Feb 16, 2016 at 10:39:05PM -0800, shalin mehta wrote: > Hello, > > Should I send this patch again due the spelling mistake in the patch > description? > Yes. And put what Oleg Drokin said about it worked because there were no users in the changelog. regards, dan carpenter

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 set and >> clear th

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

2016-02-15 Thread Drokin, Oleg
On Feb 14, 2016, at 10:37 PM, Shalin Mehta wrote: > 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(-)

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

2016-02-15 Thread Drokin, Oleg
On Feb 15, 2016, at 9:28 PM, Greg KH wrote: > On Sun, Feb 14, 2016 at 07:37:02PM -0800, Shalin Mehta wrote: >> The parentehsis are fixed in the macro for the ldlm lock to set and >> clear the flags. > > "fixed" in what way? Did you test-build this patch? Also, you > mispelled a word hereā€¦ The

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

2016-02-15 Thread Greg KH
On Sun, Feb 14, 2016 at 07:37:02PM -0800, Shalin Mehta wrote: > The parentehsis are fixed in the macro for the ldlm lock to set and > clear the flags. "fixed" in what way? Did you test-build this patch? Also, you mispelled a word here... thanks, greg k-h

[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/lustre_dlm_f