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....@gmail.com>
---
 drivers/staging/wilc1000/host_interface.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/wilc1000/host_interface.c 
b/drivers/staging/wilc1000/host_interface.c
index c02a007..e284afb 100644
--- a/drivers/staging/wilc1000/host_interface.c
+++ b/drivers/staging/wilc1000/host_interface.c
@@ -4294,11 +4294,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;
@@ -4322,7 +4321,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;
 }
-- 
2.6.0

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to