Re: radeon unified driver

[Felix <fx...@gm...>]

Am Samstag, den 19.02.2005, 01:11 +0100 schrieb Nicolai Haehnle:
> Hi,
>=20
> On Saturday 19 February 2005 00:46, Roland Scheidegger wrote:
> > There is some problem with driconf, it seems to have some problems=20
> > because the driver's name (radeon) does not match what it expects=20
> > (r200). Likewise, I couldn't figure out how you'd have 2 separate confi=
g=20
> > sections for both r100 and r200, currently you'll get all options of th=
e=20
> > r200 (though it won't work for that chip family...), some options just=20
> > won't do anything on r100.
>=20
> When I started working on the R300 driver, I did some similar work so tha=
t=20
> the R300 driver should in theory be able to handle R200 as well (this R20=
0=20
> support has certainly gone to hell by now because of all the hacking that=
=20
> has been going on).
>=20
> The point is, I also faced the driconf issue, and you can see how I=20
> attempted to tackle it at=20
> http://cvs.sourceforge.net/viewcvs.py/r300/r300_driver/r300/radeon_screen=
.c?rev=3D1.7&view=3Dauto
> My solution is probably not that good, but it might give you some ideas.

This is not going to work with the GUI configuration tool. It looks for
a symbol called __driConfigOptions in the driver module, so with your
change driconf would claim that the driver doesn't support
configuration. It can't call the driver to probe on which hardware it's
running, so you can't use different sets of options for different
hardware supported by the same driver.

I have the same problem in the savage driver. Some options just don't
make sense on Savage3D-based hardware, so they have no effect in this
case (for example floating-point depth buffer). So, when I read the
configuration I have code like this in the driver:

   imesa->float_depth =3D driQueryOptionb(&imesa->optionCache, "float_depth=
") &&
       savageScreen->chipset >=3D S3_SAVAGE4;

Additionally you could put hardware-specific options into separate
sections, so that the user can tell, which options will have an effect
on her hardware.

>=20
> cu,
> Nicolai

Regards,
  Felix

--=20
| Felix K=FChling <fx...@gm...>                     http://fxk.de.vu |
| PGP Fingerprint: 6A3C 9566 5B30 DDED 73C3  B152 151C 5CC1 D888 E595 |