Message ID | 20190904125427.13943-5-jacopo@jmondi.org |
---|---|
State | Superseded |
Headers | show |
Series |
|
Related | show |
Hi Jacopo, Thank you for the patch. On Wed, Sep 04, 2019 at 02:54:24PM +0200, Jacopo Mondi wrote: > Add two missing tags from the request template generated by the HAL. > The tags are reported as missing by the cros_camera_test tool. > > Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > src/android/camera_device.cpp | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp > index bd69198ec482..969c8192c316 100644 > --- a/src/android/camera_device.cpp > +++ b/src/android/camera_device.cpp > @@ -611,6 +611,17 @@ const camera_metadata_t *CameraDevice::constructDefaultRequestSettings(int type) > &faceDetectMode, 1); > METADATA_ASSERT(ret); > > + uint8_t noiseReduction = ANDROID_NOISE_REDUCTION_MODE_OFF; > + ret = add_camera_metadata_entry(requestTemplate_, > + ANDROID_NOISE_REDUCTION_MODE, &noiseReduction, 1); > + METADATA_ASSERT(ret); > + > + uint8_t aberrationMode = ANDROID_COLOR_CORRECTION_ABERRATION_MODE_OFF; > + ret = add_camera_metadata_entry(requestTemplate_, > + ANDROID_COLOR_CORRECTION_ABERRATION_MODE, > + &aberrationMode, 1); > + METADATA_ASSERT(ret); > + > ret = add_camera_metadata_entry(requestTemplate_, > ANDROID_CONTROL_CAPTURE_INTENT, > &captureIntent, 1);
Hi Jacopo, Thanks for your work. On 2019-09-04 14:54:24 +0200, Jacopo Mondi wrote: > Add two missing tags from the request template generated by the HAL. > The tags are reported as missing by the cros_camera_test tool. > > Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se> > --- > src/android/camera_device.cpp | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp > index bd69198ec482..969c8192c316 100644 > --- a/src/android/camera_device.cpp > +++ b/src/android/camera_device.cpp > @@ -611,6 +611,17 @@ const camera_metadata_t *CameraDevice::constructDefaultRequestSettings(int type) > &faceDetectMode, 1); > METADATA_ASSERT(ret); > > + uint8_t noiseReduction = ANDROID_NOISE_REDUCTION_MODE_OFF; > + ret = add_camera_metadata_entry(requestTemplate_, > + ANDROID_NOISE_REDUCTION_MODE, &noiseReduction, 1); > + METADATA_ASSERT(ret); > + > + uint8_t aberrationMode = ANDROID_COLOR_CORRECTION_ABERRATION_MODE_OFF; > + ret = add_camera_metadata_entry(requestTemplate_, > + ANDROID_COLOR_CORRECTION_ABERRATION_MODE, > + &aberrationMode, 1); > + METADATA_ASSERT(ret); > + > ret = add_camera_metadata_entry(requestTemplate_, > ANDROID_CONTROL_CAPTURE_INTENT, > &captureIntent, 1); > -- > 2.23.0 > > _______________________________________________ > libcamera-devel mailing list > libcamera-devel@lists.libcamera.org > https://lists.libcamera.org/listinfo/libcamera-devel
diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp index bd69198ec482..969c8192c316 100644 --- a/src/android/camera_device.cpp +++ b/src/android/camera_device.cpp @@ -611,6 +611,17 @@ const camera_metadata_t *CameraDevice::constructDefaultRequestSettings(int type) &faceDetectMode, 1); METADATA_ASSERT(ret); + uint8_t noiseReduction = ANDROID_NOISE_REDUCTION_MODE_OFF; + ret = add_camera_metadata_entry(requestTemplate_, + ANDROID_NOISE_REDUCTION_MODE, &noiseReduction, 1); + METADATA_ASSERT(ret); + + uint8_t aberrationMode = ANDROID_COLOR_CORRECTION_ABERRATION_MODE_OFF; + ret = add_camera_metadata_entry(requestTemplate_, + ANDROID_COLOR_CORRECTION_ABERRATION_MODE, + &aberrationMode, 1); + METADATA_ASSERT(ret); + ret = add_camera_metadata_entry(requestTemplate_, ANDROID_CONTROL_CAPTURE_INTENT, &captureIntent, 1);
Add two missing tags from the request template generated by the HAL. The tags are reported as missing by the cros_camera_test tool. Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> --- src/android/camera_device.cpp | 11 +++++++++++ 1 file changed, 11 insertions(+)