[PATCH 05/10] xfree86: bus: enable sparc code usage as its definition

Mark Kettenis mark.kettenis at xs4all.nl
Sun May 2 14:24:20 PDT 2010


> From: Tiago Vignatti <tiago.vignatti at nokia.com>
> Date: Sun,  2 May 2010 23:07:41 +0300
> 
> This is here since the initial revision but it's not synchronized with the
> definitions in both xf86.h and xf86sbusBus.h

Huh?  The extra declarations in xf86.h and xf86sbusBus.h are harmless,
so your conclusion that these !defined(__OpenBSD__) clauses can go is
flawed.  If you really insist on synchronizing things, please adjust
the headers.

The reason for this !defined(__OpenBSD__) is that the code in
xserver/hw/xfree86/common/xf86sbusBus.c is too OS-specific to be
usable on OpenBSD.  I have a (much simpler) OpenBSD-specific
replacement for that code.  I should probably talk to Matthieu to see
how we can integrate that code nack upstream.  Until that's take care
of those !defined(__OpenBSD__) makes sure that official Xorg releases
stand a chance to compile on OpenBSD/sparc and OpenBSD/sparc64.  You
won't be able to use the accelerated drivers for SBus and UPA grphics
hardware, but PCI hardware and the unuaccelerated wsfb framebuffer
driver should work fine.

> Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>

Nacked-by: Mark Kettenis <kettenis at openbsd.org>

> diff --git a/hw/xfree86/common/xf86Bus.c b/hw/xfree86/common/xf86Bus.c
> index 788de0b..7f30b18 100644
> --- a/hw/xfree86/common/xf86Bus.c
> +++ b/hw/xfree86/common/xf86Bus.c
> @@ -68,7 +68,7 @@ void
>  xf86BusProbe(void)
>  {
>      xf86PciProbe();
> -#if (defined(__sparc__) || defined(__sparc)) && !defined(__OpenBSD__)
> +#if (defined(__sparc__) || defined(__sparc))
>      xf86SbusProbe();
>  #endif
>  }
> @@ -481,7 +481,7 @@ void
>  xf86PostProbe(void)
>  {
>      if (fbSlotClaimed && (pciSlotClaimed
> -#if (defined(__sparc__) || defined(__sparc)) && !defined(__OpenBSD__)
> +#if (defined(__sparc__) || defined(__sparc))
>  	    || sbusSlotClaimed
>  #endif
>  	    ))
> -- 
> 1.6.0.4
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 


More information about the xorg-devel mailing list