On 5/7/2026 6:33 PM, Dmitry Baryshkov wrote:
> Use freshly defined helper instead of using the raw value from the
> database.
> 
> Reviewed-by: Konrad Dybcio <[email protected]>
> Signed-off-by: Dmitry Baryshkov <[email protected]>

Reviewed-by: Akhil P Oommen <[email protected]>

-Akhil

> ---
>  drivers/gpu/drm/msm/adreno/a6xx_gpu.c   | 2 +-
>  drivers/gpu/drm/msm/adreno/a8xx_gpu.c   | 4 ++--
>  drivers/gpu/drm/msm/adreno/adreno_gpu.c | 2 +-
>  3 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c 
> b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> index 76c681614416..5c17565b0499 100644
> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.c
> @@ -745,7 +745,7 @@ static void a6xx_set_ubwc_config(struct msm_gpu *gpu)
>       BUG_ON(cfg->highest_bank_bit < 13);
>       u32 hbb = cfg->highest_bank_bit - 13;
>       bool rgb565_predicator = cfg->ubwc_enc_version >= UBWC_4_0;
> -     u32 level2_swizzling_dis = !(cfg->ubwc_swizzle & 
> UBWC_SWIZZLE_ENABLE_LVL2);
> +     u32 level2_swizzling_dis = !(qcom_ubwc_swizzle(cfg) & 
> UBWC_SWIZZLE_ENABLE_LVL2);
>       bool ubwc_mode = qcom_ubwc_get_ubwc_mode(cfg);
>       bool amsbc = cfg->ubwc_enc_version >= UBWC_3_0;
>       bool min_acc_len_64b;
> diff --git a/drivers/gpu/drm/msm/adreno/a8xx_gpu.c 
> b/drivers/gpu/drm/msm/adreno/a8xx_gpu.c
> index 1923f904d37d..53def136e0fc 100644
> --- a/drivers/gpu/drm/msm/adreno/a8xx_gpu.c
> +++ b/drivers/gpu/drm/msm/adreno/a8xx_gpu.c
> @@ -275,8 +275,8 @@ static void a8xx_set_ubwc_config(struct msm_gpu *gpu)
>  {
>       struct adreno_gpu *adreno_gpu = to_adreno_gpu(gpu);
>       const struct qcom_ubwc_cfg_data *cfg = adreno_gpu->ubwc_config;
> -     u32 level2_swizzling_dis = !(cfg->ubwc_swizzle & 
> UBWC_SWIZZLE_ENABLE_LVL2);
> -     u32 level3_swizzling_dis = !(cfg->ubwc_swizzle & 
> UBWC_SWIZZLE_ENABLE_LVL3);
> +     u32 level2_swizzling_dis = !(qcom_ubwc_swizzle(cfg) & 
> UBWC_SWIZZLE_ENABLE_LVL2);
> +     u32 level3_swizzling_dis = !(qcom_ubwc_swizzle(cfg) & 
> UBWC_SWIZZLE_ENABLE_LVL3);
>       bool rgba8888_lossless = false, fp16compoptdis = false;
>       bool yuvnotcomptofc = false, min_acc_len_64b = false;
>       bool rgb565_predicator = false, amsbc = false;
> diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.c 
> b/drivers/gpu/drm/msm/adreno/adreno_gpu.c
> index bf1572156b0b..d26985c88115 100644
> --- a/drivers/gpu/drm/msm/adreno/adreno_gpu.c
> +++ b/drivers/gpu/drm/msm/adreno/adreno_gpu.c
> @@ -430,7 +430,7 @@ int adreno_get_param(struct msm_gpu *gpu, struct 
> msm_context *ctx,
>               *value = adreno_gpu->has_ray_tracing;
>               return 0;
>       case MSM_PARAM_UBWC_SWIZZLE:
> -             *value = adreno_gpu->ubwc_config->ubwc_swizzle;
> +             *value = qcom_ubwc_swizzle(adreno_gpu->ubwc_config);
>               return 0;
>       case MSM_PARAM_MACROTILE_MODE:
>               *value = qcom_ubwc_macrotile_mode(adreno_gpu->ubwc_config);
> 

Reply via email to