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

Message ID 20210701113442.111718-4-naush@raspberrypi.com
State Superseded
Headers show
Series
  • Raspberry Pi: Enable imx477 long exposure modes
Related show

Commit Message

Naushir Patuck July 1, 2021, 11:34 a.m. UTC 
With the recent change to allow long exposures on the imx477, the existing 100s
limit was not adequate.

Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
Reviewed-by: David Plowman <david.plowman@raspberrypi.com>
---
 src/ipa/raspberrypi/raspberrypi.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Laurent Pinchart July 1, 2021, 11:56 p.m. UTC | #1 
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@raspberrypi.com>
> Reviewed-by: David Plowman <david.plowman@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@ideasonboard.com>

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

Patch
diff mbox series 

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;
 
 /*
  * Determine the minimum allowable inter-frame duration to run the controller