disp: msm: sde: update dither, unsharp along with VLUT

Dither, unsharp should not be enable when VLUT
is disabled. This change ties dither, unsharp to VLUT
enable/disable and not during init property.

Change-Id: Idfad899a13252b22104c9746c86f4e158d9b0980
Signed-off-by: Gopikrishnaiah Anandan <agopik@codeaurora.org>
Signed-off-by: Anjaneya Prasad Musunuri <quic_aprasad@quicinc.com>
This commit is contained in:
Anjaneya Prasad Musunuri
2023-11-20 18:06:35 -08:00
committed by Vinoth Jayaram
parent 04536e868f
commit 54da6b5312

View File

@@ -32,7 +32,7 @@
* When disabling INIT property, we don't want to reset those bits since
* they are needed for both LTM histogram and VLUT.
*/
#define REG_DMA_LTM_INIT_ENABLE_OP_MASK 0xFFFF8CAB
#define REG_DMA_LTM_INIT_ENABLE_OP_MASK 0x1100153
#define REG_DMA_LTM_INIT_DISABLE_OP_MASK 0xFFFF8CAF
#define REG_DMA_LTM_ROI_OP_MASK 0xFEFFFFFF
/**
@@ -3794,22 +3794,16 @@ void reg_dmav1_setup_ltm_initv1(struct sde_hw_dspp *ctx, void *cfg)
}
if (init_param->init_param_01) {
if (ltm_vlut_ops_mask[dspp_idx[i]] & ltm_vlut)
opmode |= BIT(6);
ltm_vlut_ops_mask[dspp_idx[i]] |= ltm_dither;
opmode |= ((init_param->init_param_02 & 0x7) << 12);
} else {
opmode &= ~BIT(6);
ltm_vlut_ops_mask[dspp_idx[i]] &= ~ltm_dither;
}
if (init_param->init_param_03) {
if (ltm_vlut_ops_mask[dspp_idx[i]] & ltm_vlut)
opmode |= BIT(4);
ltm_vlut_ops_mask[dspp_idx[i]] |= ltm_unsharp;
opmode |= ((init_param->init_param_04 & 0x3) << 8);
} else {
opmode &= ~BIT(4);
ltm_vlut_ops_mask[dspp_idx[i]] &= ~ltm_unsharp;
}
@@ -3988,8 +3982,6 @@ void reg_dmav1_setup_ltm_roiv1(struct sde_hw_dspp *ctx, void *cfg)
return;
}
if (ltm_vlut_ops_mask[dspp_idx[i]] & ltm_vlut)
opmode |= BIT(24);
ltm_vlut_ops_mask[dspp_idx[i]] |= ltm_roi;
REG_DMA_SETUP_OPS(dma_write_cfg, 0x04, &opmode, sizeof(opmode),