On Fri, Oct 10, 2008 at 12:40 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2008-10-10 at 05:22 -0700, Luis R. Rodriguez wrote: > >> ecode = ath9k_hw_process_ini(ah, chan, macmode); >> - if (ecode != 0) >> + if (ecode != 0) { >> + ecode = -EINVAL; >> goto bad; > > didn't this actually want to pass up ecode? Fuck yeah, bad: if (status) *status = ecode; return false; 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