Author: Prasanna Karthik <mkarthi3@visteon.com> Removed semicolon at end of switch statement,error reported by Coccinelle Signed-off-by: Prasanna Karthik Signed-off-by: Marcel Holtmann — drivers/bluetooth/dtl1_cs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/drivers/bluetooth/dtl1_cs.c b/drivers/bluetooth/dtl1_cs.c index 09c130d..84135c5 100644 — a/drivers/bluetooth/dtl1_cs.c +++ b/drivers/bluetooth/dtl1_cs.c @@ -406,7 +406,7 @@ static int dtl1_hci_send_frame(struct hci_dev *hdev, struct sk_buff …
Category: Linux
Jul 05 2015
Bluetooth: bfusb: Coding style fix reported by coccinelle
Author: Prasanna Karthik <mkarthi3@visteon.com> Removed semicolon at the end of switch case statement Signed-off-by: Prasanna Karthik Signed-off-by: Marcel Holtmann — drivers/bluetooth/bfusb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/drivers/bluetooth/bfusb.c b/drivers/bluetooth/bfusb.c index fcfb72e..a5c4d05 100644 — a/drivers/bluetooth/bfusb.c +++ b/drivers/bluetooth/bfusb.c @@ -492,7 +492,7 @@ static int bfusb_send_frame(struct hci_dev *hdev, struct sk_buff *skb) …
Jul 04 2015
sched, sysctl: Delete an unnecessary check before unregister_sysctl_table()
Author: Markus Elfring <elfring@users.sourceforge.net> The unregister_sysctl_table() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Peter Zijlstra (Intel) Cc: Linus Torvalds Cc: Mike Galbraith Cc: Peter Zijlstra Cc: Thomas Gleixner Link: …
Jul 04 2015
perf probe: Delete an unnecessary check before the function call “strfilter__delete”
Author: Markus Elfring <elfring@users.sourceforge.net> The strfilter__delete() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Acked-by: Masami Hiramatsu Cc: Julia Lawall Cc: Peter Zijlstra Cc: kernel-janitors@vger.kernel.org Link: http://lkml.kernel.org/r/5597751A.5000506@users.sourceforge.net Signed-off-by: Arnaldo Carvalho de …
Jul 03 2015
hwmon: (w83627ehf) Use swap() in w82627ehf_swap_tempreg()
Author: Fabian Frederick <fabf@skynet.be> Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick Signed-off-by: Jean Delvare — drivers/hwmon/w83627ehf.c | 26 +++++——————— 1 file changed, 5 insertions(+), 21 deletions(-) diff –git a/drivers/hwmon/w83627ehf.c b/drivers/hwmon/w83627ehf.c index b10353b..697007a 100644 — a/drivers/hwmon/w83627ehf.c +++ b/drivers/hwmon/w83627ehf.c @@ -1937,27 +1937,11 @@ static inline void w83627ehf_init_device(struct w83627ehf_data …
Jul 03 2015
ARM: pxa: Use module_platform_driver
Author: Vaishali Thakkar <vthakkar1994@gmail.com> Use module_platform_driver for drivers whose init and exit functions only register and unregister, respectively. A simplified version of the Coccinelle semantic patch that performs this transformation is as follows: @a@ identifier f, x; @@ -static f(…) { return platform_driver_register(&x); } @b depends on a@ identifier e, a.x; @@ -static e(…) { …
Jul 02 2015
netlink: Delete an unnecessary check before the function call “module_put”
Author: Markus Elfring <elfring@users.sourceforge.net> The module_put() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: David S. Miller — net/netlink/af_netlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) …
Jul 02 2015
net-RDS: Delete an unnecessary check before the function call “module_put”
Author: Markus Elfring <elfring@users.sourceforge.net> The module_put() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: David S. Miller — net/rds/transport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) …
Jul 02 2015
net-ipv6: Delete an unnecessary check before the function call “free_percpu”
Author: Markus Elfring <elfring@users.sourceforge.net> The free_percpu() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: David S. Miller — net/ipv6/route.c | 5 +—- 1 file changed, 1 insertion(+), 4 deletions(-) …
Jul 02 2015
ASoC: wm8996: Remove spurious IRQF_ONESHOT flag
Author: Axel Lin <axel.lin@ingics.com> This reverts ed043aebe6ec (“ASoC: wm8996: Pass the IRQF_ONESHOT flag”). The coccinelle warnings is false positive because the original code does set IRQF_ONESHOT by “trigger |= IRQF_ONESHOT;”. Signed-off-by: Axel Lin Acked-by: Charles Keepax Signed-off-by: Mark Brown — sound/soc/codecs/wm8996.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-) diff –git a/sound/soc/codecs/wm8996.c …