HACK: exynos5: gscaler: do not skip s_fmt/reqbufs on gsc_convert path
authorDima Zavin <dima@android.com>
Sat, 4 Aug 2012 08:15:15 +0000 (01:15 -0700)
committerDima Zavin <dima@android.com>
Sun, 5 Aug 2012 04:41:06 +0000 (21:41 -0700)
This fixes the bug where still capture always captures the same
image and video recording only taking first image.

The issue is that reqbufs is reset to 0 and fmt is not set
when the buffers try to be enqueued and stremaing turned on in the path
from csc_convert().

This is temporary as the gscaler code really needs a huge cleanup.

Bug: 6925368
Change-Id: Ia8e3aee0e0b50c6a69e1eda219e484713088c042
Signed-off-by: Dima Zavin <dima@android.com>
libgscaler/exynos_gscaler.c

index 821d2583e5fa7a86fea9abef24f8cd688815ddc6..924d01ecbe84aafdd573ddc5b31e9239685e7a7c 100755 (executable)
@@ -675,7 +675,6 @@ static bool m_exynos_gsc_set_format(
 
         // skip s_fmt
         ALOGV("%s::fmt, crop is same with old-one, so skip s_fmt crop..", __func__);
-        return true;
     }
 
 set_hw: