Update VtsHalEvsV1_*TargetTest
- Removes a requirement of the landscape aspect ratio of the captured
frames because the Extended View System now allows the video streams
in any aspect ratio
- Increases the maximum latency of the video stream start request from
500 ms to 1 second to accommodate the camera system design with the
additional processing units outside of the head unit
- Reduces a number of additional buffers to allocate from 6 to 2 in
CameraStreamBuffering test, to be consistent with comments
Bug: 204835095
Bug: 305642713
Test: atest VtsHalEvsV1_0TargetTest and
atest VtsHalEvsV1_1TargetTest
Change-Id: Id41bde10964b650f78e76d4f97a4588367f7c7fc
Merged-In: Id41bde10964b650f78e76d4f97a4588367f7c7fc
(cherry picked from commit 7bc98ce6ed
)
This commit is contained in:
parent
775b6eca77
commit
ab78601274
2 changed files with 3 additions and 14 deletions
|
@ -17,8 +17,7 @@
|
|||
#define LOG_TAG "VtsHalEvsTest"
|
||||
|
||||
|
||||
// These values are called out in the EVS design doc (as of Mar 8, 2017)
|
||||
static const int kMaxStreamStartMilliseconds = 500;
|
||||
static const int kMaxStreamStartMilliseconds = 1000;
|
||||
static const int kMinimumFramesPerSecond = 10;
|
||||
|
||||
static const int kSecondsToMilliseconds = 1000;
|
||||
|
@ -332,11 +331,6 @@ TEST_P(EvsHidlTest, CameraStreamPerformance) {
|
|||
printf("Measured time to first frame %0.2f ms\n", timeToFirstFrame * kNanoToMilliseconds);
|
||||
ALOGI("Measured time to first frame %0.2f ms", timeToFirstFrame * kNanoToMilliseconds);
|
||||
|
||||
// Check aspect ratio
|
||||
unsigned width = 0, height = 0;
|
||||
frameHandler->getFrameDimension(&width, &height);
|
||||
EXPECT_GE(width, height);
|
||||
|
||||
// Wait a bit, then ensure we get at least the required minimum number of frames
|
||||
sleep(5);
|
||||
nsecs_t end = systemTime(SYSTEM_TIME_MONOTONIC);
|
||||
|
@ -370,7 +364,7 @@ TEST_P(EvsHidlTest, CameraStreamBuffering) {
|
|||
ALOGI("Starting CameraStreamBuffering test");
|
||||
|
||||
// Arbitrary constant (should be > 1 and not too big)
|
||||
static const unsigned int kBuffersToHold = 6;
|
||||
static const unsigned int kBuffersToHold = 2;
|
||||
|
||||
// Get the camera list
|
||||
loadCameraList();
|
||||
|
|
|
@ -498,11 +498,6 @@ TEST_P(EvsHidlTest, CameraStreamPerformance) {
|
|||
<< std::scientific << timeToFirstFrame * kNanoToMilliseconds
|
||||
<< " ms.";
|
||||
|
||||
// Check aspect ratio
|
||||
unsigned width = 0, height = 0;
|
||||
frameHandler->getFrameDimension(&width, &height);
|
||||
EXPECT_GE(width, height);
|
||||
|
||||
// Wait a bit, then ensure we get at least the required minimum number of frames
|
||||
sleep(5);
|
||||
nsecs_t end = systemTime(SYSTEM_TIME_MONOTONIC);
|
||||
|
@ -540,7 +535,7 @@ TEST_P(EvsHidlTest, CameraStreamBuffering) {
|
|||
LOG(INFO) << "Starting CameraStreamBuffering test";
|
||||
|
||||
// Arbitrary constant (should be > 1 and not too big)
|
||||
static const unsigned int kBuffersToHold = 6;
|
||||
static const unsigned int kBuffersToHold = 2;
|
||||
|
||||
// Get the camera list
|
||||
loadCameraList();
|
||||
|
|
Loading…
Reference in a new issue