On Fri, 2011-04-15 at 12:36 +0530, Rajkumar Manoharan wrote: > > > @@ -150,6 +149,10 @@ static void ath9k_deinit_device(struct ath9k_htc_priv *priv) > > > { > > > struct ieee80211_hw *hw = priv->hw; > > > > > > + if (priv->debug.debugfs_phy) { > > > + debugfs_remove_recursive(priv->debug.debugfs_phy); > > > + priv->debug.debugfs_phy = NULL; > > > + } > > > > It would be consistent to have this chunk of code in ath9k_htc_exit_debug(). > > > I think mac80211 handles debugfs_remove_recursive. isn't it? > If so, Shall I remove it? cfg80211 will remove all per wiphy debugfs files, but you might want to remove them before so nobody accesses them while you're uninitialising. johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html