[libcamera-devel] [PATCH v2 16/17] Revert "android: capabilities: Cap frame rate to 30 FPS"

Jacopo Mondi jacopo at jmondi.org
Tue Sep 7 21:41:06 CEST 2021


This reverts commit 257df463ad0e0862c211b03742140a99dd8cc0ef.
---
 src/android/camera_capabilities.cpp | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/src/android/camera_capabilities.cpp b/src/android/camera_capabilities.cpp
index cff1e30a9567..deaec3282fd7 100644
--- a/src/android/camera_capabilities.cpp
+++ b/src/android/camera_capabilities.cpp
@@ -632,29 +632,6 @@ int CameraCapabilities::initializeStreamConfigurations()
 
 			int64_t minFrameDuration = frameDurations->second.min().get<int64_t>() * 1000;
 			int64_t maxFrameDuration = frameDurations->second.max().get<int64_t>() * 1000;
-
-			/*
-			 * Cap min frame duration to 30 FPS.
-			 *
-			 * 30 frames per second has been validated as the most
-			 * opportune frame rate for quality tuning, and power
-			 * vs performances budget on Intel IPU3.
-			 *
-			 * \todo This is a platform-specific decision that needs
-			 * to be abstracted and delegated to the configuration
-			 * file.
-			 *
-			 * \todo libcamera only allows to control frame duration
-			 * through the per-request controls::FrameDuration
-			 * control. If we cap the durations here, we should be
-			 * capable of configuring the camera to operate at such
-			 * duration without requiring to have the FrameDuration
-			 * control to be specified for each Request. Defer this
-			 * to the in-development configuration API rework.
-			 */
-			if (minFrameDuration < 1e9 / 30.0)
-				minFrameDuration = 1e9 / 30.0;
-
 			streamConfigurations_.push_back({
 				res, androidFormat, minFrameDuration, maxFrameDuration,
 			});
-- 
2.32.0



More information about the libcamera-devel mailing list