From: Ajay Singh <ajay.kat...@microchip.com>

Clubbed mac status constants inside the enum constant and also added
'WILC_' prefix to have better namespace.

Signed-off-by: Ajay Singh <ajay.kat...@microchip.com>
---
 drivers/staging/wilc1000/host_interface.c         | 15 ++++++++-------
 drivers/staging/wilc1000/linux_wlan.c             |  4 ++--
 drivers/staging/wilc1000/wilc_wfi_cfgoperations.c |  2 +-
 drivers/staging/wilc1000/wilc_wlan_if.h           |  8 +++++---
 4 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/drivers/staging/wilc1000/host_interface.c 
b/drivers/staging/wilc1000/host_interface.c
index c200d90..46b1a00 100644
--- a/drivers/staging/wilc1000/host_interface.c
+++ b/drivers/staging/wilc1000/host_interface.c
@@ -797,7 +797,8 @@ static void handle_connect(struct work_struct *work)
                        }
 
                        conn_attr->result(CONN_DISCONN_EVENT_CONN_RESP,
-                                         &conn_info, MAC_STATUS_DISCONNECTED,
+                                         &conn_info,
+                                         WILC_MAC_STATUS_DISCONNECTED,
                                          NULL, conn_attr->arg);
                        hif_drv->hif_state = HOST_IF_IDLE;
                        kfree(conn_info.req_ies);
@@ -857,7 +858,7 @@ static void handle_connect_timeout(struct work_struct *work)
 
                hif_drv->usr_conn_req.conn_result(CONN_DISCONN_EVENT_CONN_RESP,
                                                  &info,
-                                                 MAC_STATUS_DISCONNECTED,
+                                                 WILC_MAC_STATUS_DISCONNECTED,
                                                  NULL,
                                                  hif_drv->usr_conn_req.arg);
 
@@ -1286,7 +1287,7 @@ static inline void host_int_parse_assoc_resp_info(struct 
wilc_vif *vif,
 
        memset(&conn_info, 0, sizeof(struct connect_info));
 
-       if (mac_status == MAC_STATUS_CONNECTED) {
+       if (mac_status == WILC_MAC_STATUS_CONNECTED) {
                u32 assoc_resp_info_len;
 
                memset(hif_drv->assoc_resp, 0, MAX_ASSOC_RESP_FRAME_SIZE);
@@ -1311,7 +1312,7 @@ static inline void host_int_parse_assoc_resp_info(struct 
wilc_vif *vif,
        if (hif_drv->usr_conn_req.bssid) {
                memcpy(conn_info.bssid, hif_drv->usr_conn_req.bssid, 6);
 
-               if (mac_status == MAC_STATUS_CONNECTED &&
+               if (mac_status == WILC_MAC_STATUS_CONNECTED &&
                    conn_info.status == WLAN_STATUS_SUCCESS) {
                        memcpy(hif_drv->assoc_bssid,
                               hif_drv->usr_conn_req.bssid, ETH_ALEN);
@@ -1331,7 +1332,7 @@ static inline void host_int_parse_assoc_resp_info(struct 
wilc_vif *vif,
                                          &conn_info, mac_status, NULL,
                                          hif_drv->usr_conn_req.arg);
 
-       if (mac_status == MAC_STATUS_CONNECTED &&
+       if (mac_status == WILC_MAC_STATUS_CONNECTED &&
            conn_info.status == WLAN_STATUS_SUCCESS) {
                wilc_set_power_mgmt(vif, 0, 0);
 
@@ -1423,10 +1424,10 @@ static void handle_rcvd_gnrl_async_info(struct 
work_struct *work)
                mac_status  = rcvd_info->buffer[7];
                if (hif_drv->hif_state == HOST_IF_WAITING_CONN_RESP) {
                        host_int_parse_assoc_resp_info(vif, mac_status);
-               } else if ((mac_status == MAC_STATUS_DISCONNECTED) &&
+               } else if ((mac_status == WILC_MAC_STATUS_DISCONNECTED) &&
                           (hif_drv->hif_state == HOST_IF_CONNECTED)) {
                        host_int_handle_disconnect(vif);
-               } else if ((mac_status == MAC_STATUS_DISCONNECTED) &&
+               } else if ((mac_status == WILC_MAC_STATUS_DISCONNECTED) &&
                           (hif_drv->usr_scan_req.scan_result)) {
                        del_timer(&hif_drv->scan_timer);
                        if (hif_drv->usr_scan_req.scan_result)
diff --git a/drivers/staging/wilc1000/linux_wlan.c 
b/drivers/staging/wilc1000/linux_wlan.c
index 76c9012..274c4f6 100644
--- a/drivers/staging/wilc1000/linux_wlan.c
+++ b/drivers/staging/wilc1000/linux_wlan.c
@@ -162,7 +162,7 @@ void wilc_mac_indicate(struct wilc *wilc)
        s8 status;
 
        wilc_wlan_cfg_get_val(wilc, WID_STATUS, &status, 1);
-       if (wilc->mac_status == MAC_STATUS_INIT) {
+       if (wilc->mac_status == WILC_MAC_STATUS_INIT) {
                wilc->mac_status = status;
                complete(&wilc->sync_event);
        } else {
@@ -624,7 +624,7 @@ static int wilc_wlan_initialize(struct net_device *dev, 
struct wilc_vif *vif)
        struct wilc *wl = vif->wilc;
 
        if (!wl->initialized) {
-               wl->mac_status = MAC_STATUS_INIT;
+               wl->mac_status = WILC_MAC_STATUS_INIT;
                wl->close = 0;
 
                wlan_init_locks(dev);
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c 
b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
index 7b753c1..9038f8c 100644
--- a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
+++ b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
@@ -470,7 +470,7 @@ static void cfg_connect_result(enum conn_event 
conn_disconn_evt,
 
                connect_status = conn_info->status;
 
-               if (mac_status == MAC_STATUS_DISCONNECTED &&
+               if (mac_status == WILC_MAC_STATUS_DISCONNECTED &&
                    conn_info->status == WLAN_STATUS_SUCCESS) {
                        connect_status = WLAN_STATUS_UNSPECIFIED_FAILURE;
                        wilc_wlan_set_bssid(priv->dev, null_bssid,
diff --git a/drivers/staging/wilc1000/wilc_wlan_if.h 
b/drivers/staging/wilc1000/wilc_wlan_if.h
index dc40743..923b07c 100644
--- a/drivers/staging/wilc1000/wilc_wlan_if.h
+++ b/drivers/staging/wilc1000/wilc_wlan_if.h
@@ -24,9 +24,11 @@
  *
  ********************************************/
 
-#define MAC_STATUS_INIT                        -1
-#define MAC_STATUS_CONNECTED           1
-#define MAC_STATUS_DISCONNECTED                0
+enum {
+       WILC_MAC_STATUS_INIT = -1,
+       WILC_MAC_STATUS_DISCONNECTED = 0,
+       WILC_MAC_STATUS_CONNECTED = 1
+};
 
 struct tx_complete_data {
        int size;
-- 
2.7.4

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

Reply via email to