mirror of
https://github.com/Evolution-X/hardware_interfaces
synced 2026-02-01 11:36:00 +00:00
Merge "Handle logical camera devices properly" into main am: 20768c89d7 am: b034621d77
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2901098 Change-Id: I9339a9827484c85b49be11b52fa1a5fb86fc8a6a Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -1399,6 +1399,12 @@ TEST_P(EvsAidlTest, HighPriorityCameraClient) {
|
||||
|
||||
// Test each reported camera
|
||||
for (auto&& cam : mCameraInfo) {
|
||||
bool isLogicalCam = false;
|
||||
if (getPhysicalCameraIds(cam.id, isLogicalCam); isLogicalCam) {
|
||||
LOG(INFO) << "Skip a logical device, " << cam.id;
|
||||
continue;
|
||||
}
|
||||
|
||||
// Request available display IDs
|
||||
uint8_t targetDisplayId = 0;
|
||||
std::vector<uint8_t> displayIds;
|
||||
@@ -1973,6 +1979,13 @@ TEST_P(EvsAidlTest, CameraStreamExternalBuffering) {
|
||||
|
||||
// Test each reported camera
|
||||
for (auto&& cam : mCameraInfo) {
|
||||
bool isLogicalCam = false;
|
||||
getPhysicalCameraIds(cam.id, isLogicalCam);
|
||||
if (isLogicalCam) {
|
||||
LOG(INFO) << "Skip a logical device, " << cam.id;
|
||||
continue;
|
||||
}
|
||||
|
||||
// Read a target resolution from the metadata
|
||||
Stream targetCfg = getFirstStreamConfiguration(
|
||||
reinterpret_cast<camera_metadata_t*>(cam.metadata.data()));
|
||||
@@ -2014,9 +2027,6 @@ TEST_P(EvsAidlTest, CameraStreamExternalBuffering) {
|
||||
}
|
||||
}
|
||||
|
||||
bool isLogicalCam = false;
|
||||
getPhysicalCameraIds(cam.id, isLogicalCam);
|
||||
|
||||
std::shared_ptr<IEvsCamera> pCam;
|
||||
ASSERT_TRUE(mEnumerator->openCamera(cam.id, targetCfg, &pCam).isOk());
|
||||
EXPECT_NE(pCam, nullptr);
|
||||
@@ -2027,11 +2037,6 @@ TEST_P(EvsAidlTest, CameraStreamExternalBuffering) {
|
||||
// Request to import buffers
|
||||
int delta = 0;
|
||||
auto status = pCam->importExternalBuffers(buffers, &delta);
|
||||
if (isLogicalCam) {
|
||||
ASSERT_FALSE(status.isOk());
|
||||
continue;
|
||||
}
|
||||
|
||||
ASSERT_TRUE(status.isOk());
|
||||
EXPECT_GE(delta, kBuffersToHold);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user