diff options
author | Matthias Schiffer <mschiffer@universe-factory.net> | 2016-11-15 12:47:21 -0500 |
---|---|---|
committer | Kalle Valo <kvalo@qca.qualcomm.com> | 2016-11-23 08:58:21 -0500 |
commit | 91851cc7a939039bd401adb6ca3da4402bec1d0c (patch) | |
tree | 84a41c01ac21ec98d331239a591dc55777fe19be /drivers/net/wireless/ath | |
parent | 5a401f36ba3080cded5a65fc671648e7cf285ee4 (diff) |
ath9k: fix ath9k_hw_gpio_get() to return 0 or 1 on success
Commit b2d70d4944c1 ("ath9k: make GPIO API to support both of WMAC and
SOC") refactored ath9k_hw_gpio_get() to support both WMAC and SOC GPIOs,
changing the return on success from 1 to BIT(gpio). This broke some callers
like ath_is_rfkill_set(). This doesn't fix any known bug in mainline at the
moment, but should be fixed anyway.
Instead of fixing all callers, change ath9k_hw_gpio_get() back to only
return 0 or 1.
Fixes: b2d70d4944c1 ("ath9k: make GPIO API to support both of WMAC and SOC")
Cc: <stable@vger.kernel.org> # v4.7+
Signed-off-by: Matthias Schiffer <mschiffer@universe-factory.net>
[kvalo@qca.qualcomm.com: mention that doesn't fix any known bug]
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Diffstat (limited to 'drivers/net/wireless/ath')
-rw-r--r-- | drivers/net/wireless/ath/ath9k/hw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c index 14b13f07cd1f..a35f78be8dec 100644 --- a/drivers/net/wireless/ath/ath9k/hw.c +++ b/drivers/net/wireless/ath/ath9k/hw.c | |||
@@ -2792,7 +2792,7 @@ u32 ath9k_hw_gpio_get(struct ath_hw *ah, u32 gpio) | |||
2792 | WARN_ON(1); | 2792 | WARN_ON(1); |
2793 | } | 2793 | } |
2794 | 2794 | ||
2795 | return val; | 2795 | return !!val; |
2796 | } | 2796 | } |
2797 | EXPORT_SYMBOL(ath9k_hw_gpio_get); | 2797 | EXPORT_SYMBOL(ath9k_hw_gpio_get); |
2798 | 2798 | ||