arch/sparc/kernel/pci_sun4v.c: removes unnecessary semicolon

Author: Peter Senna Tschudin <peter.senna@gmail.com> removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin Signed-off-by: David S. Miller — arch/sparc/kernel/pci_sun4v.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)   diff –git a/arch/sparc/kernel/pci_sun4v.c b/arch/sparc/kernel/pci_sun4v.c index 7661e84..051b69c 100644 — a/arch/sparc/kernel/pci_sun4v.c +++ b/arch/sparc/kernel/pci_sun4v.c @@ -594,7 +594,7 @@ static int __devinit pci_sun4v_iommu_init(struct pci_pbm_info *pbm) printk(KERN_ERR …

Continue reading

drivers/isdn/gigaset/common.c: Remove useless kfree

Author: Peter Senna Tschudin <peter.senna@gmail.com> Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { … kfree@p2(x); … return …; } @unchanged exists@ position r.p1,r.p2; expression e

hwspinlock/core: move the dereference below the NULL test

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Ohad Ben-Cohen — drivers/hwspinlock/hwspinlock_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)   diff –git a/drivers/hwspinlock/hwspinlock_core.c b/drivers/hwspinlock/hwspinlock_core.c index 1201a15..db713c0 100644 — a/drivers/hwspinlock/hwspinlock_core.c +++ b/drivers/hwspinlock/hwspinlock_core.c @@ …

Continue reading

[IA64] kexec: Move the dereference below the NULL test

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Tony Luck — arch/ia64/kernel/machine_kexec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)   diff –git a/arch/ia64/kernel/machine_kexec.c b/arch/ia64/kernel/machine_kexec.c index 070e8ef..5151a64 100644 — a/arch/ia64/kernel/machine_kexec.c +++ b/arch/ia64/kernel/machine_kexec.c @@ …

Continue reading

ARM: davinci: dm644x evm: move pointer dereference below NULL check

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Pointer dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Sekhar Nori — arch/arm/mach-davinci/board-dm644x-evm.c | 5 +++– 1 file changed, 3 insertions(+), 2 deletions(-)   diff –git a/arch/arm/mach-davinci/board-dm644x-evm.c b/arch/arm/mach-davinci/board-dm644x-evm.c index f22572ce..f8a99ee 100644 — a/arch/arm/mach-davinci/board-dm644x-evm.c +++ b/arch/arm/mach-davinci/board-dm644x-evm.c @@ …

Continue reading

usb: otg: move the dereference below the NULL test

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Felipe Balbi — drivers/usb/otg/fsl_otg.c | 34 ++++++++++++++++++++++++———- 1 file changed, 24 insertions(+), 10 deletions(-)   diff –git a/drivers/usb/otg/fsl_otg.c b/drivers/usb/otg/fsl_otg.c index 23c798c..c19d1d7 100644 — a/drivers/usb/otg/fsl_otg.c +++ b/drivers/usb/otg/fsl_otg.c @@ …

Continue reading

watchdog: move the dereference below the NULL test

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Wim Van Sebroeck — drivers/watchdog/watchdog_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)   diff –git a/drivers/watchdog/watchdog_core.c b/drivers/watchdog/watchdog_core.c index 6aa46a9..3796434 100644 — a/drivers/watchdog/watchdog_core.c +++ b/drivers/watchdog/watchdog_core.c …

Continue reading

caif: move the dereference below the NULL test

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: David S. Miller — net/caif/cfsrvl.c | 5 +++– 1 file changed, 3 insertions(+), 2 deletions(-)   diff –git a/net/caif/cfsrvl.c b/net/caif/cfsrvl.c index dd485f6..ba217e9 100644 — a/net/caif/cfsrvl.c +++ b/net/caif/cfsrvl.c …

Continue reading

cciss: remove unneeded memset()

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> The memory return by kzalloc() or kmem_cache_zalloc() has already be set to zero, so remove useless memset(0). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Cc: Mike Miller Cc: Jens Axboe Cc: Stephen M. Cameron Signed-off-by: Andrew Morton Signed-off-by: Jens Axboe — drivers/block/cciss.c | …

Continue reading

NFC: Move the pn544_hci dereference below the NULL test

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Samuel Ortiz — drivers/nfc/pn544_hci.c | 9 ++++++— 1 file changed, 6 insertions(+), 3 deletions(-)   diff –git a/drivers/nfc/pn544_hci.c b/drivers/nfc/pn544_hci.c index e681da2..7b0c217 100644 — a/drivers/nfc/pn544_hci.c +++ b/drivers/nfc/pn544_hci.c @@ …

Continue reading