brcm80211: avoid code duplication on set tx power
authorLuis R. Rodriguez <mcgrof@qca.qualcomm.com>
Mon, 28 Nov 2011 21:38:49 +0000 (16:38 -0500)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 30 Nov 2011 20:08:29 +0000 (15:08 -0500)
Both cases are doing the same so treat the switch cases
for both as an "or".

Signed-off-by: Luis R. Rodriguez <mcgrof@qca.qualcomm.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c

index fc98981..f23b0c3 100644 (file)
@@ -1447,12 +1447,6 @@ brcmf_cfg80211_set_tx_power(struct wiphy *wiphy,
        case NL80211_TX_POWER_AUTOMATIC:
                break;
        case NL80211_TX_POWER_LIMITED:
-               if (dbm < 0) {
-                       WL_ERR("TX_POWER_LIMITED - dbm is negative\n");
-                       err = -EINVAL;
-                       goto done;
-               }
-               break;
        case NL80211_TX_POWER_FIXED:
                if (dbm < 0) {
                        WL_ERR("TX_POWER_FIXED - dbm is negative\n");