Author: Julia Lawall <Julia.Lawall@lip6.fr> The mxr_layer_ops structures are never modified, so declare them as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Sylwester Nawrocki Signed-off-by: Mauro Carvalho Chehab — drivers/media/platform/s5p-tv/mixer.h | 2 +- drivers/media/platform/s5p-tv/mixer_grp_layer.c | 2 +- drivers/media/platform/s5p-tv/mixer_video.c | 2 +- drivers/media/platform/s5p-tv/mixer_vp_layer.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) …
Category: Linux
Nov 15 2015
iio: adc: xilinx: constify iio_buffer_setup_ops structure
Author: Julia Lawall <Julia.Lawall@lip6.fr> The iio_buffer_setup_ops structures are never modified, so declare this one as const, like the others. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Acked-by: Lars-Peter Clausen Signed-off-by: Jonathan Cameron — drivers/iio/adc/xilinx-xadc-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/drivers/iio/adc/xilinx-xadc-core.c b/drivers/iio/adc/xilinx-xadc-core.c index 0370624..c2b5f10 100644 — …
Nov 15 2015
clk: ti: dra7: constify clk_hw_omap_ops structure
Author: Julia Lawall <Julia.Lawall@lip6.fr> The clk_hw_omap_ops structures are never modified, so declare this one as const, like the others. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Stephen Boyd — drivers/clk/ti/apll.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/drivers/clk/ti/apll.c b/drivers/clk/ti/apll.c index f3eab6e..b336a8c 100644 — a/drivers/clk/ti/apll.c +++ b/drivers/clk/ti/apll.c …
Nov 15 2015
crypto: ixp4xx – Delete unnecessary checks before the function call “dma_pool_destroy”
Author: Markus Elfring <elfring@users.sourceforge.net> The dma_pool_destroy() function tests whether its argument is NULL and then returns immediately. Thus the test around the calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Herbert Xu — drivers/crypto/ixp4xx_crypto.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-) diff …
Nov 15 2015
KVM-async_pf: Delete an unnecessary check before the function call “kmem_cache_destroy”
Author: Markus Elfring <elfring@users.sourceforge.net> The kmem_cache_destroy() 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: Paolo Bonzini — virt/kvm/async_pf.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Nov 15 2015
batman-adv: Delete unnecessary checks before the function call “kfree_skb”
Author: Markus Elfring <elfring@users.sourceforge.net> The kfree_skb() function tests whether its argument is NULL and then returns immediately. Thus the test around the calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Marek Lindner Signed-off-by: Antonio Quartulli — net/batman-adv/main.c | 2 +- net/batman-adv/network-coding.c | 4 +— net/batman-adv/send.c | …
Nov 15 2015
pinctrl: imx1-core: add missing of_node_put
Author: Vaishali Thakkar <vthakkar1994@gmail.com> for_each_child_of_node performs an of_node_get on each iteration, so of_node_put is required on break out of the loop. This is done using Coccinelle. And semantic patch used for this is as follows: @@ expression root,e; local idexpression child; @@ for_each_child_of_node(root, child) { … when != of_node_put(child) when != e = child ( …
Nov 14 2015
Bluetooth: Delete an unnecessary check before the function call “kfree_skb”
Author: Markus Elfring <elfring@users.sourceforge.net> The kfree_skb() 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: Marcel Holtmann — net/bluetooth/cmtp/core.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Nov 14 2015
mac802154: Delete an unnecessary check before the function call “kfree_skb”
Author: Markus Elfring <elfring@users.sourceforge.net> The kfree_skb() 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: Marcel Holtmann — net/mac802154/rx.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Nov 14 2015
mmc: pwrseq: constify mmc_pwrseq_ops structures
Author: Julia Lawall <Julia.Lawall@lip6.fr> The mmc_pwrseq_ops structures are never modified, so declare them as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Ulf Hansson — drivers/mmc/core/pwrseq.h | 2 +- drivers/mmc/core/pwrseq_emmc.c | 2 +- drivers/mmc/core/pwrseq_simple.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff –git a/drivers/mmc/core/pwrseq.h b/drivers/mmc/core/pwrseq.h index 096da48..133de04 100644 …
