Category: Linux

ASoC: rk3036: fix platform_no_drv_owner.cocci warnings

Author: kbuild test robot <lkp@intel.com> sound/soc/codecs/inno_rk3036.c:480:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: Fengguang Wu Signed-off-by: Mark Brown — sound/soc/codecs/inno_rk3036.c | 1 – 1 file changed, 1 deletion(-)   diff –git a/sound/soc/codecs/inno_rk3036.c b/sound/soc/codecs/inno_rk3036.c index 24677a8..9b6e884 100644 …

Continue reading

ALSA: ctxfi: constify rsc ops structures

Author: Julia Lawall <Julia.Lawall@lip6.fr> The various rsc ops structures are never modified, so declare them as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Takashi Iwai — sound/pci/ctxfi/ctamixer.c | 6 +++— sound/pci/ctxfi/ctamixer.h | 2 +- sound/pci/ctxfi/ctdaio.c | 10 +++++—– sound/pci/ctxfi/ctdaio.h | 4 ++– sound/pci/ctxfi/ctresource.c | 2 +- sound/pci/ctxfi/ctresource.h | 2 +- sound/pci/ctxfi/ctsrc.c …

Continue reading

video: constify geode ops structures

Author: Julia Lawall <Julia.Lawall@lip6.fr> These geode ops structures are never modified, so declare them as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Tomi Valkeinen — drivers/video/fbdev/geode/display_gx1.c | 2 +- drivers/video/fbdev/geode/display_gx1.h | 2 +- drivers/video/fbdev/geode/geodefb.h | 4 ++– drivers/video/fbdev/geode/video_cs5530.c | 2 +- drivers/video/fbdev/geode/video_cs5530.h | 2 +- 5 files changed, 6 insertions(+), 6 …

Continue reading

dwc_eth_qos: Delete an unnecessary check before the function call “of_node_put”

Author: Markus Elfring <elfring@users.sourceforge.net> The of_node_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 — drivers/net/ethernet/synopsys/dwc_eth_qos.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   …

Continue reading

nilfs2: drop null test before destroy functions

Author: Julia Lawall <Julia.Lawall@lip6.fr> Remove unneeded NULL test. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ expression x; @@ -if (x != NULL) \(kmem_cache_destroy\|mempool_destroy\|dma_pool_destroy\)(x); // Signed-off-by: Julia Lawall Signed-off-by: Ryusuke Konishi Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds — fs/nilfs2/super.c | 12 ++++——– 1 file changed, 4 insertions(+), 8 deletions(-)   …

Continue reading

iommu/arm-smmu: Delete an unnecessary check before free_io_pgtable_ops()

Author: Markus Elfring <elfring@users.sourceforge.net> The free_io_pgtable_ops() 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: Will Deacon — drivers/iommu/arm-smmu.c | 4 +— 1 file changed, 1 insertion(+), 3 deletions(-)   diff …

Continue reading

HID: wacom: Delete an unnecessary check before kobject_put()

Author: Markus Elfring <elfring@users.sourceforge.net> The kobject_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: Jiri Kosina — drivers/hid/wacom_sys.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff …

Continue reading

power: bq2415x_charger: Delete unnecessary checks before the function call “of_node_put”

Author: Markus Elfring <elfring@users.sourceforge.net> The of_node_put() 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 Acked-by: Pali Rohár Signed-off-by: Sebastian Reichel — drivers/power/bq2415x_charger.c | 6 ++—- 1 file changed, 2 insertions(+), 4 …

Continue reading

GPU-DRM: Delete unnecessary checks before drm_property_unreference_blob()

Author: Markus Elfring <elfring@users.sourceforge.net> The drm_property_unreference_blob() function tests whether its argument is NULL and then returns immediately. Thus the tests around the calls are not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Link: http://patchwork.freedesktop.org/patch/msgid/563C8B3E.405@users.sourceforge.net Signed-off-by: Daniel Vetter — drivers/gpu/drm/drm_atomic.c | 9 +++—— drivers/gpu/drm/drm_atomic_helper.c | 5 ++— 2 files changed, …

Continue reading

cxl: Delete an unnecessary check before the function call “kfree”

Author: Markus Elfring <elfring@users.sourceforge.net> The kfree() 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 Reviewed-by: Andrew Donnellan Acked-by: Ian Munsie Signed-off-by: Michael Ellerman — drivers/misc/cxl/context.c | 3 +– 1 file changed, …

Continue reading