Quentin LAMBERT
Author's posts
Oct 30 2014
staging: rtl8188eu: os_dep: Removed unnecessary return.
Author: Gulsah Kose <gulsah.1004@gmail.com> This patch fixes WARNING: void function return statements are not generally useful checkpatch.pl warning in usb_intf.c by using this coccinelle script @r@ identifier i; @@ void i(…) { … -return; } Signed-off-by: Gulsah Kose Reviewed-by: Daniel Baluta Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8188eu/os_dep/usb_intf.c | 2 — 1 file changed, 2 deletions(-) …
Oct 29 2014
staging: rtl8188eu: core: Removed unnecessary parenthesis.
Author: Gulsah Kose <gulsah.1004@gmail.com> This patch removes unnecessarry parenthesis in rtw_security.c by using this coccinelle script: @r1@ expression e1,e2; @@ if – ((e1 == e2)) + (e1 == e2) {…} Signed-off-by: Gulsah Kose Acked-by: Julia Lawall Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8188eu/core/rtw_security.c | 4 ++– 1 file changed, 2 insertions(+), 2 deletions(-) diff –git a/drivers/staging/rtl8188eu/core/rtw_security.c …
Oct 29 2014
usb: gadget: fix ptr_ret.cocci warnings
Author: Fengguang Wu <fengguang.wu@intel.com> drivers/usb/gadget/udc/r8a66597-udc.c:1849:1-3: WARNING: PTR_ERR_OR_ZERO can be used Use PTR_ERR_OR_ZERO rather than if(IS_ERR(…)) + PTR_ERR Generated by: scripts/coccinelle/api/ptr_ret.cocci Signed-off-by: Fengguang Wu Signed-off-by: Felipe Balbi — drivers/usb/gadget/udc/r8a66597-udc.c | 5 +—- 1 file changed, 1 insertion(+), 4 deletions(-) diff –git a/drivers/usb/gadget/udc/r8a66597-udc.c b/drivers/usb/gadget/udc/r8a66597-udc.c index b63a527..be31b57 100644 — a/drivers/usb/gadget/udc/r8a66597-udc.c +++ b/drivers/usb/gadget/udc/r8a66597-udc.c @@ -1845,10 +1845,7 @@ static …
Oct 29 2014
staging: rtl8188eu: core: Removed unnecessary return keyword.
Author: Gulsah Kose <gulsah.1004@gmail.com> This patch fixes WARNING: void function return statements are not generally useful checkpatch.pl warning in rtw_efuse.c by using this coccinelle script @r@ identifier i; @@ void i(…) { … -return; } Signed-off-by: Gulsah Kose Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8188eu/core/rtw_efuse.c | 1 – 1 file changed, 1 deletion(-) diff –git a/drivers/staging/rtl8188eu/core/rtw_efuse.c …
Oct 28 2014
staging: ft1000: Remove curly braces for single statement blocks
Author: Ebru Akagunduz <ebru.akagunduz@gmail.com> This patch removes curly braces for single statement blocks using following coccinelle script: @@ expression e1; @@ – if (e1) { + if (e1) return …; – } Signed-off-by: Ebru Akagunduz Acked-by: Julia Lawall Signed-off-by: Greg Kroah-Hartman — drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 12 ++++——– 1 file changed, 4 insertions(+), 8 deletions(-) diff …
Oct 26 2014
staging: rtl8723au: Remove duplicated argument to ||
Author: Roberta Dobrescu <roberta.dobrescu@gmail.com> This patch removes duplicated argument to ||, fixing the following warning detected using coccinelle tool: duplicated argument to && or ||. Signed-off-by: Roberta Dobrescu Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8723au/core/rtw_wlan_util.c | 1 – 1 file changed, 1 deletion(-) diff –git a/drivers/staging/rtl8723au/core/rtw_wlan_util.c b/drivers/staging/rtl8723au/core/rtw_wlan_util.c index bc63d6f..69d9e0f 100644 — a/drivers/staging/rtl8723au/core/rtw_wlan_util.c +++ b/drivers/staging/rtl8723au/core/rtw_wlan_util.c @@ -1322,7 …
Oct 26 2014
staging: rtl8723au: core: Remove uneeded return variable
Author: Roberta Dobrescu <roberta.dobrescu@gmail.com> This patch removes uneeded return variables, using only ‘return _SUCCESS’ instead. It fixes the following warning detected by coccinelle: Unneeded variable. It was done using the following semantic patch: @@ identifier ret; type T; expression e; @@ -T ret = e; … when != ret when strict -return ret; +return e; …
Oct 26 2014
staging: rtl8723au: os_dep: Remove uneeded return variable
Author: Roberta Dobrescu <roberta.dobrescu@gmail.com> This patch removes uneeded return variables, using only ‘return _SUCCESS’ instead. It fixes the following warning detected by coccinelle: Unneeded variable. It was done using the following semantic patch: @@ identifier ret; type T; expression e; @@ -T ret = e; … when != ret when strict -return ret; +return e; …
Oct 26 2014
staging: rtl8723au: hal: Remove uneeded return variable
Author: Roberta Dobrescu <roberta.dobrescu@gmail.com> This patch removes uneeded return variables, just using ‘return _SUCCESS’ or ‘return HCI_STATUS_SUCCESS’ instead. This fixes the following warning detected using coccinelle: Unneeded variable. It was done using the following semantic patch: @@ identifier ret; type T; expression e; @@ -T ret = e; … when != ret when strict -return …
Oct 26 2014
staging: rtl8723au: Remove uneeded semicolon
Author: Roberta Dobrescu <roberta.dobrescu@gmail.com> This patch fixes the following warning detected using coccinelle: Unneeded semicolon. Signed-off-by: Roberta Dobrescu Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8723au/core/rtw_ap.c | 2 +- drivers/staging/rtl8723au/core/rtw_recv.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff –git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging/rtl8723au/core/rtw_ap.c index d00255f..e394d12 100644 — a/drivers/staging/rtl8723au/core/rtw_ap.c +++ b/drivers/staging/rtl8723au/core/rtw_ap.c @@ -636,7 +636,7 @@ static void …