Quentin LAMBERT
Author's posts
Feb 02 2015
ata: Delete 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: Tejun Heo — drivers/ata/pata_cs5530.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-) diff …
Feb 02 2015
EDAC: Delete unnecessary checks before pci_dev_put()
Author: Markus Elfring <elfring@users.sourceforge.net> The pci_dev_put() function tests whether its argument is NULL and thus the test around 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/54CFC12C.9010002@users.sourceforge.net Signed-off-by: Borislav Petkov — drivers/edac/i82443bxgx_edac.c | 4 +— drivers/edac/i82860_edac.c | 9 ++——- drivers/edac/i82875p_edac.c | 4 +— drivers/edac/i82975x_edac.c | …
Feb 02 2015
dm snapshot: remove unnecessary NULL checks before vfree() calls
Author: Markus Elfring <elfring@users.sourceforge.net> The vfree() function performs input parameter validation. Thus the NULL pointer test around vfree() calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Mike Snitzer — drivers/md/dm-snap-persistent.c | 14 ++++———- 1 file changed, 4 insertions(+), 10 deletions(-) diff –git a/drivers/md/dm-snap-persistent.c b/drivers/md/dm-snap-persistent.c index …
Feb 02 2015
[media] staging: bcm2048: Delete an unnecessary check before the function call “video_unregister_device”
Author: Markus Elfring <elfring@users.sourceforge.net> The video_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: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab — drivers/staging/media/bcm2048/radio-bcm2048.c | 4 +— 1 file changed, 1 insertion(+), …
Feb 02 2015
netlabel: Deletion of an unnecessary check before the function call “cipso_v4_doi_free”
Author: Markus Elfring <elfring@users.sourceforge.net> The cipso_v4_doi_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 Acked-by: Paul Moore Signed-off-by: David S. Miller — net/netlabel/netlabel_cipso_v4.c | 3 +– 1 file changed, 1 insertion(+), …
Feb 02 2015
netlabel: Deletion of an unnecessary check before the function call “cipso_v4_doi_putdef”
Author: Markus Elfring <elfring@users.sourceforge.net> The cipso_v4_doi_putdef() 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 Acked-by: Paul Moore Signed-off-by: David S. Miller — net/netlabel/netlabel_mgmt.c | 3 +– 1 file changed, 1 insertion(+), …
Feb 01 2015
[media] mn88472: 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 Signed-off-by: Mauro Carvalho Chehab — drivers/staging/media/mn88472/mn88472.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) …
Feb 01 2015
jfs: Deletion of an unnecessary check before the function call “unload_nls”
Author: Markus Elfring <elfring@users.sourceforge.net> The unload_nls() 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: Dave Kleikamp — fs/jfs/super.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Jan 31 2015
net: sctp: Deletion of 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 Acked-By: Neil Horman Signed-off-by: David S. Miller — net/sctp/associola.c | 3 +– 1 file changed, 1 insertion(+), …
Jan 30 2015
[media] media: platform: fix platform_no_drv_owner.cocci warnings
Author: Fengguang Wu <fengguang.wu@intel.com> drivers/media/platform/am437x/am437x-vpfe.c:2767: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 CC: Benoit Parrot Signed-off-by: Fengguang Wu Acked-by: Lad, Prabhakar Signed-off-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab — drivers/media/platform/am437x/am437x-vpfe.c | 1 – 1 file changed, 1 …