staging: ks7010: refactor ks_wlan_set_tx_gain function

This commit refactors ks_wlan_set_rx_gain function to
improve readability:
    - error condition is handling the error to avoid an 'else'
    - ternary operator is used to clean if-else block assignment.

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_wlan_net.c b/drivers/staging/ks7010/ks_wlan_net.c
index 2fbacc56abdd9be3c00af8ae412412aa71fd5655..81889a5b2251253997829900fe80a28c088ce7f2 100644 (file)
--- a/drivers/staging/ks7010/ks_wlan_net.c
+++ b/drivers/staging/ks7010/ks_wlan_net.c
@@ -2290,16 +2290,11 @@ static int ks_wlan_set_tx_gain(struct net_device *dev,
        if (priv->sleep_mode == SLP_SLEEP)
                return -EPERM;
        /* for SLEEP MODE */
-       if (*uwrq >= 0 && *uwrq <= 0xFF)        /* 0-255 */
-               priv->gain.tx_gain = (uint8_t)*uwrq;
-       else
+       if (*uwrq < 0 || *uwrq > 0xFF)
                return -EINVAL;
 
-       if (priv->gain.tx_gain < 0xFF)
-               priv->gain.tx_mode = 1;
-       else
-               priv->gain.tx_mode = 0;
-
+       priv->gain.tx_gain = (uint8_t)*uwrq;
+       priv->gain.tx_mode = (priv->gain.tx_gain < 0xFF) ? 1 : 0;
        hostif_sme_enqueue(priv, SME_SET_GAIN);
        return 0;
 }