On Wed, Apr 13, 2011 at 8:34 PM, Kalle Valo <kvalo@xxxxxxxxxx> wrote: > "John W. Linville" <linville@xxxxxxxxxxxxx> writes: > >>> Looks like this v2 did not found its way to wireless-next [1]. >> >> That's what happens, when it is posted after I've already merged the v1. :-) >> >> I fixed things up and will commit the difference with the following message: >> >> Â Â ath5k: improve comments for optimized tx descriptor setup >> >> Â Â Comment the use of local variables to reduce the number of load/store >> Â Â operations on uncached memory, in hopes of not losing this optimization >> Â Â accidentally in the future. > > Thanks John. It was my fault as I was so slow to comment. But I blame > the sun for this, it's so amazing to see the sun again after six > months! ;) > > -- > Kalle Valo > Nice, the diff went into wireless-next-2.6 GIT [1] (even comments are welcome!). I would enjoy in the future, if you add by yourself a Reported-by as I saw from Linus Torvalds adding it to commits where maintainers simply forget it. Without reviewing/testing/reporting the code won't get better and more stable. I had tested all 10 or 11 recent ath5k patches, but lazy or simply trust the work by Felix. Not sure if I should add a Tested-by to all single patches of patchset or it's OK to reply to the 1st in series? Often I see a summary describing the patchset, something like "0/7: My super patchset". Replying only to 0/7 would make life easier... but I am *still* learning from git use-case-2-use-case. Ted (maintainer of ext4) told me even he is distinguising between Reported-and-tested-by and Reported-by/Tested-by. As a summary, don't forget to give credits to people. (P.S. I am currently here on a lame-ass IBM T40p with Pentium-M... I will contribute till it explodes someday). - Sedat - P.S. I am currently here on a lame-ass IBM T40p with Pentium-M... I will contribute till it explodes someday. [1] http://git.us.kernel.org/?p=linux/kernel/git/linville/wireless-next-2.6.git;a=commit;h=8962d87129ec0a820d17ac44cbf3f51010ad8db8 -- 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