Change if logic to handle invalid case for operation_mode
at first avoiding an 'else' path.

Signed-off-by: Sergio Paracuellos <sergio.paracuel...@gmail.com>
---
 drivers/staging/ks7010/ks_wlan_net.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/ks7010/ks_wlan_net.c 
b/drivers/staging/ks7010/ks_wlan_net.c
index 6625d57..1387783 100644
--- a/drivers/staging/ks7010/ks_wlan_net.c
+++ b/drivers/staging/ks7010/ks_wlan_net.c
@@ -1042,10 +1042,9 @@ static int ks_wlan_set_power(struct net_device *dev,
        if (vwrq->power.disabled) {
                priv->reg.power_mgmt = POWER_MGMT_ACTIVE;
        } else {
-               if (priv->reg.operation_mode == MODE_INFRASTRUCTURE)
-                       priv->reg.power_mgmt = POWER_MGMT_SAVE1;
-               else
+               if (priv->reg.operation_mode != MODE_INFRASTRUCTURE)
                        return -EINVAL;
+               priv->reg.power_mgmt = POWER_MGMT_SAVE1;
        }
 
        hostif_sme_enqueue(priv, SME_POW_MNGMT_REQUEST);
-- 
2.7.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to