[libcamera-devel] [PATCH v4 2/4] android: Disallow external location in HAL config
Umang Jain
umang.jain at ideasonboard.com
Tue Aug 3 16:02:35 CEST 2021
Error out on any camera's location if set to "external", in the HAL
configuration file. The HAL configuration file is meant to override the
location property, and overriding an internal camera location to
external doesn't make sense.
Signed-off-by: Umang Jain <umang.jain at ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>
---
src/android/camera_hal_config.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/android/camera_hal_config.cpp b/src/android/camera_hal_config.cpp
index 833cf4ba..7126aba4 100644
--- a/src/android/camera_hal_config.cpp
+++ b/src/android/camera_hal_config.cpp
@@ -127,8 +127,6 @@ int CameraHalConfig::Private::parseCameraLocation(CameraConfigData *cameraConfig
cameraConfigData->facing = CAMERA_FACING_FRONT;
else if (location == "back")
cameraConfigData->facing = CAMERA_FACING_BACK;
- else if (location == "external")
- cameraConfigData->facing = CAMERA_FACING_EXTERNAL;
else
return -EINVAL;
--
2.31.1
More information about the libcamera-devel
mailing list