staging: rtl8188eu: dont mix success and error path
authorSudip Mukherjee <sudipm.mukherjee@gmail.com>
Tue, 21 Jul 2015 13:27:27 +0000 (18:57 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Jul 2015 03:59:55 +0000 (20:59 -0700)
Success and error path was mixed. Separate them by directly returning 0
from the success path. In the process remove the variable which became
unused.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/os_dep/usb_intf.c

index 1c9558ff1c19ff31f82963a2a60b5a20a814f92f..29b7ea6dc16d1a6e7e6025be7494ae972f9af05c 100644 (file)
@@ -474,7 +474,6 @@ static void rtw_usb_if1_deinit(struct adapter *if1)
 static int rtw_drv_init(struct usb_interface *pusb_intf, const struct usb_device_id *pdid)
 {
        struct adapter *if1 = NULL;
-       int status = _FAIL;
        struct dvobj_priv *dvobj;
 
        RT_TRACE(_module_hci_intfs_c_, _drv_err_, ("+rtw_drv_init\n"));
@@ -495,13 +494,12 @@ static int rtw_drv_init(struct usb_interface *pusb_intf, const struct usb_device
 
        RT_TRACE(_module_hci_intfs_c_, _drv_err_, ("-871x_drv - drv_init, success!\n"));
 
-       status = _SUCCESS;
+       return 0;
 
 free_dvobj:
-       if (status != _SUCCESS)
-               usb_dvobj_deinit(pusb_intf);
+       usb_dvobj_deinit(pusb_intf);
 exit:
-       return status == _SUCCESS ? 0 : -ENODEV;
+       return -ENODEV;
 }
 
 /*
This page took 0.033228 seconds and 5 git commands to generate.