From: Emil Velikov <emil.veli...@collabora.com> 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. Cc: Robert Foss <robert.f...@collabora.com> Cc: Tomasz Figa <tf...@chromium.org> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- 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 bea06fd55dd..9f2124fe349 100644 --- a/src/egl/drivers/dri2/platform_android.c +++ b/src/egl/drivers/dri2/platform_android.c @@ -1236,19 +1236,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, @@ -1274,11 +1261,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: -- 2.18.0 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev