Articles de cet auteur
Jul 26 2013
MIPS:Netlogic: Remove redundant value in operation.
Author: Alexandru Juncu <alexj@rosedu.org> Removed parameters checked twice in logical OR operation. Suggested by coccinelle and manually verified. Signed-off-by: Alexandru Juncu Cc: jchandra@broadcom.com Cc: linux-mips@linux-mips.org Cc: linux-kernel@vger.kernel.org Patchwork: https://patchwork.linux-mips.org/patch/5627/ Signed-off-by: Ralf Baechle — arch/mips/netlogic/xlp/usb-init.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff –git a/arch/mips/netlogic/xlp/usb-init.c b/arch/mips/netlogic/xlp/usb-init.c index ef3897e..d5378ef 100644 — a/arch/mips/netlogic/xlp/usb-init.c +++ …
Jul 19 2013
pcmcia: synclink_cs: replace sum of bitmasks with OR operation.
Author: Alexandru Juncu <alexj@rosedu.org> Suggested by coccinelle and manually verified. Signed-off-by: Alexandru Juncu Signed-off-by: Greg Kroah-Hartman — drivers/char/pcmcia/synclink_cs.c | 26 +++++++++++++————- 1 file changed, 13 insertions(+), 13 deletions(-) diff –git a/drivers/char/pcmcia/synclink_cs.c b/drivers/char/pcmcia/synclink_cs.c index 5c5cc00..d39cca6 100644 — a/drivers/char/pcmcia/synclink_cs.c +++ b/drivers/char/pcmcia/synclink_cs.c @@ -1182,14 +1182,14 @@ static irqreturn_t mgslpc_isr(int dummy, void *dev_id) } count++; – if …
Jul 18 2013
mISDN: replace sum of bitmasks with OR operation.
Author: Alexandru Juncu <alexj@rosedu.org> Suggested by coccinelle and manually verified. Signed-off-by: Alexandru Juncu Signed-off-by: David S. Miller — drivers/isdn/hardware/mISDN/hfcpci.c | 12 ++++++—— 1 file changed, 6 insertions(+), 6 deletions(-) diff –git a/drivers/isdn/hardware/mISDN/hfcpci.c b/drivers/isdn/hardware/mISDN/hfcpci.c index a7e4939..7f910c7 100644 — a/drivers/isdn/hardware/mISDN/hfcpci.c +++ b/drivers/isdn/hardware/mISDN/hfcpci.c @@ -1307,11 +1307,11 @@ mode_hfcpci(struct bchannel *bch, int bc, int protocol) } if (fifo2 …
Jul 15 2013
ALSA: hdspm – remove unneeded semicolon
Author: Fengguang Wu <fengguang.wu@intel.com> sound/pci/rme9652/hdspm.c:1110:2-3: Unneeded semicolon Generated by: coccinelle/misc/semicolon.cocci Reported-by: Fengguang Wu Signed-off-by: Takashi Iwai — sound/pci/rme9652/hdspm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/sound/pci/rme9652/hdspm.c b/sound/pci/rme9652/hdspm.c index a3a71ac..ec6335e 100644 — a/sound/pci/rme9652/hdspm.c +++ b/sound/pci/rme9652/hdspm.c @@ -1250,7 +1250,7 @@ static int hdspm_rate_multiplier(struct hdspm *hdspm, int rate) else if (hdspm->control_register & …
Jul 12 2013
Staging: silicom: Remove useless unneeded semicolons
Author: Vladimir <gg.kaspersky@gmail.com> Found using coccinelle tool. Signed-off-by: Vladimir Cernov Signed-off-by: Greg Kroah-Hartman — drivers/staging/silicom/bpctl_mod.c | 18 +++++++++——— 1 file changed, 9 insertions(+), 9 deletions(-) diff –git a/drivers/staging/silicom/bpctl_mod.c b/drivers/staging/silicom/bpctl_mod.c index 32bc530..495272d 100644 — a/drivers/staging/silicom/bpctl_mod.c +++ b/drivers/staging/silicom/bpctl_mod.c @@ -1809,7 +1809,7 @@ int cmnd_off(struct bpctl_dev *pbpctl_dev) else data_pulse(pbpctl_dev, CMND_OFF); ret = 0; – }; + } …
Jul 11 2013
bnx2x: fix tunneling CSUM calculation
Author: Dmitry Kravkov <dmitry@broadcom.com> Since commit c957d09ffda417f6c8e3d1f10e2b05228607d6d7 « bnx2x: Remove sparse and coccinelle warnings » driver provided wrong partial csum for HW in tunneing scenarios. Reported-by: Eric Dumazet CC: Alexander Duyck CC: Pravin Shelar CC: Cong Wang Signed-off-by: Dmitry Kravkov Tested-by: Eric Dumazet Signed-off-by: David S. Miller — drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 6 +++— 1 file changed, 3 insertions(+), …
Jul 05 2013
staging: lustre: use ERR_CAST() function
Author: Laurent Navet <laurent.navet@gmail.com> use ERR_CAST() function instead of ERR_PTR() and PTR_ERR() found using coccinelle and err_cast.cocci Signed-off-by: Laurent Navet Signed-off-by: Greg Kroah-Hartman — drivers/staging/lustre/lustre/llite/llite_nfs.c | 2 +- drivers/staging/lustre/lustre/llite/rw.c | 2 +- drivers/staging/lustre/lustre/obdclass/local_storage.c | 4 ++– 3 files changed, 4 insertions(+), 4 deletions(-) diff –git a/drivers/staging/lustre/lustre/llite/llite_nfs.c b/drivers/staging/lustre/lustre/llite/llite_nfs.c index b830004..32362ec 100644 — a/drivers/staging/lustre/lustre/llite/llite_nfs.c +++ b/drivers/staging/lustre/lustre/llite/llite_nfs.c …
Jul 04 2013
drivers/parport/share.c: use kzalloc
Author: Alexandru Gheorghiu <gheorghiuandru@gmail.com> Replaced calls to kmalloc and memset with kzalloc. Patch found using coccinelle. Signed-off-by: Alexandru Gheorghiu Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds — drivers/parport/share.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff –git a/drivers/parport/share.c b/drivers/parport/share.c index a848e02..6a83ee1 100644 — a/drivers/parport/share.c +++ b/drivers/parport/share.c @@ -282,14 +282,13 @@ struct parport …
Jun 25 2013
bnx2x: Remove sparse and coccinelle warnings
Author: Yuval Mintz <yuvalmin@broadcom.com> This patch solves several sparse issues as well as an unneeded semicolon found via coccinelle. Signed-off-by: Yuval Mintz Signed-off-by: Dmitry Kravkov Signed-off-by: Ariel Elior Signed-off-by: Eilon Greenstein Signed-off-by: David S. Miller — drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 9 ++++++— drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c | 8 +++++— drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 2 +- 3 files changed, 12 insertions(+), 7 deletions(-) …
Jun 18 2013
HID: wiimote: fix coccinelle warnings
Author: Jiri Kosina <jkosina@suse.cz> drivers/hid/hid-wiimote-modules.c:569:2-3: Unneeded semicolon Generated by: coccinelle/misc/semicolon.cocci Reported-by: Fengguang Wu Signed-off-by: Jiri Kosina — drivers/hid/hid-wiimote-modules.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/drivers/hid/hid-wiimote-modules.c b/drivers/hid/hid-wiimote-modules.c index 8229d0a..68f67f0 100644 — a/drivers/hid/hid-wiimote-modules.c +++ b/drivers/hid/hid-wiimote-modules.c @@ -566,7 +566,7 @@ static void wiimod_ir_in_ir(struct wiimote_data *wdata, const __u8 *ir, break; default: return; …