<div dir="ltr"><div dir="ltr">Hi David,<div><br></div><div>Thank you for your review comments.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 10 Dec 2020 at 08:45, David Plowman <<a href="mailto:david.plowman@raspberrypi.com">david.plowman@raspberrypi.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">Hi Naush<br>
<br>
Thanks for the patch, I have been eagerly awaiting this set!<br>
<br>
On Wed, 9 Dec 2020 at 10:26, Naushir Patuck <<a href="mailto:naush@raspberrypi.com" target="_blank">naush@raspberrypi.com</a>> wrote:<br>
><br>
> Add a float array control (controls::FrameDurations) to specify the<br>
> minimum and maximum (in that order) frame duration to be used by the<br>
> camera sensor.<br>
><br>
> Signed-off-by: Naushir Patuck <<a href="mailto:naush@raspberrypi.com" target="_blank">naush@raspberrypi.com</a>><br>
> ---<br>
>  src/libcamera/control_ids.yaml | 13 +++++++++++++<br>
>  1 file changed, 13 insertions(+)<br>
><br>
> diff --git a/src/libcamera/control_ids.yaml b/src/libcamera/control_ids.yaml<br>
> index 6d6f0fee..cd44a620 100644<br>
> --- a/src/libcamera/control_ids.yaml<br>
> +++ b/src/libcamera/control_ids.yaml<br>
> @@ -554,4 +554,17 @@ controls:<br>
>  Â  Â  Â  Â  detection, additional format conversions etc) count as an additional<br>
>  Â  Â  Â  Â  pipeline stage.<br>
><br>
> +  - FrameDurations:<br>
> +  Â  Â  type: float<br>
> +  Â  Â  description: |<br>
> +  Â  Â  Â  Specifies the minimum and maximum (in that order) allowable frame<br>
> +  Â  Â  Â  duration, in micro-seconds, for the sensor to use. This could also limit<br>
> +  Â  Â  Â  the largest exposure times the sensor can use. For example, if a maximum<br>
> +  Â  Â  Â  frame duration of 33ms is requested (corresponding to 30 frames per<br>
> +  Â  Â  Â  second), the sensor will not be able raise the exposure time above 33ms.<br>
> +  Â  Â  Â  Note that the sensor may not always be able to provide the requested<br>
> +  Â  Â  Â  frame duration limits depending on its mode configuration.<br>
<br>
Is it worth remarking that they can be set to the same values to give<br>
a fixed fps mode of operations? Or is that too obvious to mention?<br></blockquote><div><br></div><div>No harm in adding it I suppose.  Will make the change and post an update.</div><div><br></div><div>Regards,</div><div>Naush</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>
Reviewed-by: David Plowman <<a href="mailto:david.plowman@raspberrypi.com" target="_blank">david.plowman@raspberrypi.com</a>><br>
Tested-by: David Plowman <<a href="mailto:david.plowman@raspberrypi.com" target="_blank">david.plowman@raspberrypi.com</a>><br>
<br>
Best regards<br>
David<br>
<br>
> +<br>
> +  Â  Â  Â  \sa ExposureTime<br>
> +  Â  Â  size: [2]<br>
>  ...<br>
> --<br>
> 2.25.1<br>
><br>
> _______________________________________________<br>
> libcamera-devel mailing list<br>
> <a href="mailto:libcamera-devel@lists.libcamera.org" target="_blank">libcamera-devel@lists.libcamera.org</a><br>
> <a href="https://lists.libcamera.org/listinfo/libcamera-devel" rel="noreferrer" target="_blank">https://lists.libcamera.org/listinfo/libcamera-devel</a><br>
</blockquote></div></div>