[libcamera-devel] [PATCH v2 4/4] ipa: raspberrypi: Fix default frame durations calculation
David Plowman
david.plowman at raspberrypi.com
Mon Mar 27 15:33:16 CEST 2023
Hi Naush
Thanks for the patch.
On Mon, 27 Mar 2023 at 10:34, Naushir Patuck via libcamera-devel
<libcamera-devel at lists.libcamera.org> wrote:
>
> Fix a bug in the default frame durations calculation where the min/max
> values are swapped round. This is a rarely travelled code path, so has
> not actually caused a reported failure.
>
> Signed-off-by: Naushir Patuck <naush at raspberrypi.com>
> ---
> src/ipa/raspberrypi/raspberrypi.cpp | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/ipa/raspberrypi/raspberrypi.cpp b/src/ipa/raspberrypi/raspberrypi.cpp
> index c10e57e07ab0..66a020d022cd 100644
> --- a/src/ipa/raspberrypi/raspberrypi.cpp
> +++ b/src/ipa/raspberrypi/raspberrypi.cpp
> @@ -1486,8 +1486,8 @@ void IPARPi::applyFrameDurations(Duration minFrameDuration, Duration maxFrameDur
> * This will only be applied once AGC recalculations occur.
> * The values may be clamped based on the sensor mode capabilities as well.
> */
> - minFrameDuration_ = minFrameDuration ? minFrameDuration : defaultMaxFrameDuration;
> - maxFrameDuration_ = maxFrameDuration ? maxFrameDuration : defaultMinFrameDuration;
> + minFrameDuration_ = minFrameDuration ? minFrameDuration : defaultMinFrameDuration;
> + maxFrameDuration_ = maxFrameDuration ? maxFrameDuration : defaultMaxFrameDuration;
Indeed, good spot!
Reviewed-by: David Plowman <david.plowman at raspberrypi.com>
Thanks!
David
> minFrameDuration_ = std::clamp(minFrameDuration_,
> mode_.minFrameDuration, mode_.maxFrameDuration);
> maxFrameDuration_ = std::clamp(maxFrameDuration_,
> --
> 2.34.1
>
More information about the libcamera-devel
mailing list