drmOpenByName: remove redundant drmAvailable check
authorDaniel Kurtz <djkurtz@chromium.org>
Thu, 17 Apr 2014 04:08:01 +0000 (12:08 +0800)
committerThierry Reding <treding@nvidia.com>
Tue, 13 May 2014 10:14:19 +0000 (12:14 +0200)
drmOpenByName() is a static function that is only called by drmOpen().
drmOpen() already checks drmAvailable(), so the check in
drmOpenByName() is redundant.

Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
Signed-off-by: Thierry Reding <treding@nvidia.com>
xf86drm.c

index e94f2cd..85beb8c 100644 (file)
--- a/xf86drm.c
+++ b/xf86drm.c
@@ -541,19 +541,6 @@ static int drmOpenByName(const char *name)
     int           fd;
     drmVersionPtr version;
     char *        id;
-    
-    if (!drmAvailable()) {
-       if (!drm_server_info) {
-           return -1;
-       }
-       else {
-           /* try to load the kernel module now */
-           if (!drm_server_info->load_module(name)) {
-               drmMsg("[drm] failed to load kernel module \"%s\"\n", name);
-               return -1;
-           }
-       }
-    }
 
     /*
      * Open the first minor number that matches the driver name and isn't