staging: ks7010: refactor hostif_sme_power_mgmt_set function

Some minor changes have been done in this function to clean
it a bit:
    - POWER_MGMT_ACTIVE and default case are the same so
      just handle that with the same block of code,
    - POWER_MGMT_SAVE1 replaces if-else with a ternary operator.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c
index 19c8887200f59955589ea98a12fd3c5535dfdc72..fe19243e192baea146389122547c6347e7a2de1a 100644 (file)
--- a/drivers/staging/ks7010/ks_hostif.c
+++ b/drivers/staging/ks7010/ks_hostif.c
@@ -1962,22 +1962,12 @@ void hostif_sme_power_mgmt_set(struct ks_wlan_private *priv)
        unsigned long mode, wake_up, receive_dtims;
 
        switch (priv->reg.power_mgmt) {
-       case POWER_MGMT_ACTIVE:
-               mode = POWER_ACTIVE;
+       case POWER_MGMT_SAVE1:
+               mode = (priv->reg.operation_mode == MODE_INFRASTRUCTURE) ?
+                       POWER_SAVE : POWER_ACTIVE;
                wake_up = 0;
                receive_dtims = 0;
                break;
-       case POWER_MGMT_SAVE1:
-               if (priv->reg.operation_mode == MODE_INFRASTRUCTURE) {
-                       mode = POWER_SAVE;
-                       wake_up = 0;
-                       receive_dtims = 0;
-               } else {
-                       mode = POWER_ACTIVE;
-                       wake_up = 0;
-                       receive_dtims = 0;
-               }
-               break;
        case POWER_MGMT_SAVE2:
                if (priv->reg.operation_mode == MODE_INFRASTRUCTURE) {
                        mode = POWER_SAVE;
@@ -1989,6 +1979,7 @@ void hostif_sme_power_mgmt_set(struct ks_wlan_private *priv)
                        receive_dtims = 0;
                }
                break;
+       case POWER_MGMT_ACTIVE:
        default:
                mode = POWER_ACTIVE;
                wake_up = 0;