]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/msm/mdss: use new helper to set min_acc length
authorDmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Wed, 20 May 2026 14:51:15 +0000 (17:51 +0300)
committerDmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Fri, 22 May 2026 13:43:12 +0000 (16:43 +0300)
Use freshly defined helper instead of hardcoding the checks in the
driver.

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Patchwork: https://patchwork.freedesktop.org/patch/726500/
Link: https://lore.kernel.org/r/20260520-ubwc-rework-v5-8-72f2749bc807@oss.qualcomm.com
drivers/gpu/drm/msm/msm_mdss.c

index 06c3b68f5e23418200e004b3a1c7a37dbb6f4207..e8231a5720aaf0f4261a95e1dbc431da55169c6e 100644 (file)
@@ -172,8 +172,7 @@ static void msm_mdss_4x_setup_ubwc(struct msm_mdss *msm_mdss)
        u32 value = MDSS_UBWC_STATIC_UBWC_SWIZZLE(data->ubwc_swizzle & 0x1) |
                    MDSS_UBWC_STATIC_HIGHEST_BANK_BIT(data->highest_bank_bit - 13);
 
-       if (data->ubwc_enc_version == UBWC_1_0)
-               value |= MDSS_UBWC_STATIC_UBWC_MIN_ACC_LEN(1);
+       value |= MDSS_UBWC_STATIC_UBWC_MIN_ACC_LEN(qcom_ubwc_min_acc_length_64b(data));
 
        writel_relaxed(value, msm_mdss->mmio + REG_MDSS_UBWC_STATIC);
 }
@@ -190,8 +189,7 @@ static void msm_mdss_5x_setup_ubwc(struct msm_mdss *msm_mdss)
        if (data->ubwc_enc_version == UBWC_3_0)
                value |= MDSS_UBWC_STATIC_UBWC_AMSBC;
 
-       if (data->ubwc_enc_version == UBWC_1_0)
-               value |= MDSS_UBWC_STATIC_UBWC_MIN_ACC_LEN(1);
+       value |= MDSS_UBWC_STATIC_UBWC_MIN_ACC_LEN(qcom_ubwc_min_acc_length_64b(data));
 
        writel_relaxed(value, msm_mdss->mmio + REG_MDSS_UBWC_STATIC);
 }
@@ -209,8 +207,7 @@ static void msm_mdss_6x_setup_ubwc(struct msm_mdss *msm_mdss)
        if (data->macrotile_mode)
                value |= MDSS_UBWC_STATIC_MACROTILE_MODE;
 
-       if (data->ubwc_enc_version == UBWC_1_0)
-               value |= MDSS_UBWC_STATIC_UBWC_MIN_ACC_LEN(1);
+       value |= MDSS_UBWC_STATIC_UBWC_MIN_ACC_LEN(qcom_ubwc_min_acc_length_64b(data));
 
        writel_relaxed(value, msm_mdss->mmio + REG_MDSS_UBWC_STATIC);