Quentin LAMBERT
Author's posts
Nov 24 2014
ASoC: sigmadsp: fix simple_return.cocci warnings
Author: kbuild test robot <fengguang.wu@intel.com> sound/soc/codecs/sigmadsp.c:656:1-4: WARNING: end returns can be simpified and declaration on line 636 can be dropped Simplify a trivial if-return sequence. Possibly combine with a preceding function call. Generated by: scripts/coccinelle/misc/simple_return.cocci Signed-off-by: Fengguang Wu Signed-off-by: Mark Brown — sound/soc/codecs/sigmadsp.c | 7 +—— 1 file changed, 1 insertion(+), 6 deletions(-) diff …
Nov 24 2014
platform: x86: Deletion of checks before 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 For msi-wmi.c: Acked-by: Anisse Astier Signed-off-by: Darren Hart — drivers/platform/x86/asus-laptop.c | 3 +– drivers/platform/x86/asus-wmi.c | 3 +– …
Nov 23 2014
staging: android: ion: Deletion of unnecessary checks before two function calls
Author: Markus Elfring <elfring@users.sourceforge.net> The functions ion_heap_destroy() and vfree() perform 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: Greg Kroah-Hartman — drivers/staging/android/ion/ion.c | 6 ++—- drivers/staging/android/ion/ion_dummy_driver.c | 6 ++—- drivers/staging/android/ion/tegra/tegra_ion.c | 6 ++—- 3 files changed, 6 …
Nov 23 2014
video: uvesafb: Deletion of an unnecessary check before the function call “platform_device_put”
Author: Markus Elfring <elfring@users.sourceforge.net> The platform_device_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/uvesafb.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Nov 23 2014
video: fbdev-VIA: Deletion of an unnecessary check before the function call “framebuffer_release”
Author: Markus Elfring <elfring@users.sourceforge.net> The framebuffer_release() 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/via/viafbdev.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Nov 23 2014
video: fbdev-MMP: Deletion of an unnecessary check before the function call “mmp_unregister_path”
Author: Markus Elfring <elfring@users.sourceforge.net> The mmp_unregister_path() 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/mmp/hw/mmp_ctrl.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Nov 23 2014
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 …
Nov 23 2014
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 …
Nov 23 2014
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 = …
Nov 23 2014
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 …