commit f7fa2009b579aabf8275c8c96d3e2918b18b12bf
author Shuzhen Wang <shuzhenwang@google.com> Fri Aug 31 14:24:18 2018 -0700
committer Shuzhen Wang <shuzhenwang@google.com> Fri Aug 31 21:46:28 2018 +0000
tree 0589ed000ff3ee7d257218bfcf660492d5dc8f03
parent 8eed46df6794c7cd0a8501ad33592a681738e355

Camera: Do not treat non-increasing timestamp as fatal

Certain devices generate duplicate timestamps for cases like video
recording. So do not treat non-increasing timestamps as fatal for now.

Test: Failed camera CTS on Pixel device
Bug: 113670946
Change-Id: I3708f46111edf031f48f460e4efce9f9c46f174e

services/camera/libcameraservice/device3/Camera3Stream.cpp

diff --git a/services/camera/libcameraservice/device3/Camera3Stream.cpp b/services/camera/libcameraservice/device3/Camera3Stream.cpp
index 53ff9fe..b208d9f 100644
--- a/services/camera/libcameraservice/device3/Camera3Stream.cpp
+++ b/services/camera/libcameraservice/device3/Camera3Stream.cpp
@@ -663,9 +663,8 @@
     removeOutstandingBuffer(buffer);
 
     if (timestampIncreasing && timestamp != 0 && timestamp <= mLastTimestamp) {
-        ALOGE("%s: Stream %d: timestamp %" PRId64 " is not increasing. Prev timestamp %" PRId64,
+        ALOGW("%s: Stream %d: timestamp %" PRId64 " is not increasing. Prev timestamp %" PRId64,
                 __FUNCTION__, mId, timestamp, mLastTimestamp);
-        return BAD_VALUE;
     }
     mLastTimestamp = timestamp;