Re: [PATCH libdrm] xf86drm: correct the OpenBSD DRM_MAJOR define

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 18 July 2015 at 22:20, Jonathan Gray <jsg@xxxxxxxxx> wrote:
> As far as I can tell no OpenBSD platform ever used 81
> for a drm major.  While the value was added to libdrm in 2003
> or earlier drm didn't appear in OpenBSD till 2007.
>
> Of the OpenBSD platforms that support drm amd64/macppc/sparc64
> use a major of 87, i386 uses 88.
>
> Signed-off-by: Jonathan Gray <jsg@xxxxxxxxx>
> ---
>  xf86drm.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/xf86drm.c b/xf86drm.c
> index 7d7f9c7..a2c549c 100644
> --- a/xf86drm.c
> +++ b/xf86drm.c
> @@ -72,9 +72,11 @@
>  #define DRM_MAJOR 34
>  #endif
>
> -# ifdef __OpenBSD__
> -#  define DRM_MAJOR 81
> -# endif
> +#if defined(__OpenBSD__) && defined(__i386__)
> +#define DRM_MAJOR 88
> +#elif defined(__OpenBSD__)
> +#define DRM_MAJOR 87
> +#endif
>
Bikeshed: any objections if we cascade the ifdef statements ?

Thanks
Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux