Gábor Stefanik wrote: > I'm getting a kernel oops/panic when unloading rtl8187. Bisect lead me > to commit "rtl8187: Implement TX/RX blink for LED", confirmed by local > backout. > My card is an RTL8187LvB, "Customer ID 0x00" (P5K Premium integrated > WiFi, made by Azurewave). Does this patch fix it? Larry --- Index: wireless-testing/drivers/net/wireless/rtl818x/rtl8187_leds.c =================================================================== --- wireless-testing.orig/drivers/net/wireless/rtl818x/rtl8187_leds.c +++ wireless-testing/drivers/net/wireless/rtl818x/rtl8187_leds.c @@ -212,6 +212,7 @@ void rtl8187_leds_exit(struct ieee80211_ /* turn the LED off before exiting */ queue_delayed_work(dev->workqueue, &priv->led_off, 0); cancel_delayed_work_sync(&priv->led_off); + cancel_delayed_work_sync(&priv->led_on); rtl8187_unregister_led(&priv->led_rx); } #endif /* def CONFIG_RTL8187_LED */
Index: wireless-testing/drivers/net/wireless/rtl818x/rtl8187_leds.c =================================================================== --- wireless-testing.orig/drivers/net/wireless/rtl818x/rtl8187_leds.c +++ wireless-testing/drivers/net/wireless/rtl818x/rtl8187_leds.c @@ -212,6 +212,7 @@ void rtl8187_leds_exit(struct ieee80211_ /* turn the LED off before exiting */ queue_delayed_work(dev->workqueue, &priv->led_off, 0); cancel_delayed_work_sync(&priv->led_off); + cancel_delayed_work_sync(&priv->led_on); rtl8187_unregister_led(&priv->led_rx); } #endif /* def CONFIG_RTL8187_LED */