diff options
| author | Colin Ian King <colin.king@canonical.com> | 2021-09-07 11:46:58 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-04-15 14:14:48 +0200 |
| commit | 0b85cd1cac71dd7b5b2cb682ad4e10f7b41f489a (patch) | |
| tree | af4d29b088f7492354a5c210baf4268b1db0136e /drivers/net/wireless | |
| parent | 6ca1e695e8ae3397680ecf70791758836e200725 (diff) | |
| download | linux-stable-0b85cd1cac71dd7b5b2cb682ad4e10f7b41f489a.tar.gz linux-stable-0b85cd1cac71dd7b5b2cb682ad4e10f7b41f489a.tar.bz2 linux-stable-0b85cd1cac71dd7b5b2cb682ad4e10f7b41f489a.zip | |
iwlwifi: Fix -EIO error code that is never returned
[ Upstream commit c305c94bdc18e45b5ad1db54da4269f8cbfdff6b ]
Currently the error -EIO is being assinged to variable ret when
the READY_BIT is not set but the function iwlagn_mac_start returns
0 rather than ret. Fix this by returning ret instead of 0.
Addresses-Coverity: ("Unused value")
Fixes: 7335613ae27a ("iwlwifi: move all mac80211 related functions to one place")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20210907104658.14706-1-colin.king@canonical.com
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/net/wireless')
| -rw-r--r-- | drivers/net/wireless/intel/iwlwifi/dvm/mac80211.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/intel/iwlwifi/dvm/mac80211.c b/drivers/net/wireless/intel/iwlwifi/dvm/mac80211.c index 82caae02dd09..f2e0cfa2f4a2 100644 --- a/drivers/net/wireless/intel/iwlwifi/dvm/mac80211.c +++ b/drivers/net/wireless/intel/iwlwifi/dvm/mac80211.c @@ -317,7 +317,7 @@ static int iwlagn_mac_start(struct ieee80211_hw *hw) priv->is_open = 1; IWL_DEBUG_MAC80211(priv, "leave\n"); - return 0; + return ret; } static void iwlagn_mac_stop(struct ieee80211_hw *hw) |