[libcamera-devel] [PATCH v2 3/4] ipa: raspberrypi: Increase the default max frame duration to 250s

[Laurent Pinchart]

Hi Naush,

Thank you for the patch.

On Thu, Jul 01, 2021 at 12:34:41PM +0100, Naushir Patuck wrote:
> With the recent change to allow long exposures on the imx477, the existing 100s
> limit was not adequate.
> 
> Signed-off-by: Naushir Patuck <naush at raspberrypi.com>
> Reviewed-by: David Plowman <david.plowman at raspberrypi.com>
> ---
>  src/ipa/raspberrypi/raspberrypi.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/ipa/raspberrypi/raspberrypi.cpp b/src/ipa/raspberrypi/raspberrypi.cpp
> index 4d09a84f6532..dd20c854c788 100644
> --- a/src/ipa/raspberrypi/raspberrypi.cpp
> +++ b/src/ipa/raspberrypi/raspberrypi.cpp
> @@ -62,7 +62,7 @@ using utils::Duration;
>  constexpr double DefaultAnalogueGain = 1.0;
>  constexpr Duration DefaultExposureTime = 20.0ms;
>  constexpr Duration defaultMinFrameDuration = 1.0s / 30.0;
> -constexpr Duration defaultMaxFrameDuration = 100.0s;
> +constexpr Duration defaultMaxFrameDuration = 250.0s;

I'll trust you that this is the correct value :-)

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

>  
>  /*
>   * Determine the minimum allowable inter-frame duration to run the controller

-- 
Regards,

Laurent Pinchart