From: Luca Coelho <luciano.coe...@intel.com>

We return in the if block, so it's unnecessary to have an else
statement.  Remove it.

Signed-off-by: Luca Coelho <luciano.coe...@intel.com>
---
 drivers/net/wireless/intel/iwlwifi/mvm/fw.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/fw.c 
b/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
index 7afdf0c1b1e9..aa644b82e93c 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
@@ -512,9 +512,10 @@ static int iwl_send_phy_cfg_cmd(struct iwl_mvm *mvm)
        enum iwl_ucode_type ucode_type = mvm->fwrt.cur_fw_img;
 
        if (iwl_mvm_has_unified_ucode(mvm) &&
-           !mvm->trans->cfg->tx_with_siso_diversity) {
+           !mvm->trans->cfg->tx_with_siso_diversity)
                return 0;
-       } else if (mvm->trans->cfg->tx_with_siso_diversity) {
+
+       if (mvm->trans->cfg->tx_with_siso_diversity) {
                /*
                 * TODO: currently we don't set the antenna but letting the NIC
                 * to decide which antenna to use. This should come from BIOS.
-- 
2.23.0

Reply via email to