commit ae823e773536ee9b6cf97c575463f1a9902370d9
author Saurabh Shah <saurshah@codeaurora.org> Fri Oct 05 00:08:11 2012 -0400
committer Iliyan Malchev <malchev@google.com> Sun Oct 07 17:20:17 2012 -0700
tree 377948d58574ba1c15f374a297877577cb280de7
parent 6d9d168b0a429b615a055123a9020f496e712bed

hwc: more error checks

-- numHwLayers==1 means that there is only the FB_TARGET which HWC shouldn't
   draw. Also check for blank in set.  Checking for this in prepare prevents us
   from acquiring pipes.
-- add null check for private handle
-- print out vsync timestamp data on error

bug: 7274417
related-to-bug: 7288769
Change-Id: I8dae0d0748c5226fdc33ed8f5cc5259835dec2de
Signed-off-by: Iliyan Malchev <malchev@google.com>

libgralloc/framebuffer.cpp

diff --git a/libgralloc/framebuffer.cpp b/libgralloc/framebuffer.cpp
index f3cdd30..576e021 100644
--- a/libgralloc/framebuffer.cpp
+++ b/libgralloc/framebuffer.cpp
@@ -113,7 +113,7 @@
         m->info.yoffset = hnd->offset / m->finfo.line_length;
         m->commit.var = m->info;
         if (ioctl(m->framebuffer->fd, MSMFB_DISPLAY_COMMIT, &m->commit) == -1) {
-            ALOGE("%s: FBIOPUT_VSCREENINFO failed for external, err: %s", __FUNCTION__,
+            ALOGE("%s: MSMFB_DISPLAY_COMMIT ioctl failed, err: %s", __FUNCTION__,
                     strerror(errno));
             return -errno;
         }