Category: Linux

staging: wilc1000: Modification in code to use ARRAY_SIZE macro

Author: Chaitanya Dhere <cvijaydh@visteon.com> In this patch, ARRAY_SIZE() macro is used to determine the size. This change was detected with the help of coccinelle tool. Signed-off-by: Chaitanya Dhere Signed-off-by: Greg Kroah-Hartman — drivers/staging/wilc1000/linux_wlan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)   diff –git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c index b033eb8..592b8ae 100644 — a/drivers/staging/wilc1000/linux_wlan.c +++ b/drivers/staging/wilc1000/linux_wlan.c …

Continue reading

staging: rtl8192u: Patch to modify if, else conditions

Author: Chaitanya Dhere <cvijaydh@visteon.com> In this patch, the if, else conditions are modified to remove the unnecessary equality checks. This change was detected with help of coccinelle tool. Signed-off-by: Chaitanya Dhere Signed-off-by: Greg Kroah-Hartman — drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-)   diff –git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c index d2e8b12..1742e58 100644 — …

Continue reading

regulator: wm831x: Pass the IRQF_ONESHOT flag

Author: Fabio Estevam <fabio.estevam@freescale.com> Since commit 1c6c69525b40 (“genirq: Reject bogus threaded irq requests”) threaded IRQs without a primary handler need to be requested with IRQF_ONESHOT, otherwise the request will fail. So pass the IRQF_ONESHOT flag in this case. The semantic patch that makes this change is available in scripts/coccinelle/misc/irqf_oneshot.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Mark Brown …

Continue reading

[media] radio-si470x-i2c: Pass the IRQF_ONESHOT flag

Author: Fabio Estevam <fabio.estevam@freescale.com> Since commit 1c6c69525b40 (“genirq: Reject bogus threaded irq requests”) threaded IRQs without a primary handler need to be requested with IRQF_ONESHOT, otherwise the request will fail. So pass the IRQF_ONESHOT flag in this case. The semantic patch that makes this change is available in scripts/coccinelle/misc/irqf_oneshot.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Hans Verkuil …

Continue reading

pinctrl: samsung: Fix the pointer in PTR_ERR()

Author: Fabio Estevam <fabio.estevam@freescale.com> PTR_ERR should access the value just tested by IS_ERR The semantic patch that makes this change is available in scripts/coccinelle/tests/odd_ptr_err.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Linus Walleij — drivers/pinctrl/samsung/pinctrl-samsung.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)   diff –git a/drivers/pinctrl/samsung/pinctrl-samsung.c b/drivers/pinctrl/samsung/pinctrl-samsung.c index ed165ba..0fd20c5 100644 — a/drivers/pinctrl/samsung/pinctrl-samsung.c +++ b/drivers/pinctrl/samsung/pinctrl-samsung.c @@ …

Continue reading

pinctrl: Remove .owner field

Author: Fabio Estevam <fabio.estevam@freescale.com> platform_driver does not need to set the owner field, as it will be populated by the driver core, so just remove it. The semantic patch that makes this change is available in scripts/coccinelle/api/platform_no_drv_owner.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Linus Walleij — drivers/pinctrl/freescale/pinctrl-imx7d.c | 1 – drivers/pinctrl/mediatek/pinctrl-mt6397.c | 1 – drivers/pinctrl/mediatek/pinctrl-mt8127.c | 1 …

Continue reading

irqchip/mips-gic: Use access helper irq_data_get_affinity_mask()

Author: Jiang Liu <jiang.liu@linux.intel.com> Use access helper irq_data_get_affinity_mask() to hide implementation details of struct irq_desc. [ tglx: Verified with coccinelle ] Signed-off-by: Jiang Liu Cc: Konrad Rzeszutek Wilk Cc: Tony Luck Cc: Bjorn Helgaas Cc: Benjamin Herrenschmidt Cc: Randy Dunlap Cc: Yinghai Lu Cc: Borislav Petkov Cc: Jason Cooper Link: http://lkml.kernel.org/r/1433145945-789-30-git-send-email-jiang.liu@linux.intel.com Signed-off-by: Thomas Gleixner — …

Continue reading

xen-netfront: Use setup_timer

Author: Vaishali Thakkar <vthakkar1994@gmail.com> Use the timer API function setup_timer instead of structure field assignments to initialize a timer. A simplified version of the Coccinelle semantic patch that performs this transformation is as follows: @change@ expression e, func, da; @@ -init_timer (&e); +setup_timer (&e, func, da); -e.data = da; -e.function = func; Signed-off-by: Vaishali Thakkar …

Continue reading

ethernet/intel: Use setup_timer

Author: Vaishali Thakkar <vthakkar1994@gmail.com> Use the timer API function setup_timer instead of structure field assignments to initialize a timer. A simplified version of the Coccinelle semantic patch that performs this transformation is as follows: @change@ expression e1, e2, e3, e4, a, b; @@ -init_timer(&e1); +setup_timer(&e1, a, b); … when != a = e2 when != …

Continue reading

net: dl2k: Use setup_timer

Author: Vaishali Thakkar <vthakkar1994@gmail.com> Use the timer API function setup_timer instead of structure field assignments to initialize a timer. A simplified version of the Coccinelle semantic patch that performs this transformation is as follows: @change@ expression e1, e2, e3, e4, a, b; @@ -init_timer(&e1); +setup_timer(&e1, a, b); … when != a = e2 when != …

Continue reading