staging: ks7010: fix checkpatch BRACES

Checkpatch emits CHECK: Unbalanced braces around else
statement. Statements in question are single statements so we do not
need braces. Checkpatch also warns about multiple line dereference for
this code.

Fix if/else/else if statement use of braces. Fix function argument layout
at the same time since it is the same statement.

Signed-off-by: Tobin C. Harding <me@tobin.cc>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Tobin C. Harding 2017-03-21 13:37:06 +11:00 committed by Greg Kroah-Hartman
parent 44c3cd5e04
commit 5a21405ddb
1 changed files with 9 additions and 13 deletions

View File

@ -2456,19 +2456,15 @@ void hostif_sme_execute(struct ks_wlan_private *priv, int event)
hostif_phy_information_request(priv);
break;
case SME_MIC_FAILURE_REQUEST:
if (priv->wpa.mic_failure.failure == 1) {
hostif_mic_failure_request(priv,
priv->wpa.mic_failure.
failure - 1, 0);
} else if (priv->wpa.mic_failure.failure == 2) {
hostif_mic_failure_request(priv,
priv->wpa.mic_failure.
failure - 1,
priv->wpa.mic_failure.
counter);
} else
DPRINTK(4,
"SME_MIC_FAILURE_REQUEST: failure count=%u error?\n",
if (priv->wpa.mic_failure.failure == 1)
hostif_mic_failure_request(
priv, priv->wpa.mic_failure.failure - 1, 0);
else if (priv->wpa.mic_failure.failure == 2)
hostif_mic_failure_request(
priv, priv->wpa.mic_failure.failure - 1,
priv->wpa.mic_failure.counter);
else
DPRINTK(4, "SME_MIC_FAILURE_REQUEST: failure count=%u error?\n",
priv->wpa.mic_failure.failure);
break;
case SME_MIC_FAILURE_CONFIRM: