summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMinghui Tan <mhtan@google.com>2021-07-14 14:27:07 -0700
committerMinghui Tan <mhtan@google.com>2021-07-14 21:56:18 +0000
commite3451b87d4da7467390383800c59eb607df123ee (patch)
tree865eb27b75b87093ea2d3afd4dbf0a4278f009b3
parent1d8a9e70bedf83790ef477e70dec19d8a33b8ecd (diff)
downloadcamera-e3451b87d4da7467390383800c59eb607df123ee.tar.gz
Support logical camera only for software denoise.
Bug: 193671784 Fix: 193671784 Test: Instagram, Snap Change-Id: Ifca0ddc33dfe490d2de5090ceae810b6699ab91f
-rw-r--r--common/hal/google_camera_hal/zsl_snapshot_capture_session.cc20
1 files changed, 2 insertions, 18 deletions
diff --git a/common/hal/google_camera_hal/zsl_snapshot_capture_session.cc b/common/hal/google_camera_hal/zsl_snapshot_capture_session.cc
index 56fb96d..a7c0fd8 100644
--- a/common/hal/google_camera_hal/zsl_snapshot_capture_session.cc
+++ b/common/hal/google_camera_hal/zsl_snapshot_capture_session.cc
@@ -163,17 +163,12 @@ bool ZslSnapshotCaptureSession::IsStreamConfigurationSupported(
return false;
}
- bool has_physical_camera_stream = false;
- bool has_non_physical_camera_stream = false;
bool has_jpeg_stream = false;
bool has_preview_stream = false;
- std::set<int> physical_camera_in_use;
for (const auto& stream : stream_config.streams) {
if (stream.is_physical_camera_stream) {
- has_physical_camera_stream = true;
- physical_camera_in_use.insert(stream.physical_camera_id);
- } else {
- has_non_physical_camera_stream = true;
+ ALOGE("%s: support logical camera only", __FUNCTION__);
+ return false;
}
if (utils::IsJPEGSnapshotStream(stream)) {
has_jpeg_stream = true;
@@ -194,17 +189,6 @@ bool ZslSnapshotCaptureSession::IsStreamConfigurationSupported(
return false;
}
- if (has_physical_camera_stream && has_non_physical_camera_stream) {
- ALOGE("%s: support only physical camera or logical camera, but not both",
- __FUNCTION__);
- return false;
- }
-
- if (has_physical_camera_stream && physical_camera_in_use.size() != 1) {
- ALOGE("%s: support only 1 physical camera, but %d have set up streams",
- __FUNCTION__, static_cast<int>(physical_camera_in_use.size()));
- }
-
ALOGD("%s: ZslSnapshotCaptureSession supports the stream config",
__FUNCTION__);
return true;