[PATCH v1] ipa: rpi: awb: Fix incorrect parameter retrieval

[Kieran Bingham]

Quoting Barnabás Pőcze (2024-05-20 04:50:22)
> During the refactoring in the mentioned commit the quotation
> marks around the word *fast* were missed. Unfortunately there
> was a variable named *fast* and an suitable overload for `operator[]`
> for things to compile fine.

Ouch!

> Fixes: c1597f98965461 ("ipa: raspberrypi: Use YamlParser to replace dependency on boost")

And this has been in for quite some time, Great find!


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


> Signed-off-by: Barnabás Pőcze <pobrn at protonmail.com>
> ---
>  src/ipa/rpi/controller/rpi/awb.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/ipa/rpi/controller/rpi/awb.cpp b/src/ipa/rpi/controller/rpi/awb.cpp
> index abe5906e..94ecbfc2 100644
> --- a/src/ipa/rpi/controller/rpi/awb.cpp
> +++ b/src/ipa/rpi/controller/rpi/awb.cpp
> @@ -161,7 +161,7 @@ int AwbConfig::read(const libcamera::YamlObject &params)
>                         bayes = false;
>                 }
>         }
> -       fast = params[fast].get<int>(bayes); /* default to fast for Bayesian, otherwise slow */
> +       fast = params["fast"].get<int>(bayes); /* default to fast for Bayesian, otherwise slow */
>         whitepointR = params["whitepoint_r"].get<double>(0.0);
>         whitepointB = params["whitepoint_b"].get<double>(0.0);
>         if (bayes == false)
> -- 
> 2.45.1
> 
>