Просмотр исходного кода

wlcore: print stack trace in every recovery

As recovery queuing can now occur from multiple code paths
it's convenient to know what triggered it in all cases
other than an intended recovery which is part of the
switch between single role to multi role.

Signed-off-by: Eyal Shapira <eyal@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
Eyal Shapira 13 лет назад
Родитель
Сommit
680c6055b9
2 измененных файлов с 2 добавлено и 1 удалено
  1. 0 1
      drivers/net/wireless/ti/wlcore/cmd.c
  2. 2 0
      drivers/net/wireless/ti/wlcore/main.c

+ 0 - 1
drivers/net/wireless/ti/wlcore/cmd.c

@@ -124,7 +124,6 @@ int wl1271_cmd_send(struct wl1271 *wl, u16 id, void *buf, size_t len,
 	return 0;
 	return 0;
 
 
 fail:
 fail:
-	WARN_ON(1);
 	wl12xx_queue_recovery_work(wl);
 	wl12xx_queue_recovery_work(wl);
 	return ret;
 	return ret;
 }
 }

+ 2 - 0
drivers/net/wireless/ti/wlcore/main.c

@@ -772,6 +772,8 @@ static void wl1271_fetch_nvs(struct wl1271 *wl)
 
 
 void wl12xx_queue_recovery_work(struct wl1271 *wl)
 void wl12xx_queue_recovery_work(struct wl1271 *wl)
 {
 {
+	WARN_ON(!test_bit(WL1271_FLAG_INTENDED_FW_RECOVERY, &wl->flags));
+
 	/* Avoid a recursive recovery */
 	/* Avoid a recursive recovery */
 	if (!test_and_set_bit(WL1271_FLAG_RECOVERY_IN_PROGRESS, &wl->flags)) {
 	if (!test_and_set_bit(WL1271_FLAG_RECOVERY_IN_PROGRESS, &wl->flags)) {
 		wlcore_disable_interrupts_nosync(wl);
 		wlcore_disable_interrupts_nosync(wl);