On 10/29/11 08:54, Alexandr Shadchin wrote:
bsd_pci.c have only one function osPciInit(). osPciInit() is equivalent
call xf86InitVidMem(). xf86InitVidMem() is needed only for OpenBSD
(see commit 06c0372c3a1b45005eb6d50406f77f4e93f1de1e)

Pretty sure Solaris needs xf86InitVidMem too, though I'm not in a position
to test today.   Any reason not to just set it to xf86InitVidMem for all
the platforms that used to call it in bsd_pci.c ?

diff --git a/hw/xfree86/os-support/bus/Pci.h b/hw/xfree86/os-support/bus/Pci.h
index 88560ec..821de0d 100644
--- a/hw/xfree86/os-support/bus/Pci.h
+++ b/hw/xfree86/os-support/bus/Pci.h
@@ -137,12 +137,12 @@
  #define PCI_BUS_NO_DOMAIN(bus) ((bus)&  0xffu)
  #define PCI_TAG_NO_DOMAIN(tag) ((tag)&  0x00ffff00u)

-#if defined(linux)
+#if defined(linux) || defined(__sun) || defined(__GNU__) || \
+       defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || \
+       defined(__NetBSD__) || defined(__DragonFly__)
  #define osPciInit(x) do {} while (0)
-#elif defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || \
-       defined(__OpenBSD__) || defined(__NetBSD__) || \
-       defined(__DragonFly__) || defined(__sun) || defined(__GNU__)
-extern void osPciInit(void);
+#elif defined(__OpenBSD__)
+#define osPciInit(x) do { xf86InitVidMem(); } while (0)
  #else
  #error No PCI support available for this architecture/OS combination
  #endif

--
        -Alan Coopersmith-        alan.coopersm...@oracle.com
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to