Re: [PATCH 4/4] media: staging: rkisp1: params: remove extra 'if' conditions

[Dafna Hirschfeld <dafna.hirschfeld@xxxxxxxxxxxxx>]

Hi,

Am 20.10.20 um 19:13 schrieb Helen Koike:
> Hi Dafna,
>
> On 10/19/20 5:59 PM, Dafna Hirschfeld wrote:
> > There is a repeating code pattern:
> >
> > if (a || b) {
> > 	if (a)
> > 		...
> > 	if (b)
> > 		...
> > }
> >
> > In this pattern, the first 'if' is redundant.
> > The code can be replaced with:
> >
> > if (a)
> > 	...
> > if (b)
> > 	...
> >
> > In addition, the patch corrects alignment issues
> > reported by checkpatch.
> > This solves the TODO item
> > "Fix checkpatch errors."
> >
> > Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@xxxxxxxxxxxxx>
> > ---
> >   drivers/staging/media/rkisp1/TODO            |   1 -
> >   drivers/staging/media/rkisp1/rkisp1-params.c | 420 ++++++++-----------
> >   2 files changed, 175 insertions(+), 246 deletions(-)
> >
> > diff --git a/drivers/staging/media/rkisp1/TODO b/drivers/staging/media/rkisp1/TODO
> > index e7c8398fc2ce..647ab482d032 100644
> > --- a/drivers/staging/media/rkisp1/TODO
> > +++ b/drivers/staging/media/rkisp1/TODO
> > @@ -1,5 +1,4 @@
> >   * Fix pad format size for statistics and parameters entities.
> > -* Fix checkpatch errors.
> >   * Add uapi docs. Remember to add documentation of how quantization is handled.
> >   * streaming paths (mainpath and selfpath) check if the other path is streaming
> >   in several places of the code, review this, specially that it doesn't seem it
> > diff --git a/drivers/staging/media/rkisp1/rkisp1-params.c b/drivers/staging/media/rkisp1/rkisp1-params.c
> > index a2df6b50c109..c8585b6ee918 100644
> > --- a/drivers/staging/media/rkisp1/rkisp1-params.c
> > +++ b/drivers/staging/media/rkisp1/rkisp1-params.c
> > @@ -186,7 +186,7 @@ static void rkisp1_bls_config(struct rkisp1_params *params,
> >   /* ISP LS correction interface function */
> >   static void
> >   rkisp1_lsc_correct_matrix_config(struct rkisp1_params *params,
> > -				const struct rkisp1_cif_isp_lsc_config *pconfig)
> > +				 const struct rkisp1_cif_isp_lsc_config *pconfig)
>
> I think this should be part of the alignment patch.
>
> >   {
> >   	unsigned int isp_lsc_status, sram_addr, isp_lsc_table_sel, i, j, data;
> >   
> > @@ -434,7 +434,7 @@ static void rkisp1_ctk_enable(struct rkisp1_params *params, bool en)
> >   
> >   /* ISP White Balance Mode */
> >   static void rkisp1_awb_meas_config(struct rkisp1_params *params,
> > -			const struct rkisp1_cif_isp_awb_meas_config *arg)
> > +				   const struct rkisp1_cif_isp_awb_meas_config *arg)
>
> This too.
>
> >   {
> >   	u32 reg_val = 0;
> >   	/* based on the mode,configure the awb module */
> > @@ -888,226 +888,171 @@ rkisp1_isp_isr_other_config(struct rkisp1_params *params,
> >   	module_cfg_update = new_params->module_cfg_update;
> >   	module_ens = new_params->module_ens;
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_DPCC) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_DPCC)) {
> > -		/*update dpc config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_DPCC)
> > -			rkisp1_dpcc_config(params,
> > -					   &new_params->others.dpcc_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_DPCC) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_DPCC)
> > -				rkisp1_param_set_bits(params,
> > -						      RKISP1_CIF_ISP_DPCC_MODE,
> > -						      RKISP1_CIF_ISP_DPCC_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > +	/*update dpc config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_DPCC)
> > +		rkisp1_dpcc_config(params,
> > +				   &new_params->others.dpcc_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_DPCC) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_DPCC)
> > +			rkisp1_param_set_bits(params,
> > +					      RKISP1_CIF_ISP_DPCC_MODE,
> > +					      RKISP1_CIF_ISP_DPCC_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params,
> >   						RKISP1_CIF_ISP_DPCC_MODE,
> >   						RKISP1_CIF_ISP_DPCC_ENA);
> > -		}
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_BLS) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_BLS)) {
> > -		/* update bls config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_BLS)
> > -			rkisp1_bls_config(params,
> > -					  &new_params->others.bls_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_BLS) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_BLS)
> > -				rkisp1_param_set_bits(params,
> > -						      RKISP1_CIF_ISP_BLS_CTRL,
> > -						      RKISP1_CIF_ISP_BLS_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -							RKISP1_CIF_ISP_BLS_CTRL,
> > -							RKISP1_CIF_ISP_BLS_ENA);
> > -		}
> > +	/* update bls config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_BLS)
> > +		rkisp1_bls_config(params,
> > +				  &new_params->others.bls_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_BLS) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_BLS)
> > +			rkisp1_param_set_bits(params,
> > +					      RKISP1_CIF_ISP_BLS_CTRL,
> > +					      RKISP1_CIF_ISP_BLS_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params,
> > +						RKISP1_CIF_ISP_BLS_CTRL,
> > +						RKISP1_CIF_ISP_BLS_ENA);
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_SDG) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_SDG)) {
> > -		/* update sdg config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_SDG)
> > -			rkisp1_sdg_config(params,
> > -					  &new_params->others.sdg_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_SDG) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_SDG)
> > -				rkisp1_param_set_bits(params,
> > -					RKISP1_CIF_ISP_CTRL,
> > -					RKISP1_CIF_ISP_CTRL_ISP_GAMMA_IN_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -					RKISP1_CIF_ISP_CTRL,
> > -					RKISP1_CIF_ISP_CTRL_ISP_GAMMA_IN_ENA);
> > -		}
> > +	/* update sdg config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_SDG)
> > +		rkisp1_sdg_config(params,
> > +				  &new_params->others.sdg_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_SDG) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_SDG)
> > +			rkisp1_param_set_bits(params, RKISP1_CIF_ISP_CTRL,
> > +					      RKISP1_CIF_ISP_CTRL_ISP_GAMMA_IN_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_CTRL,
> > +						RKISP1_CIF_ISP_CTRL_ISP_GAMMA_IN_ENA);
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_LSC) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_LSC)) {
> > -		/* update lsc config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_LSC)
> > -			rkisp1_lsc_config(params,
> > -					  &new_params->others.lsc_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_LSC) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_LSC)
> > -				rkisp1_param_set_bits(params,
> > -						RKISP1_CIF_ISP_LSC_CTRL,
> > -						RKISP1_CIF_ISP_LSC_CTRL_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -						RKISP1_CIF_ISP_LSC_CTRL,
> > +	/* update lsc config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_LSC)
> > +		rkisp1_lsc_config(params,
> > +				  &new_params->others.lsc_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_LSC) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_LSC)
> > +			rkisp1_param_set_bits(params, RKISP1_CIF_ISP_LSC_CTRL,
> > +					      RKISP1_CIF_ISP_LSC_CTRL_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_LSC_CTRL,
> >   						RKISP1_CIF_ISP_LSC_CTRL_ENA);
> > -		}
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_AWB_GAIN) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_AWB_GAIN)) {
> > -		/* update awb gains */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_AWB_GAIN)
> > -			rkisp1_awb_gain_config(params,
> > -					&new_params->others.awb_gain_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_AWB_GAIN) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_AWB_GAIN)
> > -				rkisp1_param_set_bits(params,
> > -					RKISP1_CIF_ISP_CTRL,
> > -					RKISP1_CIF_ISP_CTRL_ISP_AWB_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -					RKISP1_CIF_ISP_CTRL,
> > -					RKISP1_CIF_ISP_CTRL_ISP_AWB_ENA);
> > -		}
> > +	/* update awb gains */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_AWB_GAIN)
> > +		rkisp1_awb_gain_config(params, &new_params->others.awb_gain_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_AWB_GAIN) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_AWB_GAIN)
> > +			rkisp1_param_set_bits(params, RKISP1_CIF_ISP_CTRL,
> > +					      RKISP1_CIF_ISP_CTRL_ISP_AWB_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_CTRL,
> > +						RKISP1_CIF_ISP_CTRL_ISP_AWB_ENA);
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_BDM) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_BDM)) {
> > -		/* update bdm config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_BDM)
> > -			rkisp1_bdm_config(params,
> > -					  &new_params->others.bdm_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_BDM) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_BDM)
> > -				rkisp1_param_set_bits(params,
> > -						RKISP1_CIF_ISP_DEMOSAIC,
> > -						RKISP1_CIF_ISP_DEMOSAIC_BYPASS);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -						RKISP1_CIF_ISP_DEMOSAIC,
> > +	/* update bdm config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_BDM)
> > +		rkisp1_bdm_config(params,
> > +				  &new_params->others.bdm_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_BDM) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_BDM)
> > +			rkisp1_param_set_bits(params, RKISP1_CIF_ISP_DEMOSAIC,
> > +					      RKISP1_CIF_ISP_DEMOSAIC_BYPASS);
> > +		else
> > +			rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_DEMOSAIC,
> >   						RKISP1_CIF_ISP_DEMOSAIC_BYPASS);
> > -		}
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_FLT) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_FLT)) {
> > -		/* update filter config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_FLT)
> > -			rkisp1_flt_config(params,
> > -					  &new_params->others.flt_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_FLT) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_FLT)
> > -				rkisp1_param_set_bits(params,
> > -						      RKISP1_CIF_ISP_FILT_MODE,
> > -						      RKISP1_CIF_ISP_FLT_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -						RKISP1_CIF_ISP_FILT_MODE,
> > +	/* update filter config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_FLT)
> > +		rkisp1_flt_config(params,
> > +				  &new_params->others.flt_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_FLT) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_FLT)
> > +			rkisp1_param_set_bits(params,
> > +					      RKISP1_CIF_ISP_FILT_MODE,
> > +					      RKISP1_CIF_ISP_FLT_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_FILT_MODE,
> >   						RKISP1_CIF_ISP_FLT_ENA);
> > -		}
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_CTK) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_CTK)) {
> > -		/* update ctk config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_CTK)
> > -			rkisp1_ctk_config(params,
> > -					  &new_params->others.ctk_config);
> > +	/* update ctk config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_CTK)
> > +		rkisp1_ctk_config(params,
> > +				  &new_params->others.ctk_config);
> >   
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_CTK)
> > -			rkisp1_ctk_enable(params,
> > -				!!(module_ens & RKISP1_CIF_ISP_MODULE_CTK));
> > -	}
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_CTK)
> > +		rkisp1_ctk_enable(params, !!(module_ens & RKISP1_CIF_ISP_MODULE_CTK));
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_GOC) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_GOC)) {
> > -		/* update goc config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_GOC)
> > -			rkisp1_goc_config(params,
> > -					  &new_params->others.goc_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_GOC) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_GOC)
> > -				rkisp1_param_set_bits(params,
> > -					RKISP1_CIF_ISP_CTRL,
> > -					RKISP1_CIF_ISP_CTRL_ISP_GAMMA_OUT_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -					RKISP1_CIF_ISP_CTRL,
> > -					RKISP1_CIF_ISP_CTRL_ISP_GAMMA_OUT_ENA);
> > -		}
> > +	/* update goc config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_GOC)
> > +		rkisp1_goc_config(params,
> > +				  &new_params->others.goc_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_GOC) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_GOC)
> > +			rkisp1_param_set_bits(params, RKISP1_CIF_ISP_CTRL,
> > +					      RKISP1_CIF_ISP_CTRL_ISP_GAMMA_OUT_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_CTRL,
> > +						RKISP1_CIF_ISP_CTRL_ISP_GAMMA_OUT_ENA);
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_CPROC) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_CPROC)) {
> > -		/* update cproc config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_CPROC) {
> > -			rkisp1_cproc_config(params,
> > -					    &new_params->others.cproc_config);
> > -		}
> > +	/* update cproc config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_CPROC) {
> > +		rkisp1_cproc_config(params,
> > +				    &new_params->others.cproc_config);
> > +	}
>
> No need for the curly braces.
>
> >   
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_CPROC) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_CPROC)
> > -				rkisp1_param_set_bits(params,
> > -						RKISP1_CIF_C_PROC_CTRL,
> > -						RKISP1_CIF_C_PROC_CTR_ENABLE);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -						RKISP1_CIF_C_PROC_CTRL,
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_CPROC) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_CPROC)
> > +			rkisp1_param_set_bits(params, RKISP1_CIF_C_PROC_CTRL,
> > +					      RKISP1_CIF_C_PROC_CTR_ENABLE);
> > +		else
> > +			rkisp1_param_clear_bits(params, RKISP1_CIF_C_PROC_CTRL,
> >   						RKISP1_CIF_C_PROC_CTR_ENABLE);
> > -		}
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_IE) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_IE)) {
> > -		/* update ie config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_IE)
> > -			rkisp1_ie_config(params,
> > -					 &new_params->others.ie_config);
> > +	/* update ie config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_IE)
> > +		rkisp1_ie_config(params,
> > +				 &new_params->others.ie_config);
> >   
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_IE)
> > -			rkisp1_ie_enable(params,
> > -				!!(module_ens & RKISP1_CIF_ISP_MODULE_IE));
> > -	}
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_IE)
> > +		rkisp1_ie_enable(params, !!(module_ens & RKISP1_CIF_ISP_MODULE_IE));
> > +
> > +	/* update dpf  config */
>
> Could you remove this extra space between "dpf" and "config" ?

I think it is better to add your suggestions on a separate
patch so not to overload the patch.

> I'm glad to see a level of identation gone.

:-)

