diff options
author | Alexander Orzechowski <alex@ozal.ski> | 2023-06-23 17:24:38 -0400 |
---|---|---|
committer | Simon Ser <contact@emersion.fr> | 2023-11-29 13:25:06 +0100 |
commit | 91182266342587cf01b2e625dbf0682df149f3db (patch) | |
tree | 189cf95e4c6c9486cfe6aedfa12f98be66c9f624 | |
parent | ae3d7a697c3e8b137f6cf5f9208cc9dd132f541c (diff) |
render/gles2: Don't track has_alpha in pixel_format
Why track the alpha here when we can already get that information
elsewhere?
-rw-r--r-- | include/render/gles2.h | 1 | ||||
-rw-r--r-- | render/gles2/pixel_format.c | 32 | ||||
-rw-r--r-- | render/gles2/texture.c | 2 |
3 files changed, 13 insertions, 22 deletions
diff --git a/include/render/gles2.h b/include/render/gles2.h index 930cf07d..83dabb7e 100644 --- a/include/render/gles2.h +++ b/include/render/gles2.h @@ -25,7 +25,6 @@ struct wlr_gles2_pixel_format { // optional field, if empty then internalformat = format GLint gl_internalformat; GLint gl_format, gl_type; - bool has_alpha; }; struct wlr_gles2_tex_shader { diff --git a/render/gles2/pixel_format.c b/render/gles2/pixel_format.c index acaac5ce..1d9884e3 100644 --- a/render/gles2/pixel_format.c +++ b/render/gles2/pixel_format.c @@ -2,6 +2,7 @@ #include <GLES2/gl2.h> #include <GLES2/gl2ext.h> #include "render/gles2.h" +#include "render/pixel_format.h" /* * The DRM formats are little endian while the GL formats are big endian, @@ -12,100 +13,84 @@ static const struct wlr_gles2_pixel_format formats[] = { .drm_format = DRM_FORMAT_ARGB8888, .gl_format = GL_BGRA_EXT, .gl_type = GL_UNSIGNED_BYTE, - .has_alpha = true, }, { .drm_format = DRM_FORMAT_XRGB8888, .gl_format = GL_BGRA_EXT, .gl_type = GL_UNSIGNED_BYTE, - .has_alpha = false, }, { .drm_format = DRM_FORMAT_XBGR8888, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_BYTE, - .has_alpha = false, }, { .drm_format = DRM_FORMAT_ABGR8888, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_BYTE, - .has_alpha = true, }, { .drm_format = DRM_FORMAT_BGR888, .gl_format = GL_RGB, .gl_type = GL_UNSIGNED_BYTE, - .has_alpha = false, }, #if WLR_LITTLE_ENDIAN { .drm_format = DRM_FORMAT_RGBX4444, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_SHORT_4_4_4_4, - .has_alpha = false, }, { .drm_format = DRM_FORMAT_RGBA4444, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_SHORT_4_4_4_4, - .has_alpha = true, }, { .drm_format = DRM_FORMAT_RGBX5551, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_SHORT_5_5_5_1, - .has_alpha = false, }, { .drm_format = DRM_FORMAT_RGBA5551, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_SHORT_5_5_5_1, - .has_alpha = true, }, { .drm_format = DRM_FORMAT_RGB565, .gl_format = GL_RGB, .gl_type = GL_UNSIGNED_SHORT_5_6_5, - .has_alpha = false, }, { .drm_format = DRM_FORMAT_XBGR2101010, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_INT_2_10_10_10_REV_EXT, - .has_alpha = false, }, { .drm_format = DRM_FORMAT_ABGR2101010, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_INT_2_10_10_10_REV_EXT, - .has_alpha = true, }, { .drm_format = DRM_FORMAT_XBGR16161616F, .gl_format = GL_RGBA, .gl_type = GL_HALF_FLOAT_OES, - .has_alpha = false, }, { .drm_format = DRM_FORMAT_ABGR16161616F, .gl_format = GL_RGBA, .gl_type = GL_HALF_FLOAT_OES, - .has_alpha = true, }, { .drm_format = DRM_FORMAT_XBGR16161616, .gl_internalformat = GL_RGBA16_EXT, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_SHORT, - .has_alpha = false, }, { .drm_format = DRM_FORMAT_ABGR16161616, .gl_internalformat = GL_RGBA16_EXT, .gl_format = GL_RGBA, .gl_type = GL_UNSIGNED_SHORT, - .has_alpha = true, }, #endif }; @@ -151,11 +136,18 @@ const struct wlr_gles2_pixel_format *get_gles2_format_from_drm(uint32_t fmt) { const struct wlr_gles2_pixel_format *get_gles2_format_from_gl( GLint gl_format, GLint gl_type, bool alpha) { for (size_t i = 0; i < sizeof(formats) / sizeof(*formats); ++i) { - if (formats[i].gl_format == gl_format && - formats[i].gl_type == gl_type && - formats[i].has_alpha == alpha) { - return &formats[i]; + if (formats[i].gl_format != gl_format || + formats[i].gl_type != gl_type) { + continue; + } + + const struct wlr_pixel_format_info *drm_fmt = + drm_get_pixel_format_info(formats[i].drm_format); + if (drm_fmt->has_alpha != alpha) { + continue; } + + return &formats[i]; } return NULL; } diff --git a/render/gles2/texture.c b/render/gles2/texture.c index 947a2c1b..5cf0219c 100644 --- a/render/gles2/texture.c +++ b/render/gles2/texture.c @@ -216,7 +216,7 @@ static struct wlr_texture *gles2_texture_from_pixels( return NULL; } texture->target = GL_TEXTURE_2D; - texture->has_alpha = fmt->has_alpha; + texture->has_alpha = drm_fmt->has_alpha; texture->drm_format = fmt->drm_format; GLint internal_format = fmt->gl_internalformat; |