Quentin LAMBERT
Articles de cet auteur
Mar 08 2014
Staging:netlogic: Correct double assignment in xlr_net.c
Author: Iulia Manda <iulia.manda21@gmail.com> This patch removes an unnecessary assignment of variable val. I have used a coccinelle semantic patch(da.cocci) in order to find this issue. Signed-off-by: Iulia Manda Reviewed-by: Sarah Sharp Signed-off-by: Greg Kroah-Hartman — drivers/staging/netlogic/xlr_net.c | 2 — 1 file changed, 2 deletions(-) diff –git a/drivers/staging/netlogic/xlr_net.c b/drivers/staging/netlogic/xlr_net.c index fd6050e..c83e337 100644 — a/drivers/staging/netlogic/xlr_net.c …
Mar 04 2014
gpio-tz1090: Replace commas with semi-colons
Author: James Hogan <james.hogan@imgtec.com> Replace commas with semicolons between irqchip callback initialisation statements in tz1090_gpio_bank_probe. The commas appear to be a subtle remnant of when the irqchips were statically initialised. Thanks to Thomas Gleixner for spotting it while whipping up a coccinelle script. Reported-by: Thomas Gleixner Signed-off-by: James Hogan Cc: Alexandre Courbot Cc: linux-gpio@vger.kernel.org Cc: …
Mar 03 2014
asoc: soc-core: fix coccinelle warnings
Author: Nenghua Cao <nhcao@marvell.com> sound/soc/soc-core.c:2708:6-13: WARNING: Assignment of bool to 0/1 sound/soc/soc-core.c:2726:3-10: WARNING: Assignment of bool to 0/1 More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: Nenghua Cao Signed-off-by: Mark Brown — sound/soc/soc-core.c | 4 ++– 1 file changed, 2 insertions(+), 2 deletions(-) diff –git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index fe1df50..3477525 100644 — a/sound/soc/soc-core.c …
Mar 03 2014
ASoC: core: fix coccinelle warnings
Author: Nenghua Cao <nhcao@marvell.com> sound/soc/soc-core.c:2708:6-13: WARNING: Assignment of bool to 0/1 sound/soc/soc-core.c:2726:3-10: WARNING: Assignment of bool to 0/1 More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: Nenghua Cao Signed-off-by: Mark Brown — sound/soc/soc-core.c | 4 ++– 1 file changed, 2 insertions(+), 2 deletions(-) diff –git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index 393ff06..45a1faf 100644 — a/sound/soc/soc-core.c …
Feb 21 2014
regmap: fix coccinelle warnings
Author: Nenghua Cao <nhcao@marvell.com> /drivers/base/regmap/regmap.c:717:6-33: WARNING: Comparison to bool. More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: Nenghua Cao Signed-off-by: Mark Brown — drivers/base/regmap/regmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c index 9a426c0..92d9b79 100644 — a/drivers/base/regmap/regmap.c +++ b/drivers/base/regmap/regmap.c @@ -718,7 +718,7 @@ skip_format_initialization: new->window_start = …
Feb 20 2014
bnx2x: Remove hidden flow control goto from BNX2X_ALLOC macros
Author: Joe Perches <joe@perches.com> BNX2X_ALLOC macros use « goto alloc_mem_err » so these labels appear unused in some functions. Expand these macros in-place via coccinelle and some typing. Update the macros to use statement expressions and remove the BNX2X_ALLOC macro. This adds some > 80 char lines. $ cat bnx2x_pci_alloc.cocci @@ expression e1; expression e2; expression e3; …
Feb 18 2014
rtc: isl12057: use PTR_ERR_OR_ZERO to fix coccinelle warnings
Author: Fengguang Wu <fengguang.wu@intel.com> drivers/rtc/rtc-isl12057.c:278:1-3: WARNING: PTR_RET can be used Use PTR_ERR_OR_ZERO rather than if(IS_ERR(…)) + PTR_ERR Generated by: coccinelle/api/ptr_ret.cocci CC: Arnaud Ebalard CC: Jason Cooper Signed-off-by: Fengguang Wu Signed-off-by: Jason Cooper — drivers/rtc/rtc-isl12057.c | 5 +—- 1 file changed, 1 insertion(+), 4 deletions(-) diff –git a/drivers/rtc/rtc-isl12057.c b/drivers/rtc/rtc-isl12057.c index 7854a65..835b425 100644 — a/drivers/rtc/rtc-isl12057.c +++ …
Feb 15 2014
caif: delete unnecessary field initialization
Author: Julia Lawall <Julia.Lawall@lip6.fr> On success, the function netdev_alloc_skb initializes the dev field of its result to its first argument, so this doesn’t have to be done in the calling context. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression skb,privn,e; @@ skb = netdev_alloc_skb(privn,…); … when strict ( -skb->dev …
Feb 15 2014
staging: r8712u: delete unnecessary field initialization
Author: Julia Lawall <Julia.Lawall@lip6.fr> On success, the function netdev_alloc_skb initializes the dev field of its result to its first argument, so this doesn’t have to be done in the calling context. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression skb,privn,e; @@ skb = netdev_alloc_skb(privn,…); … when strict ( -skb->dev …
Feb 14 2014
staging: fpgaboot: fix coccinelle warnings
Author: Fengguang Wu <fengguang.wu@intel.com> drivers/staging/gs_fpgaboot/gs_fpgaboot.c:191:2-3: Unneeded semicolon Removes unneeded semicolon. Generated by: coccinelle/misc/semicolon.cocci CC: Insop Song Signed-off-by: Fengguang Wu Signed-off-by: Greg Kroah-Hartman — drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c b/drivers/staging/gs_fpgaboot/gs_fpgaboot.c index 31a6741..89bc84d 100644 — a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c +++ b/drivers/staging/gs_fpgaboot/gs_fpgaboot.c @@ -188,7 +188,7 @@ static int gs_read_image(struct fpgaimage *fimage) …