[PATCH 1/3] drm/amd/display: Use dcc_ind_blk value to set register directly

[Joshua Ashton]

We don't need to do this workaround if we start setting this value when we fill 
the plane attributes.

Signed-off-by: Joshua Ashton <jos...@froggi.es>
Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 ++++++-
 drivers/gpu/drm/amd/display/dc/core/dc.c          | 2 +-
 drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hubp.c | 6 ------
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index 8837259215d9..9c6f2863ba96 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -4914,10 +4914,15 @@ fill_gfx9_plane_attributes_from_modifiers(struct 
amdgpu_device *adev,
 
        if (modifier_has_dcc(modifier) && !force_disable_dcc) {
                uint64_t dcc_address = afb->address + afb->base.offsets[1];
+               bool independent_64b_blks = 
AMD_FMT_MOD_GET(DCC_INDEPENDENT_64B, modifier);
 
                dcc->enable = 1;
                dcc->meta_pitch = afb->base.pitches[1];
-               dcc->independent_64b_blks = 
AMD_FMT_MOD_GET(DCC_INDEPENDENT_64B, modifier);
+               dcc->independent_64b_blks = independent_64b_blks;
+               if (independent_64b_blks)
+                       dcc->dcc_ind_blk = hubp_ind_block_64b;
+               else
+                       dcc->dcc_ind_blk = hubp_ind_block_unconstrained;
 
                address->grph.meta_addr.low_part = lower_32_bits(dcc_address);
                address->grph.meta_addr.high_part = upper_32_bits(dcc_address);
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c 
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index f58d3956f3e2..da360691e655 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -2008,7 +2008,7 @@ static enum surface_update_type 
get_plane_info_update_type(const struct dc_surfa
        }
 
        if (u->plane_info->dcc.enable != u->surface->dcc.enable
-                       || u->plane_info->dcc.independent_64b_blks != 
u->surface->dcc.independent_64b_blks
+                       || u->plane_info->dcc.dcc_ind_blk != 
u->surface->dcc.dcc_ind_blk
                        || u->plane_info->dcc.meta_pitch != 
u->surface->dcc.meta_pitch) {
                /* During DCC on/off, stutter period is calculated before
                 * DCC has fully transitioned. This results in incorrect
diff --git a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hubp.c 
b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hubp.c
index f24612523248..eac08926b574 100644
--- a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hubp.c
+++ b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hubp.c
@@ -356,12 +356,6 @@ void hubp3_dcc_control_sienna_cichlid(struct hubp *hubp,
 {
        struct dcn20_hubp *hubp2 = TO_DCN20_HUBP(hubp);
 
-       /*Workaround until UMD fix the new dcc_ind_blk interface */
-       if (dcc->independent_64b_blks && dcc->dcc_ind_blk == 0)
-               dcc->dcc_ind_blk = 1;
-       if (dcc->independent_64b_blks_c && dcc->dcc_ind_blk_c == 0)
-               dcc->dcc_ind_blk_c = 1;
-
        REG_UPDATE_6(DCSURF_SURFACE_CONTROL,
                PRIMARY_SURFACE_DCC_EN, dcc->enable,
                PRIMARY_SURFACE_DCC_IND_BLK, dcc->dcc_ind_blk,
-- 
2.33.0