staging: wilc1000: use 'void' return type for host_int_get_assoc_res_info()
authorAjay Singh <ajay.kathat@microchip.com>
Tue, 25 Sep 2018 06:23:21 +0000 (11:53 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 25 Sep 2018 18:53:56 +0000 (20:53 +0200)
Change return type to 'void' for host_int_get_assoc_res_info() as its
return value is not used. 'rcvd_assoc_resp_info_len' parameter value is
used to know the status.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/host_interface.c

index 7729f83..237a098 100644 (file)
@@ -1388,10 +1388,10 @@ done:
        kfree(msg);
 }
 
-static s32 host_int_get_assoc_res_info(struct wilc_vif *vif,
-                                      u8 *assoc_resp_info,
-                                      u32 max_assoc_resp_info_len,
-                                      u32 *rcvd_assoc_resp_info_len)
+static void host_int_get_assoc_res_info(struct wilc_vif *vif,
+                                       u8 *assoc_resp_info,
+                                       u32 max_assoc_resp_info_len,
+                                       u32 *rcvd_assoc_resp_info_len)
 {
        int result;
        struct wid wid;
@@ -1406,11 +1406,10 @@ static s32 host_int_get_assoc_res_info(struct wilc_vif *vif,
        if (result) {
                *rcvd_assoc_resp_info_len = 0;
                netdev_err(vif->ndev, "Failed to send association response\n");
-               return -EINVAL;
+               return;
        }
 
        *rcvd_assoc_resp_info_len = wid.size;
-       return result;
 }
 
 static inline void host_int_free_user_conn_req(struct host_if_drv *hif_drv)