Messages in this thread |  | | Date | Wed, 3 Oct 2001 04:52:57 +0200 | From | David Weinehall <> | Subject | Re: [PATCH] Intel 830 support for agpgart |
| |
On Tue, Oct 02, 2001 at 10:20:43PM -0400, Robert Love wrote: > On Tue, 2001-10-02 at 20:16, David Weinehall wrote: > > If the only differences between the different cards are the nr of > > aperture-sizes and the status-register settings, why not have a struct > > which contains all the valid cards, and use a scan-routine?! > > I suppose we could, and this is a good idea -- although we'd be > reapproaching the size of the current implementation which would be > theoretically faster, too.
Afaik, speed is not really an issue (it's not like you're going to notice a difference anyway, even if you had a struct with 100 different adapters in it.) As for reapproaching the size of the current implementation, the difference is that you get one single function that you don't have to change. You just add one single line to the struct for each adapter.
Clean design is a virtue ;-)
> There are only 3 possibilities right now (i830, i840, and everything > else). > > Hmm, maybe I will code this nonetheless...
It's your call.
/David _ _ // David Weinehall <tao@acc.umu.se> /> Northern lights wander \\ // Project MCA Linux hacker // Dance across the winter sky // \> http://www.acc.umu.se/~tao/ </ Full colour fire </ - 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/
|  |