Mesa (master): egl/android: remove droid_probe_driver()

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Fri Aug 24 10:58:05 UTC 2018


Module: Mesa
Branch: master
Commit: d1211f3112c0e7c710a380ddc0887a44f461e1bb
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=d1211f3112c0e7c710a380ddc0887a44f461e1bb

Author: Emil Velikov <emil.velikov at collabora.com>
Date:   Wed Aug  8 15:05:56 2018 +0100

egl/android: remove droid_probe_driver()

The function name is misleading - it effectively checks if
loader_get_driver_for_fd fails. Which can happen only only on strdup
error - a close to impossible scenario.

Drop the function - we call the loader API at at later stage.

Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
Reviewed-by: Tomasz Figa <tfiga at chromium.org>

---

 src/egl/drivers/dri2/platform_android.c | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/src/egl/drivers/dri2/platform_android.c b/src/egl/drivers/dri2/platform_android.c
index ee6c2791b1..de53376c9f 100644
--- a/src/egl/drivers/dri2/platform_android.c
+++ b/src/egl/drivers/dri2/platform_android.c
@@ -1404,19 +1404,6 @@ error:
    return false;
 }
 
-static bool
-droid_probe_driver(int fd)
-{
-   char *driver_name;
-
-   driver_name = loader_get_driver_for_fd(fd);
-   if (driver_name == NULL)
-      return false;
-
-   free(driver_name);
-   return true;
-}
-
 typedef enum {
    probe_fail = -1,
    probe_success = 0,
@@ -1442,11 +1429,6 @@ droid_probe_device(_EGLDisplay *disp, int fd, const char *vendor)
       goto cleanup;
    }
 
-   if (!droid_probe_driver(fd)) {
-      ret = probe_fail;
-      goto cleanup;
-   }
-
    ret = probe_success;
 
 cleanup:




More information about the mesa-commit mailing list