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: Peter Zijlstra (Intel) Cc: Arnaldo Carvalho de Melo Cc: H. Peter Anvin Cc: Linus Torvalds Link: …
Category: Linux
Feb 03 2015
pwm: Remove unnecessary check before 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: Thierry Reding — drivers/pwm/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff …
Feb 03 2015
ti-soc-thermal: Delete an unnecessary check before the function call “cpufreq_cooling_unregister”
Author: Markus Elfring <elfring@users.sourceforge.net> The cpufreq_cooling_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: Eduardo Valentin — drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff …
Feb 02 2015
sata_mv: Delete unnecessary checks before the function call “phy_power_off”
Author: Markus Elfring <elfring@users.sourceforge.net> The phy_power_off() 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/sata_mv.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-) diff …
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(+), …
