Re: radeon-pre-2

[Dave Airlie]

>
> You're probably right, but it still doesn't follow that this driver must
> include all the fbdev and DRM code as well. Both fbdev and the DRM could
> use that driver, e.g., just like ide_cd and ide_disk use the IDE driver.
>

I think your wrong, look at drivers/video/aty/radeon* and tell me what in
there is capable of being abstracted from the hardware, every file access
lowlevel registers for something or other, be it mode setting or I2C, now
accessing lowlevels while the CP is running on a radeon is a one way
express to the land of the lockups... (think mode setting a second head,
while a 3d app is running on the first head...), the lowlevel driver can
provide a DRM and FB interface to fbcon and 3d stuff, but the lowlevel
driver needs all the code to do both...

The other thing I think some people are confusing is 2.4 fbdev and 2.6...
there is no console support in 2.6 fbdev drivers, it is all in the fbcon
stuff, so the fbdev drivers are only doing 2d mode setting and monitor
detection, some points I've considered are:

1. It doesn't matter where the code lives, fbdev/DRM need to start talking
about things
2. A generic interface between the two is probably going to be impossible,
graphics card interfaces aren't based on a standard like IDE (stop using
IDE for comparisons, it isn't the same IDE is a standard, and is designed
for things like cd drivers and hard drives, graphics card have no standard
interfaces, just because your pet card has a nice 2D/3D split it doesn't
mean that all do, if someone provides a better analogy feel free to use it
but can we all agree to drop the IDE comparisons now...)

I'm interested in seeing what Alan comes up with, even in a non-working
form .. I much prefer the evolution of these things than complete new
solutions... but I also think linking the fb and drm code together will
remove alot of the headaches and result in a more maintainable system
longterm, even if shortterm there are some ugly hacks..

Dave.

-- 
David Airlie, Software Engineer
http://www.skynet.ie/~airlied / airlied at skynet.ie
pam_smb / Linux DECstation / Linux VAX / ILUG person

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/