Hello Johan,
> the Degas code however, is on a sorry state of disrepair, It's still missing
> big parts of functionality and is so riddled with #warning and #error
> statement that even thinking of compiling it would be a waste of time.
> however, if somebody is interested in looking it over, feel free to do so.
Some comments from a really insane...
* all symbols visible with 'nm objectfile.o' should be prefixed with
the metalanguage prefix.
* Don't use pcicfg space for other purposes than initial configuration.
Especially do not use it in _mode_setup() to distinguish device versions.
Use flags or a chipset_version field in the <meta>_t structure instead.
* calc_partials() is a "permedianism". The Permedia can only accelerate
frame buffers with certain widths. calc_partials() calculates the next
supported width.
* bpd, bpf, bpc and bpp refer to the number of bits per
{d}ot (unit transfered to the dot port)
{f}rame (unit stored per pixel and frame)
{c}ommon (unit stored common to all frames)
{p}ixel = bpc + (stereo ? 2 : 1) * frames * bpf
* Drivers must not share region information. E.g. you must not give the
DAC driver access to the memory control region claimed by the chipset
driver except through the DacOut() functions. If you have to access
indexed registers, declare a 'static inline' function in the
ramdac/clock driver.
* The kgi/ directory is reserved for kgi extension headers.
Chipset register declarations should go to chipset/Matrox/Gx00.h in your
case. This has the benefit you have them at hand where you work the
most time.
* I found it useful to convert the header to the naming conventions first.
Especially as this gives a good overview what registers do what.
These are the comments that came to mind when I had a quick glance over it
yesterday. Keep asking/working on it!
Thanks,
Steffen
----------------- e-mail: [EMAIL PROTECTED] -----------------
