exynos4: gralloc: we have an ALIGN macro, use it
authorSimon Shields <keepcalm444@gmail.com>
Fri, 15 Apr 2016 12:44:30 +0000 (22:44 +1000)
committerGerrit Code Review <gerrit@cyanogenmod.org>
Fri, 15 Apr 2016 17:10:36 +0000 (10:10 -0700)
Change-Id: I1ef089b004d1d1e3f2ae525638954536ca881c7a

exynos4/hal/libgralloc_ump/alloc_device.cpp

index 467e6f7b6afc0be3ebf65291e4c345b21e2b0e08..9c9c7b481651d3b81e63525492531b2f9e66a980 100644 (file)
@@ -372,8 +372,8 @@ static int alloc_device_alloc(alloc_device_t* dev, int w, int h, int format,
         format == OMX_COLOR_FormatYUV420SemiPlanar) {
         /* FIXME: there is no way to return the vstride */
         int vstride;
-        stride = (w + 15) & ~15;
-        vstride = (h + 15) & ~15;
+        stride = EXYNOS4_ALIGN(w, 16);
+        vstride = EXYNOS4_ALIGN(h, 16);
         switch (format) {
         case HAL_PIXEL_FORMAT_YCbCr_420_SP:
         case HAL_PIXEL_FORMAT_YCbCr_420_SP_TILED:
@@ -403,7 +403,6 @@ static int alloc_device_alloc(alloc_device_t* dev, int w, int h, int format,
             return -EINVAL;
         }
     } else {
-        int align = 8;
         int bpp = 0;
         switch (format) {
         case HAL_PIXEL_FORMAT_RGBA_8888:
@@ -422,7 +421,7 @@ static int alloc_device_alloc(alloc_device_t* dev, int w, int h, int format,
         default:
             return -EINVAL;
         }
-        size_t bpr = (w*bpp + (align-1)) & ~(align-1);
+        size_t bpr = EXYNOS4_ALIGN((w*bpp), 8);
         size = bpr * h;
         stride = bpr / bpp;
         stride_raw = bpr;