Author: Markus Elfring <elfring@users.sourceforge.net> The snd_util_memhdr_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: Takashi Iwai — sound/pci/trident/trident_main.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Catégorie : Linux
Dec 02 2014
ALSA: echoaudio: Deletion of a check before release_and_free_resource()
Author: Markus Elfring <elfring@users.sourceforge.net> The release_and_free_resource() 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: Takashi Iwai — sound/pci/echoaudio/echoaudio.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Dec 02 2014
ALSA: asihpi: 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: Takashi Iwai — sound/pci/asihpi/hpioctl.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff –git a/sound/pci/asihpi/hpioctl.c b/sound/pci/asihpi/hpioctl.c index e457eb8..6aa677e …
Dec 02 2014
ASoC: mop500: Deletion of unnecessary checks before the function call « 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: Mark Brown — sound/soc/ux500/mop500.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-) diff …
Dec 02 2014
ASoC: fsi: Deletion of unnecessary checks before the function call « clk_disable »
Author: Markus Elfring <elfring@users.sourceforge.net> The clk_disable() 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: Mark Brown — sound/soc/sh/fsi.c | 9 +++—— 1 file changed, 3 insertions(+), 6 deletions(-) diff …
Dec 02 2014
of: Delete unnecessary check before calling « 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: Grant Likely — drivers/of/unittest.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Dec 02 2014
lustre: Deletion of unnecessary checks before three function calls
Author: Markus Elfring <elfring@users.sourceforge.net> The functions free_ll_remote_perm(), free_rmtperm_hash() and iput() test whether their argument is NULL and then return immediately. Thus the test around their calls 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/lustre/lustre/llite/remote_perm.c | 5 ++— drivers/staging/lustre/lustre/llite/statahead.c | 3 +– 2 files …
Dec 01 2014
[media] mn88473: Remove uneeded check before 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/mn88473/mn88473.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) …
Dec 01 2014
ALSA: dice: fix semicolon.cocci warnings
Author: Fengguang Wu <fengguang.wu@intel.com> sound/firewire/dice/dice-transaction.c:34:2-3: Unneeded semicolon Removes unneeded semicolon. Generated by: scripts/coccinelle/misc/semicolon.cocci CC: Takashi Sakamoto Signed-off-by: Fengguang Wu Signed-off-by: Takashi Iwai — sound/firewire/dice/dice-transaction.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/sound/firewire/dice/dice-transaction.c b/sound/firewire/dice/dice-transaction.c index a9b98e0..1fe304c 100644 — a/sound/firewire/dice/dice-transaction.c +++ b/sound/firewire/dice/dice-transaction.c @@ -31,7 +31,7 @@ static u64 get_subaddr(struct snd_dice *dice, …
Dec 01 2014
[media] lirc_zilog: Deletion of unnecessary checks before 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: Mauro Carvalho Chehab — drivers/staging/media/lirc/lirc_zilog.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-) diff –git a/drivers/staging/media/lirc/lirc_zilog.c b/drivers/staging/media/lirc/lirc_zilog.c index …
