![]() | |||||||||||||
Messages in this thread |
On Sul, 2003-06-22 at 09:25, Paul Rolland wrote: > Hello, > > > didn't work. I did spend a few hours on the patch and I was willing to > > spend another one to see the kernel fixed. But if he does not > > care, why > > should I ? > > Maybe because some others *do* appreciate the work you did, and the > patch being available... Don't get me wrong - I am glad he's done a lot of detective work on this and on other stuff. That isn't the only problem however - we have cases where there are entire ranges of PCI idents that have one driver and others requiring more care that we dont handle well right now either (eg 440GX). Another problem is that the table entries get used which means the table itself cannot be __init. That makes me wonder if we should in fact have a set of detect functions that fill in a struct irq_router * and remove most of the array while fixing the other stuff. That cleans up the detect, removes the 440GX ifdefs properly with some care, removes the 440GX bios dmi table entries, and fixes SiS. It isnt that Davide's patch makes it too ugly, its already too ugly, Davide and also 440GX just hit the places that make it rather obvious it wants a proper clean up Alan - 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/ | ||||||||||||
| Last update: 2005-03-22 12:36 [W:0.446 / U:0.410 seconds] ©2003-2008 Jasper Spaans | |||||||||||||