staging: wilc1000: remove pstrWFIDrv in host_int_remove_key
authorChaehyun Lim <chaehyun.lim@gmail.com>
Mon, 5 Oct 2015 10:35:01 +0000 (19:35 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 8 Oct 2015 09:58:33 +0000 (10:58 +0100)
This patch remove pstrWFIDrv in host_int_remove_key. There is no need to
make another variable to check if first argument is NULL or not.
It is able to use wfi_drv directly that is first argument of this
function.

Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/host_interface.c

index 7acd744efed62636c8a65b65cea06691b2a76108..1e465522d4e87459273794f3053d8c10a02b94ae 100644 (file)
@@ -4191,11 +4191,10 @@ s32 host_int_remove_key(tstrWILC_WFIDrv *hWFIDrv, const u8 *pu8StaAddress)
 int host_int_remove_wep_key(tstrWILC_WFIDrv *wfi_drv, u8 index)
 {
        int result = 0;
-       tstrWILC_WFIDrv *pstrWFIDrv = (tstrWILC_WFIDrv *)wfi_drv;
        struct host_if_msg msg;
 
 
-       if (pstrWFIDrv == NULL) {
+       if (wfi_drv == NULL) {
                result = -EFAULT;
                PRINT_ER("Failed to send setup multicast config packet\n");
                return result;
@@ -4219,7 +4218,7 @@ int host_int_remove_wep_key(tstrWILC_WFIDrv *wfi_drv, u8 index)
        result = wilc_mq_send(&gMsgQHostIF, &msg, sizeof(struct host_if_msg));
        if (result)
                PRINT_ER("Error in sending message queue : Request to remove WEP key\n");
-       down(&(pstrWFIDrv->hSemTestKeyBlock));
+       down(&(wfi_drv->hSemTestKeyBlock));
 
        return result;
 }