[PATCH] ipa: rpi: cam_helper: imx708: Correct the frame integration diff value

[Naushir Patuck]

Hi David,

Thanks for the fix!

On Thu, 12 Dec 2024 at 10:24, David Plowman
<david.plowman at raspberrypi.com> wrote:
>
> This is the minimum permitted difference between the frame length and
> coarse exposure (both measured in lines). The correct value, taken
> from the datasheet, is 48.
>
> Signed-off-by: David Plowman <david.plowman at raspberrypi.com>

Reviewed-by: Naushir Patuck <naush at raspberrypi.com>

> ---
>  src/ipa/rpi/cam_helper/cam_helper_imx708.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/ipa/rpi/cam_helper/cam_helper_imx708.cpp b/src/ipa/rpi/cam_helper/cam_helper_imx708.cpp
> index ec83d9fd..16a017e9 100644
> --- a/src/ipa/rpi/cam_helper/cam_helper_imx708.cpp
> +++ b/src/ipa/rpi/cam_helper/cam_helper_imx708.cpp
> @@ -66,7 +66,7 @@ private:
>          * Smallest difference between the frame length and integration time,
>          * in units of lines.
>          */
> -       static constexpr int frameIntegrationDiff = 22;
> +       static constexpr int frameIntegrationDiff = 48;
>         /* Maximum frame length allowable for long exposure calculations. */
>         static constexpr int frameLengthMax = 0xffdc;
>         /* Largest long exposure scale factor given as a left shift on the frame length. */
> --
> 2.39.5
>