commit 5e3ac6bc7138e92fa04d7425a2a2535b9ae0e44a
author Bernhard Rosenkraenzer <Bernhard.Rosenkranzer@linaro.org> Fri Feb 21 01:38:44 2014 +0530
committer Amit Pundir <amit.pundir@linaro.org> Fri Feb 21 19:28:56 2014 +0530
tree 1ab0b0b07cd9c83fbf3b5d0c1366dff5324e07d6
parent 80d4eced69f4d1e9f9ef97b5f4f26868cd76ae26

gralloc: dont hardcode 32bpp as RGBX

Not every 32bpp is in RGBX format.

Change-Id: Ic3c6a0cbf475bff2fe2fc4b023c6bc58e4fc295e
Signed-off-by: Amit Pundir <amit.pundir@linaro.org>

modules/gralloc/framebuffer.cpp

diff --git a/modules/gralloc/framebuffer.cpp b/modules/gralloc/framebuffer.cpp
index 326f2ae..9d8513a 100644
--- a/modules/gralloc/framebuffer.cpp
+++ b/modules/gralloc/framebuffer.cpp
@@ -330,7 +330,7 @@
         if (status >= 0) {
             int stride = m->finfo.line_length / (m->info.bits_per_pixel >> 3);
             int format = (m->info.bits_per_pixel == 32)
-                         ? HAL_PIXEL_FORMAT_RGBX_8888
+                         ? (m->info.red.offset ? HAL_PIXEL_FORMAT_BGRA_8888 : HAL_PIXEL_FORMAT_RGBX_8888)
                          : HAL_PIXEL_FORMAT_RGB_565;
             const_cast<uint32_t&>(dev->device.flags) = 0;
             const_cast<uint32_t&>(dev->device.width) = m->info.xres;