Catégorie : Linux

[media] i2c: Deletion of an unnecessary check before the function call « rc_unregister_device »

Author: Markus Elfring <elfring@users.sourceforge.net> The rc_unregister_device() 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: Mauro Carvalho Chehab — drivers/media/i2c/ir-kbd-i2c.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   …

Lire la suite

[media] firewire: Deletion of an unnecessary check before the function call « dvb_unregister_device »

Author: Markus Elfring <elfring@users.sourceforge.net> The dvb_unregister_device() 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: Stefan Richter Signed-off-by: Mauro Carvalho Chehab — drivers/media/firewire/firedtv-ci.c | 3 +– 1 file changed, 1 insertion(+), …

Lire la suite

[media] DVB-frontends: Deletion of unnecessary checks before the function call « release_firmware »

Author: Markus Elfring <elfring@users.sourceforge.net> GIT_AUTHOR_DATE=1416472432 The release_firmware() 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: Mauro Carvalho Chehab — drivers/media/dvb-frontends/drx39xyj/drxj.c | 3 +– drivers/media/dvb-frontends/drxk_hard.c | 3 +– drivers/media/dvb-frontends/m88ds3103.c | 3 …

Lire la suite

mISDN: 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: David S. Miller — drivers/isdn/mISDN/l1oip_codec.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-)   diff –git a/drivers/isdn/mISDN/l1oip_codec.c b/drivers/isdn/mISDN/l1oip_codec.c index …

Lire la suite

HID: picoLCD: Deletion of unnecessary checks before three function calls

Author: Markus Elfring <elfring@users.sourceforge.net> The functions backlight_device_unregister(), lcd_device_unregister() and rc_unregister_device() test whether their argument is NULL and then return 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: Bruno Prémont Signed-off-by: Jiri Kosina — drivers/hid/hid-picolcd_backlight.c | 3 +– drivers/hid/hid-picolcd_cir.c | 3 …

Lire la suite

drm/vmwgfx: Deletion of an unnecessary check before the function call « vfree »

Author: Markus Elfring <elfring@users.sourceforge.net> The vfree() function performes 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 Reviewed-by: Thierry Reding Signed-off-by: Dave Airlie — drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c …

Lire la suite

drm/udl: Deletion of an unnecessary check before the function call « vunmap »

Author: Markus Elfring <elfring@users.sourceforge.net> The vunmap() function performes 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 Reviewed-by: Thierry Reding Signed-off-by: Dave Airlie — drivers/gpu/drm/udl/udl_gem.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/drivers/gpu/drm/udl/udl_gem.c …

Lire la suite

drm/tilcdc: Deletion of an unnecessary check before the function call « drm_fbdev_cma_hotplug_event »

Author: Markus Elfring <elfring@users.sourceforge.net> The drm_fbdev_cma_hotplug_event() 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: Thierry Reding Signed-off-by: Dave Airlie — drivers/gpu/drm/tilcdc/tilcdc_drv.c | 3 +– 1 file changed, 1 insertion(+), 2 …

Lire la suite

drm/edid: Deletion of an unnecessary check before the function call « release_firmware »

Author: Markus Elfring <elfring@users.sourceforge.net> The release_firmware() 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: Thierry Reding Signed-off-by: Dave Airlie — drivers/gpu/drm/drm_edid_load.c | 3 +– 1 file changed, 1 insertion(+), 2 …

Lire la suite

EDAC: Delete unnecessary check before calling 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 before the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Link: http://lkml.kernel.org/r/546CB20D.4070808@users.sourceforge.net [ Boris: commit message. ] Signed-off-by: Borislav Petkov — drivers/edac/i3000_edac.c | 3 +– drivers/edac/i3200_edac.c | …

Lire la suite