From e04f2ec6c3266336784919d719b12ab67edaa74a Mon Sep 17 00:00:00 2001 From: "Tobin C. Harding" Date: Thu, 27 Apr 2017 11:25:24 +1000 Subject: [PATCH] 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 Signed-off-by: Greg Kroah-Hartman --- drivers/staging/ks7010/ks_hostif.c | 14 +++++++------- 1 file 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++; -- 2.11.0