[PATCH 73/99] cw1200: fix crash during driver unloading.

Dmitry Tarnyagin dmitry.tarnyagin at stericsson.com
Wed Feb 29 14:15:20 UTC 2012


From: Lukasz Kucharczyk <lukasz.kucharczyk at tieto.com>

Debug utilities were disabled before disabling irq handler.
Irq handler used debug utilities what could lead to crash
sometimes.

ST-Ericsson ID: 398849
ST-Ericsson FOSS-OUT ID: NA

Change-Id: I8a8a54c841bbb7bf539ea150f3fe44e09c0f6c17
Signed-off-by: Lukasz Kucharczyk <lukasz.kucharczyk at tieto.com>
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/44221
Reviewed-by: Bartosz MARKOWSKI <bartosz.markowski at tieto.com>
Tested-by: Bartosz MARKOWSKI <bartosz.markowski at tieto.com>
---
 drivers/staging/cw1200/main.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/cw1200/main.c b/drivers/staging/cw1200/main.c
index 7a2e12d..f6ddac6 100644
--- a/drivers/staging/cw1200/main.c
+++ b/drivers/staging/cw1200/main.c
@@ -428,13 +428,13 @@ void cw1200_unregister_common(struct ieee80211_hw *dev)
 	struct cw1200_common *priv = dev->priv;
 	int i;
 
-	cw1200_debug_release(priv);
-
 	ieee80211_unregister_hw(dev);
 
 	priv->sbus_ops->irq_unsubscribe(priv->sbus_priv);
 	cw1200_unregister_bh(priv);
 
+	cw1200_debug_release(priv);
+
 #ifdef CONFIG_CW1200_LEDS
 	cw1200_unregister_leds(priv);
 #endif /* CONFIG_CW1200_LEDS */
-- 
1.7.8.3



More information about the kernel mailing list