[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1674138393-475-10-git-send-email-quic_vpolimer@quicinc.com>
Date: Thu, 19 Jan 2023 19:56:27 +0530
From: Vinod Polimera <quic_vpolimer@...cinc.com>
To: <dri-devel@...ts.freedesktop.org>, <linux-arm-msm@...r.kernel.org>,
<freedreno@...ts.freedesktop.org>, <devicetree@...r.kernel.org>
CC: Vinod Polimera <quic_vpolimer@...cinc.com>,
<linux-kernel@...r.kernel.org>, <robdclark@...il.com>,
<dianders@...omium.org>, <swboyd@...omium.org>,
<quic_kalyant@...cinc.com>, <dmitry.baryshkov@...aro.org>,
<quic_khsieh@...cinc.com>, <quic_vproddut@...cinc.com>,
<quic_bjorande@...cinc.com>, <quic_abhinavk@...cinc.com>,
<quic_sbillaka@...cinc.com>
Subject: [PATCH Resend v11 09/15] drm/msm/disp/dpu: check for crtc enable rather than crtc active to release shared resources
According to KMS documentation, The driver must not release any shared
resources if active is set to false but enable still true.
Fixes: ccc862b957c6 ("drm/msm/dpu: Fix reservation failures in modeset")
Signed-off-by: Vinod Polimera <quic_vpolimer@...cinc.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
index 1757856..29d6cda 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
@@ -651,7 +651,7 @@ static int dpu_encoder_virt_atomic_check(
if (drm_atomic_crtc_needs_modeset(crtc_state)) {
dpu_rm_release(global_state, drm_enc);
- if (!crtc_state->active_changed || crtc_state->active)
+ if (!crtc_state->active_changed || crtc_state->enable)
ret = dpu_rm_reserve(&dpu_kms->rm, global_state,
drm_enc, crtc_state, topology);
}
--
2.7.4
Powered by blists - more mailing lists