This patch removes unused return variable in this file.
Signed-off-by: Gnanachandran Dhanapal <[email protected]>
---
drivers/staging/rtl8712/os_intfs.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8712/os_intfs.c b/drivers/staging/rtl8712/os_intfs.c
index 6e776e5..b881368 100644
--- a/drivers/staging/rtl8712/os_intfs.c
+++ b/drivers/staging/rtl8712/os_intfs.c
@@ -128,7 +128,6 @@ static int netdev_close(struct net_device *pnetdev);
static uint loadparam(struct _adapter *padapter, struct net_device *pnetdev)
{
- uint status = _SUCCESS;
struct registry_priv *registry_par = &padapter->registrypriv;
registry_par->chip_version = (u8)chip_version;
@@ -172,7 +171,7 @@ static uint loadparam(struct _adapter *padapter, struct net_device *pnetdev)
registry_par->low_power = (u8)low_power;
registry_par->wifi_test = (u8) wifi_test;
r8712_initmac = initmac;
- return status;
+ return _SUCCESS;
}
static int r871x_net_set_mac_address(struct net_device *pnetdev, void *p)
@@ -273,7 +272,6 @@ void r8712_stop_drv_timers(struct _adapter *padapter)
static u8 init_default_value(struct _adapter *padapter)
{
- u8 ret = _SUCCESS;
struct registry_priv *pregistrypriv = &padapter->registrypriv;
struct xmit_priv *pxmitpriv = &padapter->xmitpriv;
struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
@@ -306,7 +304,7 @@ static u8 init_default_value(struct _adapter *padapter)
r8712_init_registrypriv_dev_network(padapter);
r8712_update_registrypriv_dev_network(padapter);
/*misc.*/
- return ret;
+ return _SUCCESS;
}
u8 r8712_init_drv_sw(struct _adapter *padapter)
--
1.7.9.5
On 04/23/2015 10:00 AM, DHANAPAL, GNANACHANDRAN (G.) wrote:
> This patch removes unused return variable in this file.
You should hold off with V2 to give others a chance to comment.
Your commit message does not exactly describe what is happening. The return
variable is not unused. More accurately, you have rewritten the routine to
always return a constant.
I have not looked at the generated code, but I suspect that gcc is already doing
that when it optimizes the code. If you really want to reduce the size of the
generated code, there is no reason for this function to be of type uint. It
should be type void and not return anything.
Larry
>
> Signed-off-by: Gnanachandran Dhanapal <[email protected]>
> ---
> drivers/staging/rtl8712/os_intfs.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/rtl8712/os_intfs.c b/drivers/staging/rtl8712/os_intfs.c
> index 6e776e5..b881368 100644
> --- a/drivers/staging/rtl8712/os_intfs.c
> +++ b/drivers/staging/rtl8712/os_intfs.c
> @@ -128,7 +128,6 @@ static int netdev_close(struct net_device *pnetdev);
>
> static uint loadparam(struct _adapter *padapter, struct net_device *pnetdev)
> {
> - uint status = _SUCCESS;
> struct registry_priv *registry_par = &padapter->registrypriv;
>
> registry_par->chip_version = (u8)chip_version;
> @@ -172,7 +171,7 @@ static uint loadparam(struct _adapter *padapter, struct net_device *pnetdev)
> registry_par->low_power = (u8)low_power;
> registry_par->wifi_test = (u8) wifi_test;
> r8712_initmac = initmac;
> - return status;
> + return _SUCCESS;
> }
>
> static int r871x_net_set_mac_address(struct net_device *pnetdev, void *p)
> @@ -273,7 +272,6 @@ void r8712_stop_drv_timers(struct _adapter *padapter)
>
> static u8 init_default_value(struct _adapter *padapter)
> {
> - u8 ret = _SUCCESS;
> struct registry_priv *pregistrypriv = &padapter->registrypriv;
> struct xmit_priv *pxmitpriv = &padapter->xmitpriv;
> struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
> @@ -306,7 +304,7 @@ static u8 init_default_value(struct _adapter *padapter)
> r8712_init_registrypriv_dev_network(padapter);
> r8712_update_registrypriv_dev_network(padapter);
> /*misc.*/
> - return ret;
> + return _SUCCESS;
> }
>
> u8 r8712_init_drv_sw(struct _adapter *padapter)
>
This patch is a coccinelle thing, the reasoning behind it is that
literals are more clear than variables.
return status; <-- Ambiguous. This could be returning anything,
you have to look at the context.
return 0; <-- Clear.
regards,
dan carpenter
On Thu, Apr 23, 2015 at 03:00:14PM +0000, DHANAPAL, GNANACHANDRAN (G.) wrote:
> This patch removes unused return variable in this file.
>
> Signed-off-by: Gnanachandran Dhanapal <[email protected]>
> ---
your From: name and Signed-off-by: name is not matching.
regards
sudip