egl/android: remove droid_probe_driver()
authorEmil Velikov <emil.velikov@collabora.com>
Wed, 8 Aug 2018 14:05:56 +0000 (15:05 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Fri, 24 Aug 2018 10:52:39 +0000 (11:52 +0100)
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@collabora.com>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>
src/egl/drivers/dri2/platform_android.c

index ee6c2791b18854cf8e3d1e8f846a26122ff435bc..de53376c9fc35c0d8041de083e97d467eb78f35d 100644 (file)
@@ -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: