Commit de3748f6 authored by Markus Elfring's avatar Markus Elfring Committed by Dmitry Torokhov

Input: adi - remove an unnecessary check

The input_free_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: default avatarMarkus Elfring <elfring@users.sourceforge.net>
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
parent 2b2f5147
...@@ -535,8 +535,7 @@ static int adi_connect(struct gameport *gameport, struct gameport_driver *drv) ...@@ -535,8 +535,7 @@ static int adi_connect(struct gameport *gameport, struct gameport_driver *drv)
} }
} }
fail2: for (i = 0; i < 2; i++) fail2: for (i = 0; i < 2; i++)
if (port->adi[i].dev) input_free_device(port->adi[i].dev);
input_free_device(port->adi[i].dev);
gameport_close(gameport); gameport_close(gameport);
fail1: gameport_set_drvdata(gameport, NULL); fail1: gameport_set_drvdata(gameport, NULL);
kfree(port); kfree(port);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment