On Wed, Oct 13, 2010 at 10:50:31AM +0530, Rajkumar Manoharan wrote: > On Tue, Oct 12, 2010 at 07:38:03PM +0530, Felix Fietkau wrote: > > + spin_lock(&common->cc_lock); > > ath9k_hw_proc_mib_event(ah); > > + spin_unlock(&common->cc_lock); > > ath9k_hw_set_interrupts(ah, ah->imask); > > } > > > shouldn't we lock ath9k_ani_reset which also restarts ani? Remember syncrhonizing ath9k_ani_reset() with spin_lock requires careful review as ath9k_htc uses mutex lock somewhere in the ani reset path. > > - Rajkumar > > -- > > 1.7.2.2 > > > > -- > > 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 > -- > 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 -- 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