summaryrefslogtreecommitdiff
path: root/src/android
diff options
context:
space:
mode:
authorJacopo Mondi <jacopo@jmondi.org>2020-12-30 17:18:51 +0100
committerJacopo Mondi <jacopo@jmondi.org>2021-01-18 10:39:14 +0100
commitcda41ff754816cfcbe087e0f6f985f7b69c3fa0f (patch)
treee9ee0d1147a8c3336e66f0eea79519ff7a8801b2 /src/android
parentdb6a3bfa6e92081c33463ce58c51ac8fae7f854c (diff)
android: camera_device: Do not default pixel array properties
Now that the pixel array properties have been defaulted in the CameraSensor class (or in the pipeline handler, for the UVC use case), they will always be reported by the libcamera::Camera and there's no need to default them in the Camera HAL. Remove defaults and assume properties are always there. Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
Diffstat (limited to 'src/android')
-rw-r--r--src/android/camera_device.cpp20
1 files changed, 2 insertions, 18 deletions
diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp
index 23be2a59..a136f8e3 100644
--- a/src/android/camera_device.cpp
+++ b/src/android/camera_device.cpp
@@ -851,7 +851,7 @@ const camera_metadata_t *CameraDevice::getStaticMetadata()
staticMetadata_->addEntry(ANDROID_JPEG_MAX_SIZE, &maxJpegBufferSize_, 1);
/* Sensor static metadata. */
- if (properties.contains(properties::PixelArraySize)) {
+ {
const Size &size =
properties.get(properties::PixelArraySize);
std::vector<int32_t> data{
@@ -860,17 +860,9 @@ const camera_metadata_t *CameraDevice::getStaticMetadata()
};
staticMetadata_->addEntry(ANDROID_SENSOR_INFO_PIXEL_ARRAY_SIZE,
data.data(), data.size());
- } else {
- /*
- * \todo Drop the default once the ov5670 and ov13858 drivers
- * are updated to report the pixel array size.
- */
- int32_t data[] = { 2592, 1944 };
- staticMetadata_->addEntry(ANDROID_SENSOR_INFO_PIXEL_ARRAY_SIZE,
- data, 2);
}
- if (properties.contains(properties::PixelArrayActiveAreas)) {
+ {
const Span<const Rectangle> &rects =
properties.get(properties::PixelArrayActiveAreas);
std::vector<int32_t> data{
@@ -881,14 +873,6 @@ const camera_metadata_t *CameraDevice::getStaticMetadata()
};
staticMetadata_->addEntry(ANDROID_SENSOR_INFO_ACTIVE_ARRAY_SIZE,
data.data(), data.size());
- } else {
- /*
- * \todo Drop the default once the ov5670 and ov13858 drivers
- * are updated to report the pixel array size.
- */
- int32_t data[] = { 0, 0, 2560, 1920 };
- staticMetadata_->addEntry(ANDROID_SENSOR_INFO_ACTIVE_ARRAY_SIZE,
- data, 4);
}
int32_t sensitivityRange[] = {