Re: [PATCH] ssb: main.c: This patch removes unnecessary return statement using spatch tool

2017-01-07 Thread kbuild test robot
Hi Rahul, [auto build test WARNING on linus/master] [also build test WARNING on v4.10-rc2 next-20170106] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH] ssb: main.c: This patch removes unnecessary return statement using spatch tool

2017-01-07 Thread kbuild test robot
Hi Rahul, [auto build test ERROR on linus/master] [also build test ERROR on v4.10-rc2 next-20170106] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH] ssb: main.c: This patch removes unnecessary return statement using spatch tool

2017-01-06 Thread Julian Calaby
Hi Rahul, On Sat, Jan 7, 2017 at 2:20 AM, Rahul Krishnan wrote: > This patch removes unnecessary return statement using spatch. This doesn't describe the patch. > Signed-off-by: Rahul Krishnan > --- > drivers/ssb/main.c | 6 ++ > 1

Re: [PATCH] ssb: main.c: This patch removes unnecessary return statement using spatch tool

2017-01-06 Thread Michael Büsch
> > @@ -1272,9 +1272,7 @@ u32 ssb_admatch_size(u32 adm) > > default: > > SSB_WARN_ON(1); > > } > > - size = (1 << (size + 1)); > > - > > - return size; > > + return (1 << (size + 1)); > > } > > EXPORT_SYMBOL(ssb_admatch_size); I'm all for

Re: [PATCH] ssb: main.c: This patch removes unnecessary return statement using spatch tool

2017-01-06 Thread Rafał Miłecki
On 6 January 2017 at 23:39, Rafał Miłecki wrote: > On 6 January 2017 at 16:20, Rahul Krishnan wrote: >> This patch removes unnecessary return statement using spatch. >> Signed-off-by: Rahul Krishnan Also an extra line

Re: [PATCH] ssb: main.c: This patch removes unnecessary return statement using spatch tool

2017-01-06 Thread Rafał Miłecki
Hi Rahul, On 6 January 2017 at 16:20, Rahul Krishnan wrote: > This patch removes unnecessary return statement using spatch. > Signed-off-by: Rahul Krishnan Please work on simplifying & making topic more accurate. You can drop "main.c:" and

[PATCH] ssb: main.c: This patch removes unnecessary return statement using spatch tool

2017-01-06 Thread Rahul Krishnan
This patch removes unnecessary return statement using spatch. Signed-off-by: Rahul Krishnan --- drivers/ssb/main.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/ssb/main.c b/drivers/ssb/main.c index d1a7507..ae3b7fe 100644 ---