Return-path: Received: from mail-vw0-f46.google.com ([209.85.212.46]:61419 "EHLO mail-vw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750748Ab1KWQIg (ORCPT ); Wed, 23 Nov 2011 11:08:36 -0500 Received: by mail-vw0-f46.google.com with SMTP id fc26so608166vbb.19 for ; Wed, 23 Nov 2011 08:08:36 -0800 (PST) From: "Luis R. Rodriguez" To: linux-wireless@vger.kernel.org Cc: "Luis R. Rodriguez" , Arend van Spriel Subject: [RFC 4/5] brcm80211: avoid code duplication on set tx power Date: Wed, 23 Nov 2011 11:08:32 -0500 Message-Id: <1322064512-16502-1-git-send-email-mcgrof@qca.qualcomm.com> (sfid-20111123_170839_794319_73C63359) Sender: linux-wireless-owner@vger.kernel.org List-ID: Both cases are doing the same so treat the switch cases for both as an "or". Cc: Arend van Spriel Signed-off-by: Luis R. Rodriguez --- .../net/wireless/brcm80211/brcmfmac/wl_cfg80211.c | 6 ------ 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c b/drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c index fc98981..f23b0c3 100644 --- a/drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c +++ b/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: 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"); -- 1.7.4.15.g7811d