staging: wilc1000: fix line over 80 char in cfg_scan_result()
authorAjay Singh <ajay.kathat@microchip.com>
Wed, 7 Mar 2018 01:32:30 +0000 (07:02 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 14 Mar 2018 11:59:04 +0000 (12:59 +0100)
Refactor cfg_scan_result() API to avoid 'line over 80 chars' issue
reported by checkpatch.pl script.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c

index 9da686b7aa8f034354626024f3c1e5d949d352a2..5395648ae66d07987ca10945cd49004e840696b0 100644 (file)
@@ -374,91 +374,97 @@ static void cfg_scan_result(enum scan_event scan_event,
        struct cfg80211_bss *bss = NULL;
 
        priv = user_void;
-       if (priv->cfg_scanning) {
-               if (scan_event == SCAN_EVENT_NETWORK_FOUND) {
-                       wiphy = priv->dev->ieee80211_ptr->wiphy;
+       if (!priv->cfg_scanning)
+               return;
+
+       if (scan_event == SCAN_EVENT_NETWORK_FOUND) {
+               wiphy = priv->dev->ieee80211_ptr->wiphy;
+
+               if (!wiphy || !network_info)
+                       return;
 
-                       if (!wiphy)
+               if (wiphy->signal_type == CFG80211_SIGNAL_TYPE_UNSPEC &&
+                   (((s32)network_info->rssi * 100) < 0 ||
+                   ((s32)network_info->rssi * 100) > 100))
+                       return;
+
+               s32Freq = ieee80211_channel_to_frequency((s32)network_info->ch,
+                                                        NL80211_BAND_2GHZ);
+               channel = ieee80211_get_channel(wiphy, s32Freq);
+
+               if (!channel)
+                       return;
+
+               if (network_info->new_network) {
+                       if (priv->rcvd_ch_cnt >= MAX_NUM_SCANNED_NETWORKS)
                                return;
 
-                       if (wiphy->signal_type == CFG80211_SIGNAL_TYPE_UNSPEC &&
-                           (((s32)network_info->rssi * 100) < 0 ||
-                           ((s32)network_info->rssi * 100) > 100))
+                       priv->rcvd_ch_cnt++;
+
+                       add_network_to_shadow(network_info, priv, join_params);
+
+                       if (memcmp("DIRECT-", network_info->ssid, 7))
                                return;
 
-                       if (network_info) {
-                               s32Freq = ieee80211_channel_to_frequency((s32)network_info->ch, NL80211_BAND_2GHZ);
-                               channel = ieee80211_get_channel(wiphy, s32Freq);
-
-                               if (!channel)
-                                       return;
-
-                               if (network_info->new_network) {
-                                       if (priv->rcvd_ch_cnt < MAX_NUM_SCANNED_NETWORKS) {
-                                               priv->rcvd_ch_cnt++;
-
-                                               add_network_to_shadow(network_info, priv, join_params);
-
-                                               if (!(memcmp("DIRECT-", network_info->ssid, 7))) {
-                                                       bss = cfg80211_inform_bss(wiphy,
-                                                                                 channel,
-                                                                                 CFG80211_BSS_FTYPE_UNKNOWN,
-                                                                                 network_info->bssid,
-                                                                                 network_info->tsf_hi,
-                                                                                 network_info->cap_info,
-                                                                                 network_info->beacon_period,
-                                                                                 (const u8 *)network_info->ies,
-                                                                                 (size_t)network_info->ies_len,
-                                                                                 (s32)network_info->rssi * 100,
-                                                                                 GFP_KERNEL);
-                                                       cfg80211_put_bss(wiphy, bss);
-                                               }
-                                       }
-                               } else {
-                                       u32 i;
+                       bss = cfg80211_inform_bss(wiphy,
+                                                 channel,
+                                                 CFG80211_BSS_FTYPE_UNKNOWN,
+                                                 network_info->bssid,
+                                                 network_info->tsf_hi,
+                                                 network_info->cap_info,
+                                                 network_info->beacon_period,
+                                                 (const u8 *)network_info->ies,
+                                                 (size_t)network_info->ies_len,
+                                                 (s32)network_info->rssi * 100,
+                                                 GFP_KERNEL);
+                       cfg80211_put_bss(wiphy, bss);
+               } else {
+                       u32 i;
 
-                                       for (i = 0; i < priv->rcvd_ch_cnt; i++) {
-                                               if (memcmp(last_scanned_shadow[i].bssid, network_info->bssid, 6) == 0) {
-                                                       last_scanned_shadow[i].rssi = network_info->rssi;
-                                                       last_scanned_shadow[i].time_scan = jiffies;
-                                                       break;
-                                               }
-                                       }
-                               }
+                       for (i = 0; i < priv->rcvd_ch_cnt; i++) {
+                               if (memcmp(last_scanned_shadow[i].bssid,
+                                          network_info->bssid, 6) == 0)
+                                       break;
                        }
-               } else if (scan_event == SCAN_EVENT_DONE) {
-                       refresh_scan(priv, false);
 
-                       mutex_lock(&priv->scan_req_lock);
+                       if (i >= priv->rcvd_ch_cnt)
+                               return;
 
-                       if (priv->scan_req) {
-                               struct cfg80211_scan_info info = {
-                                       .aborted = false,
-                               };
+                       last_scanned_shadow[i].rssi = network_info->rssi;
+                       last_scanned_shadow[i].time_scan = jiffies;
+               }
+       } else if (scan_event == SCAN_EVENT_DONE) {
+               refresh_scan(priv, false);
 
-                               cfg80211_scan_done(priv->scan_req, &info);
-                               priv->rcvd_ch_cnt = 0;
-                               priv->cfg_scanning = false;
-                               priv->scan_req = NULL;
-                       }
-                       mutex_unlock(&priv->scan_req_lock);
-               } else if (scan_event == SCAN_EVENT_ABORTED) {
-                       mutex_lock(&priv->scan_req_lock);
+               mutex_lock(&priv->scan_req_lock);
 
-                       if (priv->scan_req) {
-                               struct cfg80211_scan_info info = {
-                                       .aborted = false,
-                               };
+               if (priv->scan_req) {
+                       struct cfg80211_scan_info info = {
+                               .aborted = false,
+                       };
 
-                               update_scan_time();
-                               refresh_scan(priv, false);
+                       cfg80211_scan_done(priv->scan_req, &info);
+                       priv->rcvd_ch_cnt = 0;
+                       priv->cfg_scanning = false;
+                       priv->scan_req = NULL;
+               }
+               mutex_unlock(&priv->scan_req_lock);
+       } else if (scan_event == SCAN_EVENT_ABORTED) {
+               mutex_lock(&priv->scan_req_lock);
 
-                               cfg80211_scan_done(priv->scan_req, &info);
-                               priv->cfg_scanning = false;
-                               priv->scan_req = NULL;
-                       }
-                       mutex_unlock(&priv->scan_req_lock);
+               if (priv->scan_req) {
+                       struct cfg80211_scan_info info = {
+                               .aborted = false,
+                       };
+
+                       update_scan_time();
+                       refresh_scan(priv, false);
+
+                       cfg80211_scan_done(priv->scan_req, &info);
+                       priv->cfg_scanning = false;
+                       priv->scan_req = NULL;
                }
+               mutex_unlock(&priv->scan_req_lock);
        }
 }