Re: [PATCH 1/4] staging: ozwpan: fix missing blank line after declaration

2014-09-07 Thread Tobias Klauser
Your From: line does not contain a real name and does not match your Signed-off-by line, please check your e-mail client settings. On 2014-09-07 at 20:21:41 +0200, anicoara wrote: Changelog text is missing. Please add a short notice, describing why this change is done. > Signed-off-by: Adrian N

Re: [PATCH 2/4] staging: ozwpan: fix redundant else after break or return

2014-09-07 Thread Tobias Klauser
Your From: line does not contain a real name and does not match your Signed-off-by line, please check your e-mail client settings. On 2014-09-07 at 20:24:03 +0200, anicoara wrote: > Signed-off-by: Adrian Nicoara No changelog text? Please add a short notice, describing why this change is done.

Re: [PATCH 3/4] staging: ozwpan: fix redundant return in void function

2014-09-07 Thread Tobias Klauser
Your From: line does not contain a real name and does not match your Signed-off-by line, please check your e-mail client settings. On 2014-09-07 at 20:25:35 +0200, anicoara wrote: > Signed-off-by: Adrian Nicoara No changelog text? Please add a short notice, describing why this change is done.

Re: [PATCH 4/4] staging: ozwpan: use kmalloc_array over kmalloc with multiply

2014-09-07 Thread Tobias Klauser
Your From: line does not contain a real name and does not match your Signed-off-by line, please check your e-mail client settings. On 2014-09-07 at 20:28:25 +0200, anicoara wrote: No changelog text? Please add a short notice, describing why this change is done. > Signed-off-by: Adrian Nicoara

RE: [PATCH V2 1/1] Drivers: hv: vmbus: Enable interrupt driven flow control

2014-09-07 Thread KY Srinivasan
> -Original Message- > From: Amos Kong [mailto:kongjian...@gmail.com] > Sent: Sunday, September 7, 2014 4:44 AM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; open list; de...@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; Jason Wang > Subject: Re: [PATCH V2 1/1] Dri

[PATCH 4/4] staging: ozwpan: use kmalloc_array over kmalloc with multiply

2014-09-07 Thread anicoara
Signed-off-by: Adrian Nicoara --- Patch submitted as part of the Eudyptula challenge. drivers/staging/ozwpan/ozhcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozwpan/ozhcd.c index ba2168f..4cfe56e 100644 --- a/drivers/sta

[PATCH 3/4] staging: ozwpan: fix redundant return in void function

2014-09-07 Thread anicoara
Signed-off-by: Adrian Nicoara --- Patch submitted as part of the Eudyptula challenge. drivers/staging/ozwpan/ozproto.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/ozwpan/ozproto.c b/drivers/staging/ozwpan/ozproto.c index cae0e6f..3d3a3a8 100644 --- a/drivers/staging/ozwpan

[PATCH 2/4] staging: ozwpan: fix redundant else after break or return

2014-09-07 Thread anicoara
Signed-off-by: Adrian Nicoara --- Patch submitted as part of the Eudyptula challenge. drivers/staging/ozwpan/ozhcd.c | 8 +++- drivers/staging/ozwpan/ozpd.c | 10 -- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozw

[PATCH 1/4] staging: ozwpan: fix missing blank line after declaration

2014-09-07 Thread anicoara
Signed-off-by: Adrian Nicoara --- Patch submitted as part of the Eudyptula challenge. drivers/staging/ozwpan/ozcdev.c| 5 + drivers/staging/ozwpan/ozeltbuf.c | 1 + drivers/staging/ozwpan/ozpd.c | 6 ++ drivers/staging/ozwpan/ozproto.c | 4 drivers/staging/ozwpan/ozusbsv

[PATCH 7/7] staging: lustre: obdclass: expand the GOTO macro

2014-09-07 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // Signed-off-by: Julia Law

[PATCH 3/7] staging: lustre: lov: expand the GOTO macro

2014-09-07 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // Signed-off-by: Julia Law

[PATCH 6/7] staging: lustre: obdclass: expand the GOTO macro + break

2014-09-07 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl,rc,f; constant c; @@ - GOTO(lbl,\(rc\|rc->f\|c\)); - break; + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); - break; + rc; + goto lbl; // Signed-of

[PATCH 5/7] staging: lustre: obdclass: expand double GOTO macros

2014-09-07 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl; @@ if (...) GOTO(lbl,...); +else GOTO(lbl,...); @@ identifier lbl; expression e,e1,e2; @@ if (e) - GOTO(lbl,e1); -else GOTO(lbl,e2); + e1; +else e2; +goto lbl;

