ar9170: fix bug in iq-auto calibration value calculation
authorChristian Lamparter <chunkeey@googlemail.com>
Tue, 29 Sep 2009 18:47:25 +0000 (20:47 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 29 Sep 2009 21:25:16 +0000 (17:25 -0400)
commite16c1bb67a0010b5bad26ddc3e691655fd7456e3
treed051731baf91baced7b3811ceab61ec94b005912
parent78bd6bbf3c12f49e2cb6897bfd73a62325ed3aaa
ar9170: fix bug in iq-auto calibration value calculation

This patch fixes a embarrassing bug which was introduced by:
"[PATCH] ar9170: implement frequency calibration for one-stage/openfw"

The phy_data variable initialization has to done outside the for-loop
scope. This is because the for-loop uses u32 phy_data variable more
like a 4-byte field. But in each run only a single byte is calculated.
Therefore phy_data content needs to stay the same for at least 3 more
iterations, before the complete set can be uploaded.

Reported-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Christian Lamparter <chunkeey@googlemail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ar9170/phy.c