HACK: exynos5: gscaler: do not skip s_fmt/reqbufs on gsc_convert path

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>

diff --git a/libgscaler/exynos_gscaler.c b/libgscaler/exynos_gscaler.c
index 821d2583e5fa7a86fea9abef24f8cd688815ddc6..924d01ecbe84aafdd573ddc5b31e9239685e7a7c 100755 (executable)
--- a/libgscaler/exynos_gscaler.c
+++ b/libgscaler/exynos_gscaler.c
@@ -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: