linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH FIX] ssb: extend fix for PCI related silent reboots to all chipsets
@ 2015-05-13  5:36 Rafał Miłecki
  2015-05-20 13:38 ` Kalle Valo
  0 siblings, 1 reply; 2+ messages in thread
From: Rafał Miłecki @ 2015-05-13  5:36 UTC (permalink / raw)
  To: Kalle Valo, linux-wireless
  Cc: Hauke Mehrtens, Daniel Gimpelevich, Rafał Miłecki

Recent fix for BCM4704 reboots has to be extended as the same problem
affects Linksys WRT350N v1 (BCM4705).

Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
Reported-by: Daniel Gimpelevich <daniel@gimpelevich.san-francisco.ca.us>
---
Kalle: it's quite trivial, you may consider taking it for 4.1.
---
 drivers/ssb/driver_pcicore.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/ssb/driver_pcicore.c b/drivers/ssb/driver_pcicore.c
index 15a7ee3..5fe1c22 100644
--- a/drivers/ssb/driver_pcicore.c
+++ b/drivers/ssb/driver_pcicore.c
@@ -359,12 +359,13 @@ static void ssb_pcicore_init_hostmode(struct ssb_pcicore *pc)
 
 	/*
 	 * Accessing PCI config without a proper delay after devices reset (not
-	 * GPIO reset) was causing reboots on WRT300N v1.0.
+	 * GPIO reset) was causing reboots on WRT300N v1.0 (BCM4704).
 	 * Tested delay 850 us lowered reboot chance to 50-80%, 1000 us fixed it
 	 * completely. Flushing all writes was also tested but with no luck.
+	 * The same problem was reported for WRT350N v1 (BCM4705), so we just
+	 * sleep here unconditionally.
 	 */
-	if (pc->dev->bus->chip_id == 0x4704)
-		usleep_range(1000, 2000);
+	usleep_range(1000, 2000);
 
 	/* Enable PCI bridge BAR0 prefetch and burst */
 	val = PCI_COMMAND_MASTER | PCI_COMMAND_MEMORY;
-- 
1.8.4.5


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH FIX] ssb: extend fix for PCI related silent reboots to all chipsets
  2015-05-13  5:36 [PATCH FIX] ssb: extend fix for PCI related silent reboots to all chipsets Rafał Miłecki
@ 2015-05-20 13:38 ` Kalle Valo
  0 siblings, 0 replies; 2+ messages in thread
From: Kalle Valo @ 2015-05-20 13:38 UTC (permalink / raw)
  To: Rafał Miłecki
  Cc: linux-wireless, Hauke Mehrtens, Daniel Gimpelevich

Rafał Miłecki <zajec5@gmail.com> writes:

> Recent fix for BCM4704 reboots has to be extended as the same problem
> affects Linksys WRT350N v1 (BCM4705).
>
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> Reported-by: Daniel Gimpelevich <daniel@gimpelevich.san-francisco.ca.us>
> ---
> Kalle: it's quite trivial, you may consider taking it for 4.1.

Thanks, manually applied to wireless-drivers.git.

-- 
Kalle Valo

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-05-20 13:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-13  5:36 [PATCH FIX] ssb: extend fix for PCI related silent reboots to all chipsets Rafał Miłecki
2015-05-20 13:38 ` Kalle Valo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).