[PATCH 4/7] staging: lustre: lvfs: expand the GOTO macro

2014-09-07 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // Signed-off-by: Julia Law

[PATCH 2/7] staging: lustre: libcfs: expand the GOTO macro

2014-09-07 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // Signed-off-by: Julia Law

[PATCH 1/7] staging: lustre: ptlrpc: expand the GOTO macro

2014-09-07 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // Signed-off-by: Julia Law

[PATCH] rtl8192u: remove typedef

2014-09-07 Thread Martin Kepplinger
remove a typedef that is not even really used. Signed-off-by: Martin Kepplinger --- builds in next-20140905. drivers/staging/rtl8192u/r8192U_core.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u/r81

[PATCH] staging: netlogic: fix checkpatch.pl "LINE_SPACING" issuses

2014-09-07 Thread SeeChen Ng
Follow the checkpatch.pl "LINE_SPACING" indication: 1. Insert a blank line after function declaration. 2. Remove multiple blank lines. Signed-off-by: SeeChen Ng --- drivers/staging/netlogic/xlr_net.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/netlogic

Re: [PATCH V2 1/1] Drivers: hv: vmbus: Enable interrupt driven flow control

2014-09-07 Thread Amos Kong
On Sat, Sep 6, 2014 at 8:29 AM, K. Y. Srinivasan wrote: > > In win8 we have a feature that allows for interrupt driven flow management > for host/guest communication. For instance, if the host were blocked because > there was no space available in the ringbuffer, the host could request that > the

[PATCH 26/26] staging: rtl8188eu: Remove HalPhyRf_8188e.c

2014-09-07 Thread navin patidar
Move functions from HalPhyRf_8188e.c to phy.c . Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/Makefile |1 - drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 1209 drivers/staging/rtl8188eu/hal/phy.c| 1187 +++

[PATCH 25/26] staging: rtl8188eu: Remove HalPhyRf_8188e.h

2014-09-07 Thread navin patidar
Move macros and function declarations from HalPhyRf_8188e.h to phy.h . Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/include/HalPhyRf_8188e.h | 41 drivers/staging/rtl8188eu/include/odm_precomp.h|1 - drivers/staging/rtl8188eu/include/phy.h

[PATCH 20/26] staging: rtl8188eu: Rework function phy_LCCalibrate_8188E()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 42 ++-- 1 file changed, 24 insertions(+), 18 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c b/drivers/staging/r

[PATCH 21/26] staging: rtl8188eu: Rework function PHY_LCCalibrate_8188E()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 127 +--- drivers/staging/rtl8188eu/hal/usb_halinit.c|8 +- drivers/staging/rtl8188eu/incl

[PATCH 23/26] staging: rtl8188eu: HalPhyRf_8188e.c: Remove unused functions

2014-09-07 Thread navin patidar
Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 35 1 file changed, 35 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c b/drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c index e2c9b54..cec0ee7 100644 --- a/drive

[PATCH 24/26] staging: rtl8188eu: HalPhyRf_8188e.h :Remove unused function declaration

2014-09-07 Thread navin patidar
Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/include/HalPhyRf_8188e.h |3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/HalPhyRf_8188e.h b/drivers/staging/rtl8188eu/include/HalPhyRf_8188e.h index 6f5a66d..3c3b898 100644 --- a/drivers/staging

[PATCH 15/26] staging: rtl8188eu: Rework function _PHY_MACSettingCalibration()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages and local variables. Declare it as a static function. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 19 +-- drivers/staging/rtl8188eu/include/HalPhy

[PATCH 18/26] staging: rtl8188eu: Rework function phy_SimularityCompare_8188E()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 29 1 file changed, 9 insertions(+), 20 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal

[PATCH 22/26] staging: rtl8188eu: Rework function PHY_LCCalibrate_8188E()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 14 +- drivers/staging/rtl8188eu/hal/usb_halinit.c|4 ++-- drivers/staging/rtl8188eu/include/

[PATCH 16/26] staging: rtl8188eu: Rework function _PHY_PathAStandBy()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages and local variables. Declare it as a static function. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 11 ++- drivers/staging/rtl8188eu/include/HalPhyRf_8188e

[PATCH 17/26] staging: rtl8188eu: Rework function _PHY_PIModeSwitch()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages and local variables. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rt

[PATCH 19/26] staging: rtl8188eu: Rework function phy_IQCalibrate_8188E()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 173 +--- 1 file changed, 95 insertions(+), 78 deletions(-) diff --git a/drivers/staging/rtl8188eu/ha

