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

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

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

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

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

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

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

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

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

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 ++--

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

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

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

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

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

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

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

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 ---

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

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

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

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

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

> -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; >

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

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