summaryrefslogtreecommitdiffstats
path: root/drivers/usb/atm
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-02-05 10:33:09 -0500
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2015-03-18 11:56:40 -0400
commit533726f736c2c32956d78ad6f49ffaada110cbfd (patch)
tree816a5a24cdf390fe905324d5d70cde4251f349b7 /drivers/usb/atm
parent28ed20755897b3726f07a0418c3d2fe17b3e5c86 (diff)
ueagle-atm: Delete unnecessary checks before the function call "release_firmware"
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 <elfring@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/atm')
-rw-r--r--drivers/usb/atm/ueagle-atm.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/usb/atm/ueagle-atm.c b/drivers/usb/atm/ueagle-atm.c
index 5a459377574b..888998a7fe31 100644
--- a/drivers/usb/atm/ueagle-atm.c
+++ b/drivers/usb/atm/ueagle-atm.c
@@ -952,7 +952,7 @@ static void uea_load_page_e1(struct work_struct *work)
952 int i; 952 int i;
953 953
954 /* reload firmware when reboot start and it's loaded already */ 954 /* reload firmware when reboot start and it's loaded already */
955 if (ovl == 0 && pageno == 0 && sc->dsp_firm) { 955 if (ovl == 0 && pageno == 0) {
956 release_firmware(sc->dsp_firm); 956 release_firmware(sc->dsp_firm);
957 sc->dsp_firm = NULL; 957 sc->dsp_firm = NULL;
958 } 958 }
@@ -1074,7 +1074,7 @@ static void uea_load_page_e4(struct work_struct *work)
1074 uea_dbg(INS_TO_USBDEV(sc), "sending DSP page %u\n", pageno); 1074 uea_dbg(INS_TO_USBDEV(sc), "sending DSP page %u\n", pageno);
1075 1075
1076 /* reload firmware when reboot start and it's loaded already */ 1076 /* reload firmware when reboot start and it's loaded already */
1077 if (pageno == 0 && sc->dsp_firm) { 1077 if (pageno == 0) {
1078 release_firmware(sc->dsp_firm); 1078 release_firmware(sc->dsp_firm);
1079 sc->dsp_firm = NULL; 1079 sc->dsp_firm = NULL;
1080 } 1080 }