[PATCH 12/26] staging: rtl8188eu: Rework function reload_adda_reg()

2014-09-07 Thread navin patidar
Rename CamelCase local variables. Remove unnecessary debugging messages and local variables. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/HalPhyRf_818

[PATCH 14/26] staging: rtl8188eu: Rework function _PHY_PathADDAOn()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages and local variables. Declare it as a static function. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 28 +++- drivers/staging/rtl8188eu/include/HalPh

[PATCH 09/26] staging: rtl8188eu: Rework function pathb_fill_iqk()

2014-09-07 Thread navin patidar
Rename CamelCase local variables. Remove unnecessary debugging messages and local variables. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 46 1 file changed, 22 insertions(+), 24 deletions(-) diff --git a/drivers/staging/rtl8188eu/

[PATCH 08/26] staging: rtl8188eu: Rework function patha_fill_iqk()

2014-09-07 Thread navin patidar
Rename CamelCase local variables. Remove unnecessary debugging messages and local variables. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 53 ++-- 1 file changed, 23 insertions(+), 30 deletions(-) diff --git a/drivers/staging/rtl8188eu/

[PATCH 13/26] staging: rtl8188eu: Rework function _PHY_ReloadMACRegisters()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages and local variables. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/drivers/staging/

[PATCH 10/26] staging: rtl8188eu: Rework function _PHY_SaveADDARegisters()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages and local variables. Declare it as a static function. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 16 +++- drivers/staging/rtl8188eu/include/HalPhyRf_

[PATCH 11/26] staging: rtl8188eu: Rework function _PHY_SaveMACRegisters()

2014-09-07 Thread navin patidar
Rename CamelCase local variables and function name. Remove unnecessary debugging messages and local variables. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/drivers/staging/

[PATCH 07/26] staging: rtl8188eu: Rework function phy_PathB_IQK_8188E()

2014-09-07 Thread navin patidar
Rename CamelCase function name and local variables. Remove unnecessary debugging messages. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 29 +--- 1 file changed, 6 insertions(+), 23 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal

[PATCH 05/26] staging: rtl8188eu: Rework function phy_PathA_RxIQK()

2014-09-07 Thread navin patidar
Rename CamelCase function name and variables. Remove unnecessary debugging messages. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 78 1 file changed, 27 insertions(+), 51 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/HalP

[PATCH 04/26] staging: rtl8188eu: Rework odm_TXPowerTrackingCallback_ThermalMeter_8188E()

2014-09-07 Thread navin patidar
Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 325 drivers/staging/rtl8188eu/hal/odm.c|2 +- drivers/staging/rtl8188eu/include/HalPhyRf_8188e.h |3 +- 3 files changed, 122 insertions(+), 208 deletions(-) diff -

[PATCH 02/26] staging: rtl8188eu: Rework function ODM_TxPwrTrackAdjust88E()

2014-09-07 Thread navin patidar
Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 59 +++- drivers/staging/rtl8188eu/hal/rf.c |6 +- drivers/staging/rtl8188eu/include/HalPhyRf_8188e.h | 12 ++-- 3 files changed, 31 insertions(+), 46 deletions(-) diff -

[PATCH 06/26] staging: rtl8188eu: Rework function phy_PathA_IQK_8188E()

2014-09-07 Thread navin patidar
Rename CamelCase function name and local variables. Remove unnecessary debugging messages and local variables. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 37 +++- 1 file changed, 11 insertions(+), 26 deletions(-) diff --git a/drivers/

[PATCH 03/26] staging: rtl8188eu: Rename function odm_TxPwrTrackSetPwr88E()

2014-09-07 Thread navin patidar
Rename CamelCase function name. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c b/drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.

[PATCH 01/26] staging: rtl8188eu: Rename function ODM_GetRightChnlPlaceforIQK()

2014-09-07 Thread navin patidar
Rename CamelCase function name. Signed-off-by: navin patidar --- drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c b/drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c index 0e

[PATCH v2] staging/lustre: annotate lock/unlock in lov_stripe_lock / lov_stripe_unlock

2014-09-07 Thread Mostyn Bramley-Moore
Annotate the lock/unlock pair in lov_stripe_lock/lov_stripe_unlock to avoid sparse warning about a context imbalance. Part of the eudyptula challenge: http://eudyptula-challenge.org/ Signed-off-by: Mostyn Bramley-Moore --- drivers/staging/lustre/lustre/lov/lov_obd.c | 2 ++ 1 file changed, 2 ins