drm.h: Handle DragonFly like Linux

Commit Message

François Tigeot May 14, 2016, 7:13 a.m. UTC

The drm code in DragonFly uses a local Linux implementation which doesn't
define the __linux__ macro.

Use __DragonFly__ instead in order to not try to compile non-Linux code.
---
 include/uapi/drm/drm.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Emil Velikov May 14, 2016, 2:51 p.m. UTC | #1

Hi François,

On 14 May 2016 at 08:13, François Tigeot <[e-mail]> wrote:
> The drm code in DragonFly uses a local Linux implementation which doesn't
> define the __linux__ macro.
>
> Use __DragonFly__ instead in order to not try to compile non-Linux code.
Does that meant that the workarounds in the else statements don't work
? I doubt that anyone will mind if we update/correct them.
Alternatively if one insists on using the __linux__ route here, imho
it's better to just set the define it in your build.

All that aside, for next revision/future work please check the
documentation [1] and add your s-o-b line. Also, please base your work
against Dave's drm-next branch [2]

Thank you
Emil

[1] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches
See section 11
[2] https://cgit.freedesktop.org/~airlied/linux

Patch

diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h
index a0ebfe7..76bea05 100644
--- a/include/uapi/drm/drm.h
+++ b/include/uapi/drm/drm.h
@@ -36,7 +36,7 @@ 
 #ifndef _DRM_H_
 #define _DRM_H_
 
-#if defined(__KERNEL__) || defined(__linux__)
+#if defined(__KERNEL__) || defined(__linux__) || defined(__DragonFly__)
 
 #include <linux/types.h>
 #include <asm/ioctl.h>