[PATCH] libcamera: software_isp: Add a clarification comment to AWB

[Kieran Bingham]

Quoting Milan Zamazal (2025-04-03 13:53:55)
> The computed AWB gains are applied when constructing LUT tables rather
> than in awb.cpp itself.  This can look confusing when reading awb.cpp,
> let's add a clarifying comment.
> 
> Signed-off-by: Milan Zamazal <mzamazal at redhat.com>

I don't object:


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

> ---
>  src/ipa/simple/algorithms/awb.cpp | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/ipa/simple/algorithms/awb.cpp b/src/ipa/simple/algorithms/awb.cpp
> index 55719059..cf567e89 100644
> --- a/src/ipa/simple/algorithms/awb.cpp
> +++ b/src/ipa/simple/algorithms/awb.cpp
> @@ -40,6 +40,7 @@ void Awb::prepare(IPAContext &context,
>                   [[maybe_unused]] DebayerParams *params)
>  {
>         auto &gains = context.activeState.awb.gains;
> +       /* Just report, the gains are applied in LUT algorithm. */
>         frameContext.gains.red = gains.r();
>         frameContext.gains.blue = gains.b();
>  }
> -- 
> 2.49.0
>