This was only used in create_dumb() to blacklist planar formats. However, the start of the function already whitelists ARGB8888 (cursor) and XRGB8888 (scanout), and nothing else. So this entire function can be removed.
Reviewed-by: Lucas Stach <l.st...@pengutronix.de> Reviewed-by: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Daniel Stone <dani...@collabora.com> --- src/gbm/backends/dri/gbm_dri.c | 30 ------------------------------ 1 file changed, 30 deletions(-) diff --git a/src/gbm/backends/dri/gbm_dri.c b/src/gbm/backends/dri/gbm_dri.c index e50ce04c86..1b2cc4ce76 100644 --- a/src/gbm/backends/dri/gbm_dri.c +++ b/src/gbm/backends/dri/gbm_dri.c @@ -1022,31 +1022,6 @@ gbm_dri_bo_import(struct gbm_device *gbm, return &bo->base; } -static bool -is_planar_format(uint32_t format) -{ - switch (format) { - case GBM_FORMAT_NV12: - case GBM_FORMAT_NV21: - case GBM_FORMAT_NV16: - case GBM_FORMAT_NV61: - case GBM_FORMAT_YUV410: - case GBM_FORMAT_YVU410: - case GBM_FORMAT_YUV411: - case GBM_FORMAT_YVU411: - case GBM_FORMAT_YUV420: - case GBM_FORMAT_YVU420: - case GBM_FORMAT_YUV422: - case GBM_FORMAT_YVU422: - case GBM_FORMAT_YUV444: - case GBM_FORMAT_YVU444: - return true; - default: - return false; - } - -} - static struct gbm_bo * create_dumb(struct gbm_device *gbm, uint32_t width, uint32_t height, @@ -1068,11 +1043,6 @@ create_dumb(struct gbm_device *gbm, return NULL; } - if (is_planar_format(format)) { - errno = EINVAL; - return NULL; - } - bo = calloc(1, sizeof *bo); if (bo == NULL) return NULL; -- 2.13.0 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev