From: Tobin C. Harding Date: Tue, 21 Mar 2017 02:37:06 +0000 (+1100) Subject: staging: ks7010: fix checkpatch BRACES X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=5a21405ddbd58a4985081b2642837b978a5d7011;p=linux.git 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 Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index db10e168ab3e5..68e26f42a4c89 100644 --- a/drivers/staging/ks7010/ks_hostif.c +++ b/drivers/staging/ks7010/ks_hostif.c @@ -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: