Message ID | 20210716105631.158153-5-paul.elder@ideasonboard.com |
---|---|
State | Superseded |
Headers | show |
Series |
|
Related | show |
Hi Paul, On Fri, Jul 16, 2021 at 07:56:14PM +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@ideasonboard.com> > Reviewed-by: Laurent Pinchart <laurent.pinchart@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. It doesn't hurt indeed! Reviewed-by: Jacopo Mondi <jacopo@jmondi.org> Thanks j > --- > 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 0e93f365..3160a784 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); > + exif.setISO(ret ? *entry.data.i32 : 100); > + > exif.setFlash(Exif::Flash::FlashNotPresent); > exif.setWhiteBalance(Exif::WhiteBalance::Auto); > > -- > 2.27.0 >
diff --git a/src/android/jpeg/post_processor_jpeg.cpp b/src/android/jpeg/post_processor_jpeg.cpp index 0e93f365..3160a784 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); + exif.setISO(ret ? *entry.data.i32 : 100); + exif.setFlash(Exif::Flash::FlashNotPresent); exif.setWhiteBalance(Exif::WhiteBalance::Auto);