[PATCH v1 6/6] utils: tuning: rkisp1: Replace static AWB with new AWB module

Kieran Bingham kieran.bingham at ideasonboard.com
Mon Aug 5 15:52:38 CEST 2024


Quoting Stefan Klug (2024-08-05 13:05:07)
> The rkisp1 tuner used a static module to insert the AWB algorithm into
> the tuning file. Replace that with the new AWB module.
> 
> Signed-off-by: Stefan Klug <stefan.klug at ideasonboard.com>
> ---
>  utils/tuning/rkisp1.py | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/utils/tuning/rkisp1.py b/utils/tuning/rkisp1.py
> index f5c42a61d15e..9f40fd8bd63b 100755
> --- a/utils/tuning/rkisp1.py
> +++ b/utils/tuning/rkisp1.py
> @@ -2,6 +2,7 @@
>  # SPDX-License-Identifier: GPL-2.0-or-later
>  #
>  # Copyright (C) 2022, Paul Elder <paul.elder at ideasonboard.com>
> +# Copyright (C) 2024, Ideas On Board
>  #
>  # Tuning script for rkisp1
>  
> @@ -14,13 +15,14 @@ from libtuning.parsers import YamlParser
>  from libtuning.generators import YamlOutput
>  from libtuning.modules.lsc import LSCRkISP1
>  from libtuning.modules.agc import AGCRkISP1
> +from libtuning.modules.awb import AWBRkISP1
>  from libtuning.modules.ccm import CCMRkISP1
>  from libtuning.modules.static import StaticModule
>  
>  coloredlogs.install(level=logging.INFO, fmt='%(name)s %(levelname)s %(message)s')
>  
>  agc = AGCRkISP1(debug=[lt.Debug.Plot])
> -awb = StaticModule('Awb')
> +awb = AWBRkISP1(debug=[lt.Debug.Plot])

Ok so that's 'why' there's an RkISP1 part for now I guess.
I'm still curious if it's actually anything specific ... but fine for
now ... and as this patch stands on top of the predecessors:


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

>  blc = StaticModule('BlackLevelCorrection')
>  ccm = CCMRkISP1(debug=[lt.Debug.Plot])
>  color_processing = StaticModule('ColorProcessing')
> -- 
> 2.43.0
>


More information about the libcamera-devel mailing list