From: Dave A. <ai...@li...> - 2004-08-03 13:35:26
|
Okay I've created a drm branch drmfntbl-0-0-1 and I've commited the work I've done so far, i830, mach64, radeon and gamma should be working, I've only tested the radeon really, I don't even own a gamma (but it causes a lot of the issues :-) The current patch is at http://www.skynet.ie/~airlied/patches/dri/drm_fn_tbl.diff Ian, if you want to remove any more macros can you do it on that branch, I don't intend that branch to live very long we should pull it back onto the trunk when we are happy with it ... The only thing I can think off is maybe adding this stuff to another C file per driver but that probably isn't necessary ... the other idea I've had is to perhaps separate the function table when we get the full table done, then we can have tables per functionality group, i.e. adding the dma flush just for the gamma to that table makes it a bit ugly... Dave. -- David Airlie, Software Engineer http://www.skynet.ie/~airlied / airlied at skynet.ie pam_smb / Linux DECstation / Linux VAX / ILUG person |