Mesa (master): gbm: Remove is_planar_format dead code

Daniel Stone daniels at kemper.freedesktop.org
Tue Jul 18 21:51:24 UTC 2017


Module: Mesa
Branch: master
Commit: 7f157a21f16296f72d005d73c301c90a98d1e662
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7f157a21f16296f72d005d73c301c90a98d1e662

Author: Daniel Stone <daniels at collabora.com>
Date:   Fri Jun 16 14:28:42 2017 +0100

gbm: Remove is_planar_format dead code

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.stach at pengutronix.de>
Reviewed-by: Emil Velikov <emil.velikov at 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;




More information about the mesa-commit mailing list