<div dir="ltr"><div dir="ltr">Hi Paul, thank you for the patch.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 2, 2021 at 12:32 PM <<a href="mailto:paul.elder@ideasonboard.com">paul.elder@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Jun 02, 2021 at 12:30:21PM +0900, <a href="mailto:paul.elder@ideasonboard.com" target="_blank">paul.elder@ideasonboard.com</a> wrote:<br>
> Hi Umang,<br>
> <br>
> On Tue, Jun 01, 2021 at 04:54:55PM +0530, Umang Jain wrote:<br>
> > Report ANDROID_SENSOR_FRAME_DURATION as an available key for CTS to<br>
> <br>
> s/available key/available result key/<br>
> <br>
> It's also a valid request key, and this patch doesn't add that, so I<br>
> think it should be specified (also in the subject).<br>
<br>
By "specified" I mean that "result" should be specified.<br>
<br>
As for the subject, s/key/result key/<br>
<br>
<br>
Paul<br>
<br>
> <br>
> > read out the value of frame duration we set in CameraDevice::getResultMetadata().<br>
> > Failing to do so might fail the CTS test:<br>
> >  - android.hardware.camera2.cts.CaptureRequestTest#testNoiseReductionModeControl<br>
> > <br>
> > Fixes: 3beb1accac1d ("android: camera_device: Fix sensor frame duration")<br>
> > Signed-off-by: Umang Jain <<a href="mailto:umang.jain@ideasonboard.com" target="_blank">umang.jain@ideasonboard.com</a>><br></blockquote><div><br></div><div>First, this code adds the missing request result key. So</div><div>Reviewed-by: Hirokazu Honda <<a href="mailto:hiroh@chromium.org">hiroh@chromium.org</a>></div><div><br></div><div>However, although this is not necessarily related to this change, I wonder if those available keys should be there if and only if an entry with the key is actually added.</div><div>Could you run android.hardware.camera2.cts.CaptureResultTest while some entry is dropped and the entry is in available keys?</div><div> </div><div>-Hiro</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> > ---<br>
> >  src/android/camera_device.cpp | 1 +<br>
> >  1 file changed, 1 insertion(+)<br>
> > <br>
> > diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp<br>
> > index fddc07ff..fe332ec3 100644<br>
> > --- a/src/android/camera_device.cpp<br>
> > +++ b/src/android/camera_device.cpp<br>
> > @@ -1422,6 +1422,7 @@ const camera_metadata_t *CameraDevice::getStaticMetadata()<br>
> >             ANDROID_REQUEST_PIPELINE_DEPTH,<br>
> >             ANDROID_SCALER_CROP_REGION,<br>
> >             ANDROID_SENSOR_EXPOSURE_TIME,<br>
> > +           ANDROID_SENSOR_FRAME_DURATION,<br>
> >             ANDROID_SENSOR_ROLLING_SHUTTER_SKEW,<br>
> >             ANDROID_SENSOR_TEST_PATTERN_MODE,<br>
> >             ANDROID_SENSOR_TIMESTAMP,<br>
> > -- <br>
> > 2.31.1<br>
> > <br>
</blockquote></div></div>