CVS commit: src/sys/dev

[Matthias Drochner <drochner%netbsd.org@localhost>]

Module Name:    src
Committed By:   drochner
Date:           Fri Feb 22 19:47:06 UTC 2008

Modified Files:
        src/sys/dev/drm: drm_memory.c
        src/sys/dev/pci: agp_i810.c agpvar.h

Log Message:
Add a horrible hack to allow the DRM driver to get at the bus_space_handle
allocated by the AGP driver in the case of Intel chipset graphics.
This is different from the patch by Yorick Hardy circulated a while ago
in that it doesn't change the semantics of reference counting within
the (3rd-party) DRM code -- it just bypasses it.
Needless to say that it is uglier, but it eases future updates to
the DRM code because the change there is just 2 LOC.

Imo, a cleaner solution for all that would be to attach "agp" to "vga"
in the intel chipset graphics case, which better reflects how the hardware
is structured. This would still need a hack to the DRM code, but
it would be confined to childs of the "vga" device, without need for
global variables.
Since there is a variety of intel chipsets with AGP and/or builtin
graphics options, this would need a considerable testing effort.


To generate a diff of this commit:
cvs rdiff -r1.5 -r1.6 src/sys/dev/drm/drm_memory.c
cvs rdiff -r1.48 -r1.49 src/sys/dev/pci/agp_i810.c
cvs rdiff -r1.15 -r1.16 src/sys/dev/pci/agpvar.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.