From 08c7aa76919692fa44b0ddceb206cd78d6f263a5 Mon Sep 17 00:00:00 2001 From: nichcream Date: Tue, 13 Mar 2018 22:32:48 +0800 Subject: msm8996-common: camera: Improve min frame rate hack AFAIK, there are apps that request 15fps min frame rate; this is deliberate and we shouldn't change its default behaviour. But anything that sets the min fps the same as the max fps definitely should just stop. Change-Id: I5c3a31241009ab79edba4191910ad99d2db30f71 Signed-off-by: Davide Garberi --- camera/QCamera2/HAL/QCameraParameters.cpp | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'camera') diff --git a/camera/QCamera2/HAL/QCameraParameters.cpp b/camera/QCamera2/HAL/QCameraParameters.cpp index 57d8ad9..7832f76 100644 --- a/camera/QCamera2/HAL/QCameraParameters.cpp +++ b/camera/QCamera2/HAL/QCameraParameters.cpp @@ -6509,6 +6509,13 @@ int32_t QCameraParameters::setPreviewFpsRange(int min_fps, property_get("persist.debug.set.fixedfps", value, "0"); fixedFpsValue = atoi(value); + // Don't allow function callers to request min fps same as max fps + // I mean SnapdragonCamera. + if (max_fps >= 24000 && min_fps == max_fps) { + LOGH("min_fps %d same as max_fps %d, setting min_fps to 7000", min_fps, max_fps); + min_fps = 7000; + } + LOGD("E minFps = %d, maxFps = %d , vid minFps = %d, vid maxFps = %d", min_fps, max_fps, vid_min_fps, vid_max_fps); @@ -6519,9 +6526,8 @@ int32_t QCameraParameters::setPreviewFpsRange(int min_fps, } } snprintf(str, sizeof(str), "%d,%d", min_fps, max_fps); - LOGH("Actual preview fps range %s", str); - updateParamEntry(KEY_PREVIEW_FPS_RANGE, "7000,30000"); - LOGH("Setting the preview fps range 7000,30000"); + LOGH("Setting preview fps range %s", str); + updateParamEntry(KEY_PREVIEW_FPS_RANGE, str); cam_fps_range_t fps_range; memset(&fps_range, 0x00, sizeof(cam_fps_range_t)); fps_range.min_fps = (float)min_fps / 1000.0f; -- cgit v1.2.3