On Mon, Jul 20, 2015 at 05:06:09PM +0100, Emil Velikov wrote: > 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 ? Do you mean indent the defines or have the entire block undef ifdef __OpenBSD__? Either way would be fine with me. _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
