[libcamera-devel] [RFC PATCH v2 04/12] android: jpeg: get ISO from SENSOR_SENSITIVITY
Jacopo Mondi
jacopo at jmondi.org
Tue Apr 27 09:47:09 CEST 2021
On Thu, Apr 22, 2021 at 06:40:54PM +0900, Paul Elder wrote:
> The data for the exif ISO tag needs to come from SENSOR_SENSITIVITY. Set
> it.
>
> Signed-off-by: Paul Elder <paul.elder at ideasonboard.com>
>
> ---
> This on its own doesn't fix any CTS tests, but it prevents a test
> failure later on when we add the proper static metadata for the FULL
> hardware level.
> ---
> src/android/jpeg/post_processor_jpeg.cpp | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/src/android/jpeg/post_processor_jpeg.cpp b/src/android/jpeg/post_processor_jpeg.cpp
> index 3bf16996..237fb318 100644
> --- a/src/android/jpeg/post_processor_jpeg.cpp
> +++ b/src/android/jpeg/post_processor_jpeg.cpp
> @@ -119,7 +119,10 @@ int PostProcessorJpeg::process(const FrameBuffer &source,
> ret = requestMetadata.getEntry(ANDROID_LENS_APERTURE, &entry);
> if (ret)
> exif.setAperture(*entry.data.f);
> - exif.setISO(100);
> +
> + ret = resultMetadata->getEntry(ANDROID_SENSOR_SENSITIVITY, &entry);
I don't see this tag being filled in the result metadata...
> + exif.setISO(ret ? *entry.data.i32 : 100);
> +
> exif.setFlash(Exif::Flash::FlashNotPresent);
> exif.setWhiteBalance(Exif::WhiteBalance::Auto);
>
> --
> 2.27.0
>
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
More information about the libcamera-devel
mailing list