From a1f4fae05db0af7c1ab17d9566e6786f4bdcf7ff Mon Sep 17 00:00:00 2001 From: Ricardo Cerqueira Date: Sun, 1 Jun 2014 06:45:36 +0100 Subject: camera: Disable extra HDR frame on QCOM_HARDWARE Qualcomm camera HALs default to adding an extra zero-exposure frame to HDR snapshots; this is breaking third-party apps, and we don't use it in system-bundled apps, so disable it unless explicitly requested by the client Change-Id: Iecb868c5c344d972de7f36dc1bd9cc9fdbabaf4e --- camera/CameraParameters.cpp | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'camera') diff --git a/camera/CameraParameters.cpp b/camera/CameraParameters.cpp index d35519d..42b0884 100644 --- a/camera/CameraParameters.cpp +++ b/camera/CameraParameters.cpp @@ -257,6 +257,14 @@ void CameraParameters::set(const char *key, const char *value) //XXX ALOGE("Value \"%s\"contains invalid character (= or ;)", value); return; } +#ifdef QCOM_HARDWARE + // qcom cameras default to delivering an extra zero-exposure frame on HDR. + // The android SDK only wants one frame, so disable this unless the app + // explicitly asks for it + if (!get("hdr-need-1x")) { + mMap.replaceValueFor(String8("hdr-need-1x"), String8("false")); + } +#endif mMap.replaceValueFor(String8(key), String8(value)); } -- cgit v1.1