| From 15e14f76f85f4f0eab3b8146e1cd3c58ce272823 Mon Sep 17 00:00:00 2001 |
| From: Lorenzo Bianconi <lorenzo@kernel.org> |
| Date: Sat, 21 Sep 2019 10:44:01 +0200 |
| Subject: mt7601u: fix bbp version check in mt7601u_wait_bbp_ready |
| |
| From: Lorenzo Bianconi <lorenzo@kernel.org> |
| |
| commit 15e14f76f85f4f0eab3b8146e1cd3c58ce272823 upstream. |
| |
| Fix bbp ready check in mt7601u_wait_bbp_ready. The issue is reported by |
| coverity with the following error: |
| |
| Logical vs. bitwise operator |
| The expression's value does not depend on the operands; inadvertent use |
| of the wrong operator is a likely logic error. |
| |
| Addresses-Coverity-ID: 1309441 ("Logical vs. bitwise operator") |
| Fixes: c869f77d6abb ("add mt7601u driver") |
| Acked-by: Jakub Kicinski <kubakici@wp.pl> |
| Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org> |
| Signed-off-by: Kalle Valo <kvalo@codeaurora.org> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/net/wireless/mediatek/mt7601u/phy.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/net/wireless/mediatek/mt7601u/phy.c |
| +++ b/drivers/net/wireless/mediatek/mt7601u/phy.c |
| @@ -221,7 +221,7 @@ int mt7601u_wait_bbp_ready(struct mt7601 |
| |
| do { |
| val = mt7601u_bbp_rr(dev, MT_BBP_REG_VERSION); |
| - if (val && ~val) |
| + if (val && val != 0xff) |
| break; |
| } while (--i); |
| |