On Fri, Oct 09, 2009 at 01:08:06AM +0200, Michael Buesch wrote: > On Friday 09 October 2009 00:34:54 John W. Linville wrote: > > Albert Herranz (1): > > b43: do not stack-allocate pio rx/tx header and tail buffers > > Come on, this is _not_ funny anymore. I did _not_ ack this patch, because I do _not_ like it. > I was planning to do a better solution, but I didn't have the time, yet. > Can you _please_ either: > - Wait for my ack before you apply random b43 patches > or > - Remove me from MAINTAINERS Michael, After Albert posted his first version of the patch, you said: "Just embed it into struct b43_wl (surround it by #ifdef CONFIG_B43_PIO). No need to kzalloc then and it saves some memory. You also need to alloc 4 bytes for the tail buffer (that currently is on the stack, too)." AFAICT he complied with that request when he posted the second version. Since the patch seemed fine otherwise, I applied it; and since it is a fix I sent it on for 2.6.32. As Dave suggested, there is plenty of time to fix it "properly" for 2.6.33 and beyond. I'm happy to accomodate you. John P.S. Please understand that while some driver maintainers want to ack every patch, others see that as a burden and get annoyed with me if I don't apply reasonable patches without bothering them. It can be a bit difficult these things... -- John W. Linville Someday the world will need a hero, and you linville@xxxxxxxxxxxxx might be all we have. Be ready. -- 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
