From: Emil Velikov <emil.veli...@collabora.com>
AFAICT the extension only makes sense/can work with wl_drm.
If/as we decide to drop it there's a lot of subtleties (borderline
hacks) that can be dropped.
Add a comment with a few keywords to grep.
Cc: Daniel Stone <dani...@collabora.com>
Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
---
I think the comment is accurate, but do correct me if not.
I'm not advocating/lobbying about any removal, just preparing for the
sweet sweet moment ...
---
src/egl/drivers/dri2/egl_dri2.h | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/src/egl/drivers/dri2/egl_dri2.h b/src/egl/drivers/dri2/egl_dri2.h
index 4a52b490a87..a48ea2c662b 100644
--- a/src/egl/drivers/dri2/egl_dri2.h
+++ b/src/egl/drivers/dri2/egl_dri2.h
@@ -428,6 +428,11 @@ const __DRIconfig *
dri2_get_dri_config(struct dri2_egl_config *conf, EGLint surface_type,
EGLenum colorspace);
+/* XXX: WL_bind_wayland_display is only useful/applicable for wl_drm.
+ * Can/should we consider deprecating it as we iron most issues in wl_dmabuf?
+ * How about wl_drm overall - this will allow us to remove a lot of icky code.
+ * Search for server_drm, driver_name and authenticate.
+ */
static inline void
dri2_set_WL_bind_wayland_display(_EGLDriver *drv, _EGLDisplay *disp)
{
--
2.14.0
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
