OSDN Git Service

Enum-ectomy of vblank modesetting ioctl
authorJesse Barnes <jbarnes@jbarnes-t61.(none)>
Sun, 27 Apr 2008 00:11:18 +0000 (17:11 -0700)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Sun, 27 Apr 2008 00:11:18 +0000 (17:11 -0700)
Enum can be of pretty much any size since C leaves the choice of size up to the implementation.  So avoid using it in new interfaces like the vblank pre- & post-modeset ioctl.  Thanks to hch for spotting this.

shared-core/drm.h

index 5981dcb..da149dc 100644 (file)
@@ -555,10 +555,9 @@ union drm_wait_vblank {
        struct drm_wait_vblank_reply reply;
 };
 
-enum drm_modeset_ctl_cmd {
-       _DRM_PRE_MODESET = 1,
-       _DRM_POST_MODESET = 2,
-};
+
+#define _DRM_PRE_MODESET 1
+#define _DRM_POST_MODESET 2
 
 /**
  * DRM_IOCTL_MODESET_CTL ioctl argument type
@@ -567,7 +566,7 @@ enum drm_modeset_ctl_cmd {
  */
 struct drm_modeset_ctl {
        unsigned long arg;
-       enum drm_modeset_ctl_cmd cmd;
+       int cmd;
 };
 
 /**