Re: [PATCH 14/21] drm/msm/dpu: Support quad-pipe in SSPP checking

From: Jun Nie
Date: Tue Sep 03 2024 - 03:53:17 EST


Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> 于2024年8月29日周四 19:38写道:
>
> > @@ -1033,13 +1030,10 @@ static int dpu_plane_atomic_check(struct drm_plane *plane,
> > return -E2BIG;
> > }
> >
> > - /*
> > - * Use multirect for wide plane. We do not support dynamic
> > - * assignment of SSPPs, so we know the configuration.
> > - */
> > pipe->multirect_index = DPU_SSPP_RECT_0;
> > pipe->multirect_mode = DPU_SSPP_MULTIRECT_PARALLEL;
> >
> > + r_pipe->sspp = pipe->sspp;
>
> NAK
>
I do not understand what's your point here. If multi-rectangle mode is
used, the 2 sw_pipe share a
SSPP structure. Otherwise the right half of mixer pair does not work
and no display on that side,
because there is no SSPP to handle in r_pipe setup.

>
> --
> With best wishes
> Dmitry