Messages in this thread | | | Date | Tue, 4 Mar 2003 10:54:28 -0800 | Subject | Re: [PATCH] pcmcia: get initialization ordering right [Was: [PATCH 2.5] : i82365 & platform_bus_type] | From | Jean Tourrilhes <> |
| |
On Tue, Mar 04, 2003 at 11:48:22AM -0600, Patrick Mochel wrote: > > Surely you're sore that your code has required some modifications since > Dominik has started working on PCMCIA, and I'm sure that no harm was > intended. It's had some bumps, but IMO, he's done a great job, and the > result is a vast improvement. The least you could is give the guy some > slack, instead of whining about your own inconveniences.
I don't mind the changes, changes are usually good. In 2.5.X, I had to change my code to accomodate the new PCI interface, the removal of global IRQ, the new module interface, the various USB API changes and other changes. And actually, your work currently hasn't had any impact on the source code I follow (yet). What I mind is the lack of basic testing. From your patch, the initialisation order mixup and the other obvious bug fix I sent you, this code had zero chances of working at all and it's obvious that nobody bothered to check if it could work or not for at least two kernel releases. Yeah, I know that I should not complain because at least the code did compile (modulo other minor obvious fixes that are already in Linus BK).
> Thanks, > > -pat
Have fun...
Jean - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |