Catégorie : Linux

Bluetooth: dtl1_cs: Fixed coding style

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 …

Lire la suite

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) …

Lire la suite

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: …

Lire la suite

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 …

Lire la suite

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 …

Lire la suite

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(…) { …

Lire la suite

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(-)   …

Lire la suite

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(-)   …

Lire la suite

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(-)   …

Lire la suite

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 …

Lire la suite