staging: ks7010: refactor ks_wlan_set_phy_type function
Handle invalid values first and assign good ones at the end if it is the case. This makes code simplier. Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
7f3c8bb5bc
commit
81255d867c
|
@ -1960,17 +1960,14 @@ static int ks_wlan_set_phy_type(struct net_device *dev,
|
|||
|
||||
if (priv->sleep_mode == SLP_SLEEP)
|
||||
return -EPERM;
|
||||
/* for SLEEP MODE */
|
||||
if (*uwrq == D_11B_ONLY_MODE) { /* 0 */
|
||||
priv->reg.phy_type = D_11B_ONLY_MODE;
|
||||
} else if (*uwrq == D_11G_ONLY_MODE) { /* 1 */
|
||||
priv->reg.phy_type = D_11G_ONLY_MODE;
|
||||
} else if (*uwrq == D_11BG_COMPATIBLE_MODE) { /* 2 */
|
||||
priv->reg.phy_type = D_11BG_COMPATIBLE_MODE;
|
||||
} else {
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
if (*uwrq != D_11B_ONLY_MODE &&
|
||||
*uwrq != D_11G_ONLY_MODE &&
|
||||
*uwrq != D_11BG_COMPATIBLE_MODE)
|
||||
return -EINVAL;
|
||||
|
||||
/* for SLEEP MODE */
|
||||
priv->reg.phy_type = *uwrq;
|
||||
priv->need_commit |= SME_MODE_SET;
|
||||
return -EINPROGRESS; /* Call commit handler */
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue