[PATCH 06/12] ipa: rkisp1: agc: Make size argument to computeHistogramPredivider const

Paul Elder paul.elder at ideasonboard.com
Mon Jun 17 12:12:37 CEST 2024


On Sun, Jun 16, 2024 at 07:39:04PM +0300, Laurent Pinchart wrote:
> The Agc::computeHistogramPredivider() function doesn't need to modify
> its size parameter. Make it const.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>

> ---
>  src/ipa/rkisp1/algorithms/agc.cpp | 3 ++-
>  src/ipa/rkisp1/algorithms/agc.h   | 2 +-
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/src/ipa/rkisp1/algorithms/agc.cpp b/src/ipa/rkisp1/algorithms/agc.cpp
> index 0f60d08fd41c..9dac60bdb24e 100644
> --- a/src/ipa/rkisp1/algorithms/agc.cpp
> +++ b/src/ipa/rkisp1/algorithms/agc.cpp
> @@ -84,7 +84,8 @@ int Agc::parseMeteringModes(IPAContext &context, const YamlObject &tuningData)
>  	return 0;
>  }
>  
> -uint8_t Agc::computeHistogramPredivider(Size &size, enum rkisp1_cif_isp_histogram_mode mode)
> +uint8_t Agc::computeHistogramPredivider(const Size &size,
> +					enum rkisp1_cif_isp_histogram_mode mode)
>  {
>  	/*
>  	 * The maximum number of pixels that could potentially be in one bin is
> diff --git a/src/ipa/rkisp1/algorithms/agc.h b/src/ipa/rkisp1/algorithms/agc.h
> index 996fea71755c..9ceaa82b099e 100644
> --- a/src/ipa/rkisp1/algorithms/agc.h
> +++ b/src/ipa/rkisp1/algorithms/agc.h
> @@ -45,7 +45,7 @@ public:
>  
>  private:
>  	int parseMeteringModes(IPAContext &context, const YamlObject &tuningData);
> -	uint8_t computeHistogramPredivider(Size &size,
> +	uint8_t computeHistogramPredivider(const Size &size,
>  					   enum rkisp1_cif_isp_histogram_mode mode);
>  
>  	void fillMetadata(IPAContext &context, IPAFrameContext &frameContext,


More information about the libcamera-devel mailing list