staging: rtl8712: Remove useless variable 'fw_found'
authorThomas Vegas <thomas@grouk.net>
Sun, 1 Jun 2014 12:34:40 +0000 (14:34 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Jun 2014 23:40:07 +0000 (16:40 -0700)
The net device only exists when firmware is loaded successfully.
Firmware presence variable is only used through r871xu_dev_remove() and
this function already checks for net device existence.

Signed-off-by: Thomas Vegas <thomas@grouk.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8712/drv_types.h
drivers/staging/rtl8712/hal_init.c
drivers/staging/rtl8712/os_intfs.c
drivers/staging/rtl8712/usb_intf.c

index 3362e5e32bcc8cec607fe0dd76470537552ae091..0168745ba191752168dda76a88db92a3dba89a5d 100644 (file)
@@ -160,7 +160,6 @@ struct _adapter {
        s32     bSurpriseRemoved;
        u32     IsrContent;
        u32     ImrContent;
-       bool    fw_found;
        u8      EepromAddressSize;
        u8      hw_init_completed;
        struct task_struct *cmdThread;
index 36de7e478f3237139b51436be174f9cd97915e3e..0556de3b8904e419919224b78b8fa8678f0b8a12 100644 (file)
@@ -50,13 +50,11 @@ static void rtl871x_load_fw_cb(const struct firmware *firmware, void *context)
                struct usb_device *udev = padapter->dvobjpriv.pusbdev;
                struct usb_interface *pusb_intf = padapter->pusb_intf;
                dev_err(&udev->dev, "r8712u: Firmware request failed\n");
-               padapter->fw_found = false;
                usb_put_dev(udev);
                usb_set_intfdata(pusb_intf, NULL);
                return;
        }
        padapter->fw = firmware;
-       padapter->fw_found = true;
        /* firmware available - start netdev */
        register_netdev(padapter->pnetdev);
 }
index 6bd08213cb70fcc8d28c6266be4166a932afb20b..13debb59ad97ac2f721b768c446fd732b1ab7790 100644 (file)
@@ -345,8 +345,7 @@ u8 r8712_free_drv_sw(struct _adapter *padapter)
        r8712_free_mlme_priv(&padapter->mlmepriv);
        r8712_free_io_queue(padapter);
        _free_xmit_priv(&padapter->xmitpriv);
-       if (padapter->fw_found)
-               _r8712_free_sta_priv(&padapter->stapriv);
+       _r8712_free_sta_priv(&padapter->stapriv);
        _r8712_free_recv_priv(&padapter->recvpriv);
        mp871xdeinit(padapter);
        if (pnetdev)
index bec9d45499ad5174f4deff5ed851651d7f98e16a..beff69b3ff0c5b4b8eb574d5697e11ea3b22586f 100644 (file)
@@ -613,8 +613,7 @@ static void r871xu_dev_remove(struct usb_interface *pusb_intf)
                struct _adapter *padapter = netdev_priv(pnetdev);
 
                usb_set_intfdata(pusb_intf, NULL);
-               if (padapter->fw_found)
-                       release_firmware(padapter->fw);
+               release_firmware(padapter->fw);
                /* never exit with a firmware callback pending */
                wait_for_completion(&padapter->rtl8712_fw_ready);
                if (drvpriv.drv_registered == true)
@@ -623,8 +622,7 @@ static void r871xu_dev_remove(struct usb_interface *pusb_intf)
                flush_scheduled_work();
                udelay(1);
                /* Stop driver mlme relation timer */
-               if (padapter->fw_found)
-                       r8712_stop_drv_timers(padapter);
+               r8712_stop_drv_timers(padapter);
                r871x_dev_unload(padapter);
                r8712_free_drv_sw(padapter);