[libcamera-devel] [PATCH v4 26/32] ipa: rkisp1: awb: Store color temperature as an integer

[Kieran Bingham]

Quoting Laurent Pinchart via libcamera-devel (2022-09-08 02:41:54)
> The color temperature doesn't need floating point precision, and is
> calculated by Awb::estimateCCT() as an unsigned integer. Store it with
> the same data type in the frame context.
> 

Excepting that I suspect we don't need to store the colour temperature
in the FrameContext, but it could go directly to metadata ... this is
fine if we don't want to get there yet.


Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  src/ipa/rkisp1/ipa_context.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/ipa/rkisp1/ipa_context.h b/src/ipa/rkisp1/ipa_context.h
> index c15b908afcc8..53787320e565 100644
> --- a/src/ipa/rkisp1/ipa_context.h
> +++ b/src/ipa/rkisp1/ipa_context.h
> @@ -100,7 +100,7 @@ struct RkISP1FrameContext : public FrameContext {
>                         double blue;
>                 } gains;
>  
> -               double temperatureK;
> +               unsigned int temperatureK;
>                 bool autoEnabled;
>         } awb;
>  
> -- 
> Regards,
> 
> Laurent Pinchart
>