On Tue, Feb 08, 2011 at 07:45:53AM -0800, wwguy wrote: > > - priv->plcp_jiffies = stamp; > > - > > - if (msecs == 0) > > - return true; > > + BUG_ON(msecs == 0); > > why not just return? I understand it should not be "0", but really need > BUG_ON? > > > > > - stamp = jiffies; > > - msecs = jiffies_to_msecs(stamp - priv->plcp_jiffies); > > - priv->plcp_jiffies = stamp; > > - > > - if (msecs == 0) > > - return true; > > + BUG_ON(msecs == 0); > Same above > > > Similar question, we are very close to finish driver split, when is the > best time for this patch? The same aswer :-) I will respin, when your patches will land in John tree. Stanislaw -- 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
