Category: Linux

video: mx3fb: Deletion of an unnecessary check before the function call “backlight_device_unregister”

Author: Markus Elfring <elfring@users.sourceforge.net> The backlight_device_unregister() 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: Tomi Valkeinen — drivers/video/fbdev/mx3fb.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff …

Continue reading

drm: rcar-du: fix error return code

Author: Julia Lawall <Julia.Lawall@lip6.fr> Propagate the error code on failure. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e1,e2; @@ ( if (\(ret < 0\|ret != 0\)) { ... return ret; } | ret = 0 ) ... when != ret = e1 …

Continue reading

drm/exynos/ipp: fix error return code

Author: Julia Lawall <Julia.Lawall@lip6.fr> Propagate the returned error code on failure. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e1,e2; @@ ( if (\(ret < 0\|ret != 0\)) { ... return ret; } | ret = 0 ) ... when != ret = …

Continue reading

video: fbdev-OMAP2: Deletion of unnecessary checks before the function call “i2c_put_adapter”

Author: Markus Elfring <elfring@users.sourceforge.net> The i2c_put_adapter() 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: Tomi Valkeinen — drivers/video/fbdev/omap2/displays-new/connector-dvi.c | 9 +++—— 1 file changed, 3 insertions(+), 6 deletions(-)   diff …

Continue reading

video: fbdev-SIS: Deletion of unnecessary checks before the function call “pci_dev_put”

Author: Markus Elfring <elfring@users.sourceforge.net> The pci_dev_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: Tomi Valkeinen — drivers/video/fbdev/sis/sis_main.c | 14 +++++——— 1 file changed, 5 insertions(+), 9 deletions(-)   diff …

Continue reading

video: smscufx: Deletion of unnecessary checks before the function call “vfree”

Author: Markus Elfring <elfring@users.sourceforge.net> The vfree() function performs also input parameter validation. 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: Tomi Valkeinen — drivers/video/fbdev/smscufx.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-)   diff –git a/drivers/video/fbdev/smscufx.c b/drivers/video/fbdev/smscufx.c index d513ed6..9279e5f …

Continue reading

PCI/MSI: Rename mask/unmask_msi_irq treewide

Author: Thomas Gleixner <tglx@linutronix.de> The PCI/MSI irq chip callbacks mask/unmask_msi_irq have been renamed to pci_msi_mask/unmask_irq to mark them PCI specific. Rename all usage sites. The conversion helper functions are kept around to avoid conflicts in next and will be removed after merging into mainline. Coccinelle assisted conversion. No functional change. Signed-off-by: Thomas Gleixner Cc: Bjorn …

Continue reading

video: udlfb: Deletion of unnecessary checks before the function call “vfree”

Author: Markus Elfring <elfring@users.sourceforge.net> The vfree() function performs also input parameter validation. 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: Tomi Valkeinen — drivers/video/fbdev/udlfb.c | 9 +++—— 1 file changed, 3 insertions(+), 6 deletions(-)   diff –git a/drivers/video/fbdev/udlfb.c b/drivers/video/fbdev/udlfb.c index 046d51d..ff2b873 …

Continue reading

video: uvesafb: Deletion of an unnecessary check before the function call “uvesafb_free”

Author: Markus Elfring <elfring@users.sourceforge.net> The uvesafb_free() 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: Tomi Valkeinen — drivers/video/fbdev/uvesafb.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff …

Continue reading

video: fbdev-LCDC: Deletion of an unnecessary check before the function call “vfree”

Author: Markus Elfring <elfring@users.sourceforge.net> The vfree() function performs also input parameter validation. 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: Tomi Valkeinen — drivers/video/fbdev/sh_mobile_lcdcfb.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/drivers/video/fbdev/sh_mobile_lcdcfb.c b/drivers/video/fbdev/sh_mobile_lcdcfb.c index 2bcc84a..cfde21d …

Continue reading