Browse Source

brcm80211: avoid code duplication on set tx power

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>
Luis R. Rodriguez 13 years ago
parent
commit
e957abb60e
1 changed files with 0 additions and 6 deletions
  1. 0 6
      drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c

+ 0 - 6
drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c

@@ -1447,12 +1447,6 @@ brcmf_cfg80211_set_tx_power(struct wiphy *wiphy,
 	case NL80211_TX_POWER_AUTOMATIC:
 	case NL80211_TX_POWER_AUTOMATIC:
 		break;
 		break;
 	case NL80211_TX_POWER_LIMITED:
 	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:
 	case NL80211_TX_POWER_FIXED:
 		if (dbm < 0) {
 		if (dbm < 0) {
 			WL_ERR("TX_POWER_FIXED - dbm is negative\n");
 			WL_ERR("TX_POWER_FIXED - dbm is negative\n");