public inbox for drm-ai-reviews@public-inbox.freedesktop.org
 help / color / mirror / Atom feed
From: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
To: Rob Clark <robin.clark@oss.qualcomm.com>,
	Dmitry Baryshkov <lumag@kernel.org>,
	Abhinav Kumar <abhinav.kumar@linux.dev>,
	Jessica Zhang <jesszhan0024@gmail.com>,
	Sean Paul <sean@poorly.run>,
	Marijn Suijten <marijn.suijten@somainline.org>,
	David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konradybcio@kernel.org>,
	Akhil P Oommen <akhilpo@oss.qualcomm.com>
Cc: linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
	freedreno@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Subject: [PATCH v2 19/25] soc: qcom: ubwc: set min_acc length to 64 for all UBWC 1.0 targets
Date: Wed, 11 Mar 2026 05:23:13 +0200	[thread overview]
Message-ID: <20260311-ubwc-rework-v2-19-69f718f2a1c9@oss.qualcomm.com> (raw)
In-Reply-To: <20260311-ubwc-rework-v2-0-69f718f2a1c9@oss.qualcomm.com>

According to the documentation, the MAL should be set for all UBWC 1.0
targets, no matter what is the version of the UBWC decoders are present
on the device. The helper comes from DPU / GPU world, where there was no
separate bit to control MAL before UBWC 2.0. As the helper is now being
used by other drivers too, correct the helper to return the correct MAL
value (Iris doesn't support UBWC 1.0 devices for now, so there is no
changes of the behaviour).

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
---
 include/linux/soc/qcom/ubwc.h | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/include/linux/soc/qcom/ubwc.h b/include/linux/soc/qcom/ubwc.h
index 8355ffe40f88..ddd7b15d9ff1 100644
--- a/include/linux/soc/qcom/ubwc.h
+++ b/include/linux/soc/qcom/ubwc.h
@@ -80,9 +80,7 @@ static inline bool qcom_ubwc_get_ubwc_mode(const struct qcom_ubwc_cfg_data *cfg)
  */
 static inline bool qcom_ubwc_min_acc_length_64b(const struct qcom_ubwc_cfg_data *cfg)
 {
-	return cfg->ubwc_enc_version == UBWC_1_0 &&
-		(cfg->ubwc_dec_version == UBWC_2_0 ||
-		 cfg->ubwc_dec_version == UBWC_3_0);
+	return cfg->ubwc_enc_version == UBWC_1_0;
 }
 
 static inline bool qcom_ubwc_macrotile_mode(const struct qcom_ubwc_cfg_data *cfg)

-- 
2.47.3


  parent reply	other threads:[~2026-03-11  3:24 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-11  3:22 [PATCH v2 00/25] soc/qcom/ubwc: rework UBWC configuration database Dmitry Baryshkov
2026-03-11  3:22 ` [PATCH v2 01/25] drm/msm/mdss: correct UBWC programming sequences Dmitry Baryshkov
2026-03-11  9:01   ` Konrad Dybcio
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:22 ` [PATCH v2 02/25] soc: qcom: ubwc: define UBWC 3.1 Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:22 ` [PATCH v2 03/25] soc: qcom: ubwc: define helper for MDSS and Adreno drivers Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:22 ` [PATCH v2 04/25] drm/msm/adreno: Trust the SSoT UBWC config Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:22 ` [PATCH v2 05/25] drm/msm/adreno: use qcom_ubwc_version_tag() helper Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 06/25] drm/msm/mdss: " Dmitry Baryshkov
2026-03-11  8:52   ` Konrad Dybcio
2026-03-11  3:23 ` [PATCH v2 07/25] drm/msm/adreno: use new helper to set min_acc length Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 08/25] drm/msm/mdss: " Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 09/25] drm/msm/adreno: use new helper to set macrotile_mode Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 10/25] drm/msm/mdss: " Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 11/25] drm/msm/mdss: use new helper to set UBWC bank spreading Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 12/25] drm/msm/adreno: use new helper to set ubwc_swizzle Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 13/25] drm/msm/dpu: " Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 14/25] drm/msm/mdss: " Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 15/25] drm/msm/dpu: drop ubwc_dec_version Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:23 ` [PATCH v2 16/25] drm/msm/adreno: adapt for UBWC 3.1 support Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 17/25] drm/msm/mdss: " Dmitry Baryshkov
2026-03-11  3:23 ` [PATCH v2 18/25] drm/msm/dpu: " Dmitry Baryshkov
2026-03-11  3:23 ` Dmitry Baryshkov [this message]
2026-03-11 21:21   ` Claude review: soc: qcom: ubwc: set min_acc length to 64 for all UBWC 1.0 targets Claude Code Review Bot
2026-03-11  3:23 ` [PATCH v2 20/25] soc: qcom: ubwc: drop ubwc_dec_version Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:23 ` [PATCH v2 21/25] soc: qcom: ubwc: drop ubwc_bank_spread Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:23 ` [PATCH v2 22/25] soc: qcom: ubwc: drop macrotile_mode from the database Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:23 ` [PATCH v2 23/25] soc: qcom: ubwc: use fixed values for UBWC swizzle for UBWC < 4.0 Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:23 ` [PATCH v2 24/25] soc: qcom: ubwc: sort out the rest of the UBWC swizzle settings Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11  3:23 ` [PATCH v2 25/25] soc: qcom: ubwc: deduplicate UBWC configuration data Dmitry Baryshkov
2026-03-11 21:21   ` Claude review: " Claude Code Review Bot
2026-03-11 21:21 ` Claude review: soc/qcom/ubwc: rework UBWC configuration database Claude Code Review Bot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260311-ubwc-rework-v2-19-69f718f2a1c9@oss.qualcomm.com \
    --to=dmitry.baryshkov@oss.qualcomm.com \
    --cc=abhinav.kumar@linux.dev \
    --cc=airlied@gmail.com \
    --cc=akhilpo@oss.qualcomm.com \
    --cc=andersson@kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=freedreno@lists.freedesktop.org \
    --cc=jesszhan0024@gmail.com \
    --cc=konrad.dybcio@oss.qualcomm.com \
    --cc=konradybcio@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lumag@kernel.org \
    --cc=marijn.suijten@somainline.org \
    --cc=robin.clark@oss.qualcomm.com \
    --cc=sean@poorly.run \
    --cc=simona@ffwll.ch \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox