summaryrefslogtreecommitdiffstats
path: root/drivers/net/wireless/broadcom/b43/phy_common.c
diff options
context:
space:
mode:
authorLarry Finger <Larry.Finger@lwfinger.net>2018-11-19 19:01:24 +0100
committerKalle Valo <kvalo@codeaurora.org>2018-11-29 16:30:49 +0100
commit8ea3819c0bbef57a51d8abe579e211033e861677 (patch)
tree5851ebd19460bc38d848e10281c4d67dbd09b4a8 /drivers/net/wireless/broadcom/b43/phy_common.c
parentbrcmsmac: Use cordic-related macros from common cordic library (diff)
downloadlinux-8ea3819c0bbef57a51d8abe579e211033e861677.tar.xz
linux-8ea3819c0bbef57a51d8abe579e211033e861677.zip
b43: Fix error in cordic routine
The cordic routine for calculating sines and cosines that was added in commit 6f98e62a9f1b ("b43: update cordic code to match current specs") contains an error whereby a quantity declared u32 can in fact go negative. This problem was detected by Priit Laes who is switching b43 to use the routine in the library functions of the kernel. Fixes: 986504540306 ("b43: make cordic common (LP-PHY and N-PHY need it)") Reported-by: Priit Laes <plaes@plaes.org> Cc: Rafał Miłecki <zajec5@gmail.com> Cc: Stable <stable@vger.kernel.org> # 2.6.34 Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by: Priit Laes <plaes@plaes.org> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Diffstat (limited to 'drivers/net/wireless/broadcom/b43/phy_common.c')
-rw-r--r--drivers/net/wireless/broadcom/b43/phy_common.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/broadcom/b43/phy_common.c b/drivers/net/wireless/broadcom/b43/phy_common.c
index 85f2ca989565..ef3ffa5ad466 100644
--- a/drivers/net/wireless/broadcom/b43/phy_common.c
+++ b/drivers/net/wireless/broadcom/b43/phy_common.c
@@ -616,7 +616,7 @@ struct b43_c32 b43_cordic(int theta)
u8 i;
s32 tmp;
s8 signx = 1;
- u32 angle = 0;
+ s32 angle = 0;
struct b43_c32 ret = { .i = 39797, .q = 0, };
while (theta > (180 << 16))