diff options
| author | Tobin C. Harding <me@tobin.cc> | 2017-04-27 11:25:24 +1000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-04-28 11:47:26 +0200 |
| commit | e04f2ec6c3266336784919d719b12ab67edaa74a (patch) | |
| tree | 29b2b90a556fac35fa4fcfb6ac6d4dd42ce6f669 /drivers/staging/ks7010 | |
| parent | 2b0d92b265324cfe42839a23cb46677bb0112c2c (diff) | |
staging: ks7010: continue from loop on unmatched mac
Inside loop, code block is guarded with an 'if' statement. Instead of
guarding the block we can invert the 'if' statement conditional and
continue the loop. Doing so allows subsequent code indentation to be
reduced and aids the readability of the code.
Invert 'if' statement conditional, continue loop if new conditional
evaluates to true. Reduce subsequent code indentation level. Do not
change program logic.
Signed-off-by: Tobin C. Harding <me@tobin.cc>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/ks7010')
| -rw-r--r-- | drivers/staging/ks7010/ks_hostif.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index fede3823e95a..b19583cd5d77 100644 --- a/drivers/staging/ks7010/ks_hostif.c +++ b/drivers/staging/ks7010/ks_hostif.c @@ -825,13 +825,13 @@ void hostif_scan_indication(struct ks_wlan_private *priv) if (priv->scan_ind_count) { for (i = 0; i < priv->aplist.size; i++) { /* bssid check */ if (memcmp(ap_info->bssid, - priv->aplist.ap[i].bssid, ETH_ALEN) == 0) { - if (ap_info->frame_type == - FRAME_TYPE_PROBE_RESP) - get_ap_information(priv, ap_info, - &priv->aplist.ap[i]); - return; - } + priv->aplist.ap[i].bssid, ETH_ALEN) != 0) + continue; + + if (ap_info->frame_type == FRAME_TYPE_PROBE_RESP) + get_ap_information(priv, ap_info, + &priv->aplist.ap[i]); + return; } } priv->scan_ind_count++; |