[PATCH 4/4] media: staging: rkisp1: params: remove extra 'if' conditions
Dafna Hirschfeld
dafna.hirschfeld at collabora.com
Tue Oct 20 13:21:30 EDT 2020
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 at collabora.com>
>> ---
>> 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 at collabora.com>
>
> 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);
>> }
>> }
>>
>>
More information about the Linux-rockchip
mailing list