redonkable/alistair23-linux
redonkable/alistair23-linux
1
0
Fork 0
Code Releases Activity

staging: rtl8188eu: return an error code, not a boolean

Browse source 
"If the name of a function is an action or an imperative command,
the function should return an error-code integer." (Documentation/CodingStyle)

Several action-like functions in this driver return a boolean: _SUCCESS = 1 on
success, _FAIL = 0 on error, defined in
drivers/staging/rtl8188eu/include/osdep_service.h.

Change rtw_start_drv_threads() to return a proper 0-or-error value.

Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Greg Kroah-Hartman <gregkh@linux.com>
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Luca Ceresoli 2015-05-19 11:35:26 +02:00 committed by Greg Kroah-Hartman
parent 8b9ffb43ac
commit 7fc0406f48
1 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
drivers/staging/rtl8188eu/os_dep/os_intfs.c
View file

@ -752,21 +752,21 @@ struct net_device *rtw_init_netdev(struct adapter *old_padapter)
return pnetdev; return pnetdev;
} }
static u32 rtw_start_drv_threads(struct adapter *padapter) static int rtw_start_drv_threads(struct adapter *padapter)
{ {
u32 _status = _SUCCESS; int err = 0;
RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+rtw_start_drv_threads\n")); RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+rtw_start_drv_threads\n"));
padapter->cmdThread = kthread_run(rtw_cmd_thread, padapter, padapter->cmdThread = kthread_run(rtw_cmd_thread, padapter,
"RTW_CMD_THREAD"); "RTW_CMD_THREAD");
if (IS_ERR(padapter->cmdThread)) if (IS_ERR(padapter->cmdThread))
_status = _FAIL; err = PTR_ERR(padapter->cmdThread);
else else
/* wait for cmd_thread to run */ /* wait for cmd_thread to run */
_rtw_down_sema(&padapter->cmdpriv.terminate_cmdthread_sema); _rtw_down_sema(&padapter->cmdpriv.terminate_cmdthread_sema);
return _status; return err;
} }
void rtw_stop_drv_threads(struct adapter *padapter) void rtw_stop_drv_threads(struct adapter *padapter)
@ -979,6 +979,7 @@ u8 rtw_free_drv_sw(struct adapter *padapter)
static int _netdev_open(struct net_device *pnetdev) static int _netdev_open(struct net_device *pnetdev)
{ {
uint status; uint status;
int err;
struct adapter *padapter = (struct adapter *)rtw_netdev_priv(pnetdev); struct adapter *padapter = (struct adapter *)rtw_netdev_priv(pnetdev);
struct pwrctrl_priv *pwrctrlpriv = &padapter->pwrctrlpriv; struct pwrctrl_priv *pwrctrlpriv = &padapter->pwrctrlpriv;
@ -1002,8 +1003,8 @@ static int _netdev_open(struct net_device *pnetdev)
pr_info("MAC Address = %pM\n", pnetdev->dev_addr); pr_info("MAC Address = %pM\n", pnetdev->dev_addr);
status = rtw_start_drv_threads(padapter); err = rtw_start_drv_threads(padapter);
if (status == _FAIL) { if (err) {
pr_info("Initialize driver software resource Failed!\n"); pr_info("Initialize driver software resource Failed!\n");
goto netdev_open_error; goto netdev_open_error;
} }