[libcamera-devel] [PATCH] ipa: ipu3: fix coverity issues in AWB

Niklas Söderlund niklas.soderlund at ragnatech.se
Thu Apr 22 16:34:47 CEST 2021


Hi Jean-Michel,

Nice work, this fixes gcc7 compilation for me.

On 2021-04-22 14:41:23 +0200, Jean-Michel Hautbois wrote:
> This patch fixes two minor issues detected by converity after merging in
> master.
> 
> Reported-by: Coverity CID=320056
> Reported-by: Coverity CID=320058
> Signed-off-by: Jean-Michel Hautbois <jeanmichel.hautbois at ideasonboard.com>

Tested-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  src/ipa/ipu3/ipu3_awb.cpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/ipa/ipu3/ipu3_awb.cpp b/src/ipa/ipu3/ipu3_awb.cpp
> index 060d1e18..f8788065 100644
> --- a/src/ipa/ipu3/ipu3_awb.cpp
> +++ b/src/ipa/ipu3/ipu3_awb.cpp
> @@ -96,14 +96,14 @@ static const struct ipu3_uapi_bnr_static_config imguCssBnrDefaults = {
>  	.wb_gains_thr = { 255, 255, 255, 255 },
>  	.thr_coeffs = { 1700, 0, 31, 31, 0, 16 },
>  	.thr_ctrl_shd = { 26, 26, 26, 26 },
> -	.opt_center{ -648, 0, -366, 0 },
> +	.opt_center = { -648, 0, -366, 0 },
>  	.lut = {
>  		{ 17, 23, 28, 32, 36, 39, 42, 45,
>  		  48, 51, 53, 55, 58, 60, 62, 64,
>  		  66, 68, 70, 72, 73, 75, 77, 78,
>  		  80, 82, 83, 85, 86, 88, 89, 90 } },
>  	.bp_ctrl = { 20, 0, 1, 40, 0, 6, 0, 6, 0 },
> -	.dn_detect_ctrl{ 9, 3, 4, 0, 8, 0, 1, 1, 1, 1, 0 },
> +	.dn_detect_ctrl = { 9, 3, 4, 0, 8, 0, 1, 1, 1, 1, 0 },
>  	.column_size = 1296,
>  	.opt_center_sqr = { 419904, 133956 },
>  };
> -- 
> 2.27.0
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund


More information about the libcamera-devel mailing list