Re: [PATCH v2 8/8] drm/msm/dpu: drop now-unused condition for has_legacy_ctls

[Marijn Suijten]

On 28 February 2025 05:14:12 CET, Dmitry Baryshkov <dmitry.baryshkov at linaro.org> wrote:
>Now as we have dropped the DPU_CTL_SPLIT_DISPLAY from DPU >= 5.0
>configuration, drop the rm->has_legacy_ctl condition which short-cutted
>the check for those platforms.
>
>Suggested-by: Marijn Suijten <marijn.suijten at somainline.org>
>Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov at linaro.org>
>
>---
>
>Note, it is imposible to reoder commits in any other sensible way. The

Reorder :)

>DPU_CTL_SPLIT_DISPLAY can not be dropped before the patch that enables
>single-CTL support.

Maybe I expected this note on patch 4/6 introducing it, but it disappears after merging anyway so that's a mega-nit. Thanks for sending this!

Reviewed-by: Marijn Suijten <marijn.suijten at somainline.org>

>---
> drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
>diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c
>index 4dbc9bc7eb4f151f83055220665ee5fd238ae7ba..2557effe639b5360bc948a49b0cccdb59ee35dab 100644
>--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c
>+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c
>@@ -410,8 +410,7 @@ static int _dpu_rm_reserve_ctls(
> 
> 		DPU_DEBUG("ctl %d caps 0x%lX\n", j + CTL_0, features);
> 
>-		if (rm->has_legacy_ctls &&
>-		    needs_split_display != has_split_display)
>+		if (needs_split_display != has_split_display)
> 			continue;
> 
> 		ctl_idx[i] = j;
>