staging: ks7010: refactor code for hostif_sme_sleep_set function
authorSergio Paracuellos <sergio.paracuellos@gmail.com>
Mon, 16 Apr 2018 10:29:29 +0000 (12:29 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 23 Apr 2018 12:29:50 +0000 (14:29 +0200)
This commit refactors code for hostif_sme_sleep_set function. This
function was using a switch-case block to handle only two states
where the action to do for them is the same. Just refactor a bit
to check for return condition at first and doing the common action
after in other case.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ks7010/ks_hostif.c

index 2203362..04688e1 100644 (file)
@@ -2043,19 +2043,13 @@ void hostif_sme_power_mgmt_set(struct ks_wlan_private *priv)
        hostif_power_mgmt_request(priv, mode, wake_up, receive_dtims);
 }
 
-static
-void hostif_sme_sleep_set(struct ks_wlan_private *priv)
+static void hostif_sme_sleep_set(struct ks_wlan_private *priv)
 {
-       switch (priv->sleep_mode) {
-       case SLP_SLEEP:
-               hostif_sleep_request(priv, priv->sleep_mode);
-               break;
-       case SLP_ACTIVE:
-               hostif_sleep_request(priv, priv->sleep_mode);
-               break;
-       default:
-               break;
-       }
+       if (priv->sleep_mode != SLP_SLEEP &&
+           priv->sleep_mode != SLP_ACTIVE)
+               return;
+
+       hostif_sleep_request(priv, priv->sleep_mode);
 }
 
 static