Re: [PATCH] drivers:staging:gdm724x Fix Blank lines aren't necessary after an open brace '{'

2015-10-31 Thread Greg KH
On Fri, Oct 30, 2015 at 03:30:56PM +0100, Bogicevic Sasa wrote: > This fixes "Blank lines aren't necessary after an open brace '{'" > message from checkpatch.pl > > Signed-off-by: Bogicevic Sasa Hi, This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him a patch that has trigge

RE: [PATCH v5 5/7] PCI: irqdomain: Look up IRQ domain by fwnode_handle

2015-10-31 Thread Jake Oshins
> -Original Message- > From: Andy Shevchenko [mailto:andy.shevche...@gmail.com] > Sent: Friday, October 30, 2015 2:44 PM > To: Jake Oshins > Cc: Greg Kroah-Hartman ; KY Srinivasan > ; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; Robo Bot > ; Vitaly Kuznetso

[PATCH 6/6] Staging: lustre: mdc: Remove unused declarations

2015-10-31 Thread Shraddha Barke
The functions mdc_pack_req and mdc_getxattr_pack have been declared in header file but not used. Thus remove the declarations. Signed-off-by: Shraddha Barke --- drivers/staging/lustre/lustre/mdc/mdc_internal.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/mdc

[PATCH 4/6] Staging: lustre: lov: Remove unused function declarations

2015-10-31 Thread Shraddha Barke
These functions have been declared in header but not used anywhere. Thus drop the declarations. Signed-off-by: Shraddha Barke --- drivers/staging/lustre/lustre/lov/lov_internal.h | 8 1 file changed, 8 deletions(-) diff --git a/drivers/staging/lustre/lustre/lov/lov_internal.h b/driver

[PATCH 5/6] Staging: lustre: lov: Declare local functions as static

2015-10-31 Thread Shraddha Barke
Declare functions lov_set_add_req, lov_set_finished, lov_update_set, lov_check_and_wait_active and lov_update_statfs as static since they are used only in this particular file. Also remove corresponding declarations from header file. Signed-off-by: Shraddha Barke --- drivers/staging/lustre/lustr

[PATCH 3/6] Staging: lustre: lmv: Declare local functions as static

2015-10-31 Thread Shraddha Barke
Declare functions lmv_intent_open and lmv_intent_lookup as static since they are used only in this particular file. Also remove corresponding declarations from header file. Signed-off-by: Shraddha Barke --- drivers/staging/lustre/lustre/lmv/lmv_intent.c | 21 +++-- drivers/stag

[PATCH 2/6] Staging: lustre: lmv: Remove unused function declaration

2015-10-31 Thread Shraddha Barke
The function lmv_blocking_ast is declared in header file but not used. Hence remove the declaration. Signed-off-by: Shraddha Barke --- drivers/staging/lustre/lustre/lmv/lmv_internal.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/lmv/lmv_internal.h b/drivers

[PATCH 1/6] Staging: lustre: lclient: Remove wrapper functions

2015-10-31 Thread Shraddha Barke
The functions cl_isize_lock and cl_isize_unlock can be replaced with direct calls to ll_inode_size_lock and ll_inode_size_unlock. Thus drop the wrapper functions. Signed-off-by: Shraddha Barke --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 4 ++-- drivers/staging/lustre/lustre/llite

[PATCH 0/6] Staging: lustre: Code cleanup

2015-10-31 Thread Shraddha Barke
These patches remove wrapper functions, unused declarations and declare local functions as static in lustre. Shraddha Barke (6): Staging: lustre: lclient: Remove wrapper functions Staging: lustre: lmv: Remove unused function declaration Staging: lustre: lmv: Declare local functions as static

Re: [PATCH] drivers:staging:gdm724x Fix comparison to NULL could be written like

2015-10-31 Thread Bogicevic Sasa
On 10/31, Sudip Mukherjee wrote: On Fri, Oct 30, 2015 at 03:54:03PM +0100, Bogicevic Sasa wrote: This fixes all messages from checkpatch.pl about comparison to NULL could be written according to coding style preferences Signed-off-by: Bogicevic Sasa --- For all your next patches, either they

Re: [PATCH] drivers:staging:gdm724x Fix comparison to NULL could be written like

2015-10-31 Thread Sudip Mukherjee
On Fri, Oct 30, 2015 at 03:54:03PM +0100, Bogicevic Sasa wrote: > This fixes all messages from checkpatch.pl about comparison to NULL > could be written according to coding style preferences > > Signed-off-by: Bogicevic Sasa > --- For all your next patches, either they have already deen done or

Re: [PATCH] drivers:staging:gdm724x Fix spaces preffered around that ...

2015-10-31 Thread Sudip Mukherjee
On Fri, Oct 30, 2015 at 03:49:16PM +0100, Bogicevic Sasa wrote: > This fixes all messages from checkpatch.pl about spaces preffered around > that + > > Signed-off-by: Bogicevic Sasa > --- :( ba7f55b705b9 ("staging: gdm724x: add spaces around binary operators") regards sudip ___

Re: [PATCH] drivers:staging:gdm724x Fix please dont use multiple blank lines

2015-10-31 Thread Sudip Mukherjee
On Fri, Oct 30, 2015 at 03:36:37PM +0100, Bogicevic Sasa wrote: > This fixes the errors about the use of multiple blank lines from > checkpatch.pl > > Signed-off-by: Bogicevic Sasa > --- This has already been done. a487db8b6704 ("staging: gdm724x: remove multiple blank lines") regards sudip ___

Re: [PATCH] drivers:staging:gdm724x Fix Blank lines aren't necessary after an open brace '{'

2015-10-31 Thread Sudip Mukherjee
On Fri, Oct 30, 2015 at 03:30:56PM +0100, Bogicevic Sasa wrote: > This fixes "Blank lines aren't necessary after an open brace '{'" > message from checkpatch.pl > > Signed-off-by: Bogicevic Sasa > --- You are still using some other tree. This will not apply because of some other change. regards