[PATCH 2/3] ipa: rkisp1: cproc: Merge identical functions

[Laurent Pinchart]

Hi Stefan,

Thank you for the patch.

On Thu, Jun 13, 2024 at 10:33:19AM +0200, Stefan Klug wrote:
> convertContrast() and convertSaturation() are equal. Merge them into
> one.
> 
> Signed-off-by: Stefan Klug <stefan.klug at ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  src/ipa/rkisp1/algorithms/cproc.cpp | 15 +++++----------
>  1 file changed, 5 insertions(+), 10 deletions(-)
> 
> diff --git a/src/ipa/rkisp1/algorithms/cproc.cpp b/src/ipa/rkisp1/algorithms/cproc.cpp
> index 955298093fda..9f400a52dd96 100644
> --- a/src/ipa/rkisp1/algorithms/cproc.cpp
> +++ b/src/ipa/rkisp1/algorithms/cproc.cpp
> @@ -44,12 +44,7 @@ int convertBrightness(const float v)
>  	return std::clamp<int>(std::lround(v * 128), -128, 127);
>  }
>  
> -int convertContrast(const float v)
> -{
> -	return std::clamp<int>(std::lround(v * 128), 0, 255);
> -}
> -
> -int convertSaturation(const float v)
> +int convertContrastOrSaturation(const float v)
>  {
>  	return std::clamp<int>(std::lround(v * 128), 0, 255);
>  }
> @@ -80,8 +75,8 @@ int ColorProcessing::configure([[maybe_unused]] IPAContext &context,
>  	auto &cproc = context.activeState.cproc;
>  
>  	cproc.brightness = convertBrightness(kDefaultBrightness);
> -	cproc.contrast = convertContrast(kDefaultContrast);
> -	cproc.saturation = convertSaturation(kDefaultSaturation);
> +	cproc.contrast = convertContrastOrSaturation(kDefaultContrast);
> +	cproc.saturation = convertContrastOrSaturation(kDefaultSaturation);
>  
>  	return 0;
>  }
> @@ -113,7 +108,7 @@ void ColorProcessing::queueRequest(IPAContext &context,
>  
>  	const auto &contrast = controls.get(controls::Contrast);
>  	if (contrast) {
> -		int value = convertContrast(*contrast);
> +		int value = convertContrastOrSaturation(*contrast);
>  		if (cproc.contrast != value) {
>  			cproc.contrast = value;
>  			update = true;
> @@ -124,7 +119,7 @@ void ColorProcessing::queueRequest(IPAContext &context,
>  
>  	const auto saturation = controls.get(controls::Saturation);
>  	if (saturation) {
> -		int value = convertSaturation(*saturation);
> +		int value = convertContrastOrSaturation(*saturation);
>  		if (cproc.saturation != value) {
>  			cproc.saturation = value;
>  			update = true;

-- 
Regards,

Laurent Pinchart