Thanks,
Dafna

> With these changes
>
> Acked-by: Helen Koike <helen.koike@xxxxxxxxxxxxx>
>
> Thanks
> Helen
>
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_DPF)
> > +		rkisp1_dpf_config(params,
> > +				  &new_params->others.dpf_config);
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_DPF) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_DPF)) {
> > -		/* update dpf  config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_DPF)
> > -			rkisp1_dpf_config(params,
> > -					  &new_params->others.dpf_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_DPF) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_DPF)
> > -				rkisp1_param_set_bits(params,
> > -						   RKISP1_CIF_ISP_DPF_MODE,
> > -						   RKISP1_CIF_ISP_DPF_MODE_EN);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -						RKISP1_CIF_ISP_DPF_MODE,
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_DPF) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_DPF)
> > +			rkisp1_param_set_bits(params, RKISP1_CIF_ISP_DPF_MODE,
> > +					      RKISP1_CIF_ISP_DPF_MODE_EN);
> > +		else
> > +			rkisp1_param_clear_bits(params, RKISP1_CIF_ISP_DPF_MODE,
> >   						RKISP1_CIF_ISP_DPF_MODE_EN);
> > -		}
> >   	}
> >   
> >   	if ((module_en_update & RKISP1_CIF_ISP_MODULE_DPF_STRENGTH) ||
> >   	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_DPF_STRENGTH)) {
> >   		/* update dpf strength config */
> > -		rkisp1_dpf_strength_config(params,
> > -				&new_params->others.dpf_strength_config);
> > +		rkisp1_dpf_strength_config(params, &new_params->others.dpf_strength_config);
> >   	}
> >   }
> >   
> > @@ -1120,68 +1065,53 @@ static void rkisp1_isp_isr_meas_config(struct rkisp1_params *params,
> >   	module_cfg_update = new_params->module_cfg_update;
> >   	module_ens = new_params->module_ens;
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_AWB) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_AWB)) {
> > -		/* update awb config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_AWB)
> > -			rkisp1_awb_meas_config(params,
> > -					&new_params->meas.awb_meas_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_AWB)
> > -			rkisp1_awb_meas_enable(params,
> > -				&new_params->meas.awb_meas_config,
> > -				!!(module_ens & RKISP1_CIF_ISP_MODULE_AWB));
> > -	}
> > +	/* update awb config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_AWB)
> > +		rkisp1_awb_meas_config(params, &new_params->meas.awb_meas_config);
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_AFC) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_AFC)) {
> > -		/* update afc config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_AFC)
> > -			rkisp1_afm_config(params,
> > -					  &new_params->meas.afc_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_AFC) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_AFC)
> > -				rkisp1_param_set_bits(params,
> > -						      RKISP1_CIF_ISP_AFM_CTRL,
> > -						      RKISP1_CIF_ISP_AFM_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -							RKISP1_CIF_ISP_AFM_CTRL,
> > -							RKISP1_CIF_ISP_AFM_ENA);
> > -		}
> > -	}
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_AWB)
> > +		rkisp1_awb_meas_enable(params, &new_params->meas.awb_meas_config,
> > +				       !!(module_ens & RKISP1_CIF_ISP_MODULE_AWB));
> > +
> > +	/* update afc config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_AFC)
> > +		rkisp1_afm_config(params,
> > +				  &new_params->meas.afc_config);
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_HST) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_HST)) {
> > -		/* update hst config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_HST)
> > -			rkisp1_hst_config(params,
> > -					  &new_params->meas.hst_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_HST)
> > -			rkisp1_hst_enable(params,
> > -				&new_params->meas.hst_config,
> > -				!!(module_ens & RKISP1_CIF_ISP_MODULE_HST));
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_AFC) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_AFC)
> > +			rkisp1_param_set_bits(params,
> > +					      RKISP1_CIF_ISP_AFM_CTRL,
> > +					      RKISP1_CIF_ISP_AFM_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params,
> > +						RKISP1_CIF_ISP_AFM_CTRL,
> > +						RKISP1_CIF_ISP_AFM_ENA);
> >   	}
> >   
> > -	if ((module_en_update & RKISP1_CIF_ISP_MODULE_AEC) ||
> > -	    (module_cfg_update & RKISP1_CIF_ISP_MODULE_AEC)) {
> > -		/* update aec config */
> > -		if (module_cfg_update & RKISP1_CIF_ISP_MODULE_AEC)
> > -			rkisp1_aec_config(params,
> > -					  &new_params->meas.aec_config);
> > -
> > -		if (module_en_update & RKISP1_CIF_ISP_MODULE_AEC) {
> > -			if (module_ens & RKISP1_CIF_ISP_MODULE_AEC)
> > -				rkisp1_param_set_bits(params,
> > -						      RKISP1_CIF_ISP_EXP_CTRL,
> > -						      RKISP1_CIF_ISP_EXP_ENA);
> > -			else
> > -				rkisp1_param_clear_bits(params,
> > -							RKISP1_CIF_ISP_EXP_CTRL,
> > -							RKISP1_CIF_ISP_EXP_ENA);
> > -		}
> > +	/* update hst config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_HST)
> > +		rkisp1_hst_config(params,
> > +				  &new_params->meas.hst_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_HST)
> > +		rkisp1_hst_enable(params, &new_params->meas.hst_config,
> > +				  !!(module_ens & RKISP1_CIF_ISP_MODULE_HST));
> > +
> > +	/* update aec config */
> > +	if (module_cfg_update & RKISP1_CIF_ISP_MODULE_AEC)
> > +		rkisp1_aec_config(params,
> > +				  &new_params->meas.aec_config);
> > +
> > +	if (module_en_update & RKISP1_CIF_ISP_MODULE_AEC) {
> > +		if (module_ens & RKISP1_CIF_ISP_MODULE_AEC)
> > +			rkisp1_param_set_bits(params,
> > +					      RKISP1_CIF_ISP_EXP_CTRL,
> > +					      RKISP1_CIF_ISP_EXP_ENA);
> > +		else
> > +			rkisp1_param_clear_bits(params,
> > +						RKISP1_CIF_ISP_EXP_CTRL,
> > +						RKISP1_CIF_ISP_EXP_ENA);
> >   	}
> >   }
> >