From: Sam Day <[email protected]>
A306 and A306A implement only three VBIF XIN halt request bits.
Consequently, waiting for the full 6-bit A3xx mask times out and
prevents runtime suspend.
This issue was reproduced on a DragonBoard 410c: the request reads back
as 0x7 and the acknowledgment as 0x00070007 - the poll eventually
returns -ETIMEDOUT, and a3xx_pm_suspend fails with -EBUSY.
The downstream kgsl driver corroborates this (see link). Since the
single-character difference between A3XX_VBIF_XIN_HALT_CTRL0_MASK and
A30X_VBIF_XIN_HALT_CTRL0_MASK could be confusing to read, the narrowed
GENMASK is instead defined inline in a3xx_vbif_halt().
Fixes: be0e82b8e0c9 ("drm/msm/a3xx: Drain VBIF before GPU suspend")
Link:
https://android.googlesource.com/kernel/msm/+/c90c7feeca2f5839ad6824f816c0bd207602a2f4/drivers/gpu/msm/adreno_a3xx.c#637
Assisted-by: LLM
Signed-off-by: Sam Day <[email protected]>
---
drivers/gpu/drm/msm/adreno/a3xx_gpu.c | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/msm/adreno/a3xx_gpu.c
b/drivers/gpu/drm/msm/adreno/a3xx_gpu.c
index 0259e60b80115..f672abd5e15e4 100644
--- a/drivers/gpu/drm/msm/adreno/a3xx_gpu.c
+++ b/drivers/gpu/drm/msm/adreno/a3xx_gpu.c
@@ -500,14 +500,18 @@ static u64 a3xx_gpu_busy(struct msm_gpu *gpu, unsigned
long *out_sample_rate)
static int a3xx_vbif_halt(struct msm_gpu *gpu)
{
+ struct adreno_gpu *adreno_gpu = to_adreno_gpu(gpu);
+ u32 mask = A3XX_VBIF_XIN_HALT_CTRL0_MASK;
u32 ack;
int ret;
- gpu_write(gpu, REG_A3XX_VBIF_XIN_HALT_CTRL0,
- A3XX_VBIF_XIN_HALT_CTRL0_MASK);
+ /* A306(A) only have three VBIF XIN ports. */
+ if (adreno_is_a306(adreno_gpu) || adreno_is_a306a(adreno_gpu))
+ mask = GENMASK(2, 0);
+
+ gpu_write(gpu, REG_A3XX_VBIF_XIN_HALT_CTRL0, mask);
ret = spin_until(((ack = gpu_read(gpu, REG_A3XX_VBIF_XIN_HALT_CTRL1)) &
- A3XX_VBIF_XIN_HALT_CTRL0_MASK) ==
- A3XX_VBIF_XIN_HALT_CTRL0_MASK);
+ mask) == mask);
gpu_write(gpu, REG_A3XX_VBIF_XIN_HALT_CTRL0, 0);
if (ret)
---
base-commit: a15fac810c76397ec9f62a6fc26c4d7ab6e238a7
change-id: 20260924-a306-vbif-mask-58f7906f246d
Best regards,
--
Sam Day <[email protected]>
