[PATCH v2 03/10] staging: rtl8712: remove useless return value

Luca Ceresoli luca at lucaceresoli.net
Tue May 19 09:35:19 UTC 2015


The loadparam() function cannot fail, it's called only once and its return
value is ignored there.

Signed-off-by: Luca Ceresoli <luca at lucaceresoli.net>
Cc: Greg Kroah-Hartman <greg at kroah.com>
Cc: Larry Finger <Larry.Finger at lwfinger.net>
Cc: Florian Schilhabel <florian.c.schilhabel at googlemail.com>
---
 drivers/staging/rtl8712/os_intfs.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8712/os_intfs.c b/drivers/staging/rtl8712/os_intfs.c
index 27a875b..ef778e1 100644
--- a/drivers/staging/rtl8712/os_intfs.c
+++ b/drivers/staging/rtl8712/os_intfs.c
@@ -125,9 +125,8 @@ MODULE_PARM_DESC(initmac, "MAC-Address, default: use FUSE");
 static int netdev_open(struct net_device *pnetdev);
 static int netdev_close(struct net_device *pnetdev);
 
-static uint loadparam(struct _adapter *padapter, struct  net_device *pnetdev)
+static void 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;
@@ -171,7 +170,6 @@ 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;
 }
 
 static int r871x_net_set_mac_address(struct net_device *pnetdev, void *p)
-- 
1.9.1



More information about the devel mailing list