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. 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