staging: wilc1000: remove function pointer os_wait
authorGlen Lee <glen.lee@atmel.com>
Thu, 24 Sep 2015 09:15:05 +0000 (18:15 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 29 Sep 2015 00:39:59 +0000 (02:39 +0200)
This patch removes function pointer os_wait which is pointer of
linux_wlan_lock_timeout and just call the real name function directly.
Remove also static from linux_wlan_lock_timeout declaration.
As os_wait is deleted, structure wilc_wlan_os_func_t is useless. Delete
wilc_wlan_os_func_t, os_func and it's related codes.

Signed-off-by: Glen Lee <glen.lee@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/linux_wlan.c
drivers/staging/wilc1000/wilc_sdio.c
drivers/staging/wilc1000/wilc_wfi_netdevice.h
drivers/staging/wilc1000/wilc_wlan.c
drivers/staging/wilc1000/wilc_wlan_if.h

index 354775066fd1b40928a28640d2da324d73c3fe09..701d3179c99afc336a90f52826868da47d5cea31 100644 (file)
@@ -381,7 +381,7 @@ void linux_wlan_dbg(u8 *buff)
        PRINT_D(INIT_DBG, "%d\n", *buff);
 }
 
-static int linux_wlan_lock_timeout(void *vp, u32 timeout)
+int linux_wlan_lock_timeout(void *vp, u32 timeout)
 {
        int error = -1;
 
@@ -1052,7 +1052,6 @@ void linux_to_wlan(wilc_wlan_inp_t *nwi, linux_wlan_t *nic)
        PRINT_D(INIT_DBG, "Linux to Wlan services ...\n");
 
        nwi->os_context.os_private = (void *)nic;
-       nwi->os_func.os_wait = linux_wlan_lock_timeout;
 
 #ifdef WILC_SDIO
        nwi->io_func.io_type = HIF_SDIO;
index 48834c6846c83d480d3e309206b3a52bf15f96de..c22b35e643c7cb0b2721408652b7bf1f20bd94fb 100644 (file)
@@ -14,7 +14,6 @@
 
 typedef struct {
        void *os_context;
-       wilc_wlan_os_func_t os_func;
        u32 block_size;
        int (*sdio_cmd52)(sdio_cmd52_t *);
        int (*sdio_cmd53)(sdio_cmd53_t *);
@@ -586,7 +585,6 @@ static int sdio_init(wilc_wlan_inp_t *inp, wilc_debug_func func)
 
        g_sdio.dPrint = func;
        g_sdio.os_context = inp->os_context.os_private;
-       memcpy((void *)&g_sdio.os_func, (void *)&inp->os_func, sizeof(wilc_wlan_os_func_t));
 
        if (inp->io_func.io_init) {
                if (!inp->io_func.io_init(g_sdio.os_context)) {
index e8443064c06891e6876e15cdad703619417f3faa..af49c912558e935976eb8387c362705cd1db4246 100644 (file)
@@ -220,5 +220,6 @@ void frmw_to_linux(u8 *buff, u32 size, u32 pkt_offset);
 void linux_wlan_mac_indicate(int flag);
 void linux_wlan_rx_complete(void);
 void linux_wlan_dbg(u8 *buff);
+int linux_wlan_lock_timeout(void *vp, u32 timeout);
 
 #endif
index aa20f433b01137334d590ce951fe4e02b47fdcee..963973be13701e1eb2d82c00953b0742e4128777 100644 (file)
@@ -30,7 +30,6 @@ typedef struct {
        /**
         *      input interface functions
         **/
-       wilc_wlan_os_func_t os_func;
        wilc_wlan_io_func_t io_func;
 
        /**
@@ -223,8 +222,8 @@ static int wilc_wlan_txq_add_to_head(struct txq_entry_t *tqe)
 {
        wilc_wlan_dev_t *p = (wilc_wlan_dev_t *)&g_wlan;
        unsigned long flags;
-       if (p->os_func.os_wait(&g_linux_wlan->txq_add_to_head_cs,
-                              CFG_PKTS_TIMEOUT))
+       if (linux_wlan_lock_timeout(&g_linux_wlan->txq_add_to_head_cs,
+                                   CFG_PKTS_TIMEOUT))
                return -1;
 
        spin_lock_irqsave(&g_linux_wlan->txq_spinlock, flags);
@@ -452,7 +451,7 @@ static int wilc_wlan_txq_filter_dup_tcp_ack(void)
 
        while (Dropped > 0) {
                /*consume the semaphore count of the removed packet*/
-               p->os_func.os_wait(&g_linux_wlan->txq_event, 1);
+               linux_wlan_lock_timeout(&g_linux_wlan->txq_event, 1);
                Dropped--;
        }
 
@@ -839,8 +838,8 @@ static int wilc_wlan_handle_txq(u32 *pu32TxqCount)
                if (p->quit)
                        break;
 
-               p->os_func.os_wait(&g_linux_wlan->txq_add_to_head_cs,
-                                  CFG_PKTS_TIMEOUT);
+               linux_wlan_lock_timeout(&g_linux_wlan->txq_add_to_head_cs,
+                                       CFG_PKTS_TIMEOUT);
 #ifdef TCP_ACK_FILTER
                wilc_wlan_txq_filter_dup_tcp_ack();
 #endif
@@ -1777,8 +1776,8 @@ static int wilc_wlan_cfg_set(int start, u32 wid, u8 *buffer, u32 buffer_size, in
                if (wilc_wlan_cfg_commit(WILC_CFG_SET, drvHandler))
                        ret_size = 0;
 
-               if (p->os_func.os_wait(&g_linux_wlan->cfg_event,
-                                      CFG_PKTS_TIMEOUT)) {
+               if (linux_wlan_lock_timeout(&g_linux_wlan->cfg_event,
+                                           CFG_PKTS_TIMEOUT)) {
                        PRINT_D(TX_DBG, "Set Timed Out\n");
                        ret_size = 0;
                }
@@ -1815,8 +1814,8 @@ static int wilc_wlan_cfg_get(int start, u32 wid, int commit, u32 drvHandler)
                        ret_size = 0;
 
 
-               if (p->os_func.os_wait(&g_linux_wlan->cfg_event,
-                                      CFG_PKTS_TIMEOUT)) {
+               if (linux_wlan_lock_timeout(&g_linux_wlan->cfg_event,
+                                           CFG_PKTS_TIMEOUT)) {
                        PRINT_D(TX_DBG, "Get Timed Out\n");
                        ret_size = 0;
                }
@@ -1963,7 +1962,6 @@ int wilc_wlan_init(wilc_wlan_inp_t *inp, wilc_wlan_oup_t *oup)
        /**
         *      store the input
         **/
-       memcpy((void *)&g_wlan.os_func, (void *)&inp->os_func, sizeof(wilc_wlan_os_func_t));
        memcpy((void *)&g_wlan.io_func, (void *)&inp->io_func, sizeof(wilc_wlan_io_func_t));
        /***
         *      host interface init
index 0a377e107fa9f093a0ff2a5e8a291535456b7f11..2d6e0cef20d677d9c3b097abbd2c8a06dc14c51f 100644 (file)
@@ -74,10 +74,6 @@ typedef struct {
        u32 block_size;
 } sdio_cmd53_t;
 
-typedef struct {
-       int (*os_wait)(void *, u32);
-} wilc_wlan_os_func_t;
-
 typedef struct {
        int io_type;
        int (*io_init)(void *);
@@ -111,7 +107,6 @@ typedef struct {
 
 typedef struct {
        wilc_wlan_os_context_t os_context;
-       wilc_wlan_os_func_t os_func;
        wilc_wlan_io_func_t io_func;
 } wilc_wlan_inp_t;