diff options
author | Raja Mani <rmani@qca.qualcomm.com> | 2012-01-30 06:43:12 -0500 |
---|---|---|
committer | Kalle Valo <kvalo@qca.qualcomm.com> | 2012-01-30 14:22:55 -0500 |
commit | 3c411a434414f4320ce926bee3733e34ba0ea407 (patch) | |
tree | 272953df5a51fb4e152813667882cfe7c527f36b | |
parent | 081c7a84e969453716e2a7bd315417067c3643ad (diff) |
ath6kl: Return a proper error code when not in connected state
Error code ENOTCONN is more suitable than EINVAL to report
when the driver is not in connected state in ath6kl_wow_suspend().
I found this during code review.
Signed-off-by: Raja Mani <rmani@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
-rw-r--r-- | drivers/net/wireless/ath/ath6kl/cfg80211.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ath6kl/cfg80211.c b/drivers/net/wireless/ath/ath6kl/cfg80211.c index 1902e2fa1a68..5934e40ee55d 100644 --- a/drivers/net/wireless/ath/ath6kl/cfg80211.c +++ b/drivers/net/wireless/ath/ath6kl/cfg80211.c | |||
@@ -1916,7 +1916,7 @@ static int ath6kl_wow_suspend(struct ath6kl *ar, struct cfg80211_wowlan *wow) | |||
1916 | return -EIO; | 1916 | return -EIO; |
1917 | 1917 | ||
1918 | if (!test_bit(CONNECTED, &vif->flags)) | 1918 | if (!test_bit(CONNECTED, &vif->flags)) |
1919 | return -EINVAL; | 1919 | return -ENOTCONN; |
1920 | 1920 | ||
1921 | if (wow && (wow->n_patterns > WOW_MAX_FILTERS_PER_LIST)) | 1921 | if (wow && (wow->n_patterns > WOW_MAX_FILTERS_PER_LIST)) |
1922 | return -EINVAL; | 1922 | return -EINVAL; |