Re: [Freedreno] [PATCH 05/11] drm/msm/dpu: allow using all lm mixer stages

[Dmitry Baryshkov]

On 19/04/2023 17:41, Arnaud Vrac wrote:

The max_mixer_blendstages hw catalog property represents the number of
planes that can be blended by the lm mixer, excluding the base stage, so
adjust the check for the number of currently assigned planes accordingly.

Signed-off-by: Arnaud Vrac 
---
  drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)


Reviewed-by: Dmitry Baryshkov 

--
With best wishes
Dmitry

[Freedreno] [PATCH 05/11] drm/msm/dpu: allow using all lm mixer stages

[Arnaud Vrac]

The max_mixer_blendstages hw catalog property represents the number of
planes that can be blended by the lm mixer, excluding the base stage, so
adjust the check for the number of currently assigned planes accordingly.

Signed-off-by: Arnaud Vrac 
---
 drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c 
b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c
index 148921ed62f85..128ecdc145260 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c
@@ -882,9 +882,9 @@ static int dpu_plane_atomic_check(struct drm_plane *plane,
r_pipe->sspp = NULL;
 
pstate->stage = DPU_STAGE_BASE + pstate->base.normalized_zpos;
-   if (pstate->stage >= pdpu->catalog->caps->max_mixer_blendstages) {
-   DPU_ERROR("> %d plane stages assigned\n",
- pdpu->catalog->caps->max_mixer_blendstages - 
DPU_STAGE_0);
+   if (pstate->stage > DPU_STAGE_BASE + 
pdpu->catalog->caps->max_mixer_blendstages) {
+   DPU_ERROR("> %d plane mixer stages assigned\n",
+ pdpu->catalog->caps->max_mixer_blendstages);
return -EINVAL;
}
 

-- 
2.40.0