On Thursday 06 March 2008, Johannes Berg wrote: > > > > +#if TODO > > > +Is this correct?? > > > +#endif > > > + rt2x00lib_write_tx_desc(rt2x00dev, skb, control, queue->qid); > > > > This is incorrect, even though it theoretically would work. > > The qid != queue number, for most TX queues the value will be the same, > > but the meaning of the field is different. > > Hence my #if TODO which causes sparse to warn about the code :) I meant > to ask you but never had time to continue working on this patch. > > > Resolving this issue will also be something I can fix relatively fast after this > > patch is accepted and applied to wireless-testing. > > You could also just make a patch for exactly this and ask Ron to merge > it together with this? Well that patch would then be completely untested (I code, but have hardly time to perform the testing myself). I can already say that the patch from Ron will work despite the my remark about the different meaning of the qid field. My patch that will clean it up, and will fix the issue, will be something that needs testing before it should be send upstream... I want to prevent messes like the rt2x00 2.1.0 release, which was broken in every possible way, from happening again in wireless-testing ;) Ivo -- 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
