On Tue, Sep 14, 2010 at 7:38 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Tue, 2010-09-14 at 07:35 -0700, Luis R. Rodriguez wrote: >> On Tue, Sep 14, 2010 at 7:29 AM, Johannes Berg >> <johannes@xxxxxxxxxxxxxxxx> wrote: >> > On Tue, 2010-09-14 at 10:20 -0400, Luis R. Rodriguez wrote: >> > >> >> + del_timer_sync(&ifmgd->bcn_mon_timer); >> >> del_timer_sync(&sdata->u.mgd.conn_mon_timer); >> > >> > This is kinda inconsistent now ... and you added both these lines in >> > your patch set :-) >> >> Sorry, I do not follow, can you elaborate? > > You use "ifmgd->", but also "sdata->u.mgd." when both should be one of > those. Ah heh yeah, good catch, will resend. Luis -- 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