[PATCH v3 3/5] ipa: rkisp1: Fix histogram weights configuration

Stefan Klug stefan.klug at ideasonboard.com
Fri Feb 23 14:07:08 CET 2024


Am 18.02.24 um 17:49 schrieb Laurent Pinchart:
> The histogram weights are initialized to hardcoded 1's for each
> histogram grid cell. The code uses the wrong variable for the grid size,
> resulting in some weights having a 0 value. Fix it.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Stefan Klug <stefan.klug at ideasonboard.com>

> ---
>   src/ipa/rkisp1/algorithms/agc.cpp | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/ipa/rkisp1/algorithms/agc.cpp b/src/ipa/rkisp1/algorithms/agc.cpp
> index da705b14754c..47a6f7b26744 100644
> --- a/src/ipa/rkisp1/algorithms/agc.cpp
> +++ b/src/ipa/rkisp1/algorithms/agc.cpp
> @@ -175,7 +175,7 @@ void Agc::prepare(IPAContext &context, const uint32_t frame,
>   	/* Set an average weighted histogram. */
>   	Span<uint8_t> weights{
>   		params->meas.hst_config.hist_weight,
> -		context.hw->numHistogramBins
> +		context.hw->numHistogramWeights
>   	};
>   	std::fill(weights.begin(), weights.end(), 1);
>   	/* Step size can't be less than 3. */

-- 
Regards,

Stefan Klug



More information about the libcamera-devel mailing list