From: John Crispin Date: Wed, 4 Feb 2015 06:09:51 +0000 (+0000) Subject: ath5k: fix spontaneus AR5312 freezes X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=ee445d7cb2c0f5d9d59bc320a52e218a60df1265;p=openwrt%2Fsvn-archive%2Fopenwrt.git ath5k: fix spontaneus AR5312 freezes Signed-off-by: Sergey Ryazanov SVN-Revision: 44252 --- diff --git a/package/kernel/mac80211/patches/330-ath5k-fix-spontaneus-AR5312-freezes.patch b/package/kernel/mac80211/patches/330-ath5k-fix-spontaneus-AR5312-freezes.patch new file mode 100644 index 0000000000..aacadbaf2f --- /dev/null +++ b/package/kernel/mac80211/patches/330-ath5k-fix-spontaneus-AR5312-freezes.patch @@ -0,0 +1,45 @@ +From: Sergey Ryazanov +Date: Tue, 3 Feb 2015 22:37:26 +0300 +Subject: [PATCH] ath5k: fix spontaneus AR5312 freezes + +Sometimes while CPU have some load and ath5k doing the wireless +interface reset the whole WiSoC completely freezes. Set of tests shows +that using atomic delay function while we wait interface reset helps to +avoid such freezes. + +The easiest way to reproduce this issue: create a station interface, +start continous scan with wpa_supplicant and load CPU by something. Or +just create multiple station interfaces and put them all in continous +scan. + +This patch partially reverts the commit 1846ac3dbec0 ("ath5k: Use +usleep_range where possible"), which replaces initial udelay() +by usleep_range(). + +I do not know actual source of this issue, but all looks like that HW +freeze is caused by transaction on internal SoC bus, while wireless +block is in reset state. + +Also I should note that I do not know how many chips are affected, but I +did not see this issue with chips, other than AR5312. + +CC: Jiri Slaby +CC: Nick Kossifidis +CC: Luis R. Rodriguez +Fixes: 1846ac3dbec0 ("ath5k: Use usleep_range where possible") +Reported-by: Christophe Prevotaux +Tested-by: Christophe Prevotaux +Tested-by: Eric Bree +Signed-off-by: Sergey Ryazanov +--- +--- a/drivers/net/wireless/ath/ath5k/reset.c ++++ b/drivers/net/wireless/ath/ath5k/reset.c +@@ -478,7 +478,7 @@ ath5k_hw_wisoc_reset(struct ath5k_hw *ah + regval = ioread32(reg); + iowrite32(regval | val, reg); + regval = ioread32(reg); +- usleep_range(100, 150); ++ udelay(100); /* NB: should be atomic */ + + /* Bring BB/MAC out of reset */ + iowrite32(regval & ~val, reg);