Re: [PATCH v15 12/13] drm/msm/dpu: support plane splitting in quad-pipe case

[Jun Nie]

Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx> 于2025年9月3日周三 17:39写道：
>
> On Tue, Aug 19, 2025 at 09:31:06AM +0800, Jun Nie wrote:
> > The content of every half of screen is sent out via one interface in
> > dual-DSI case. The content for every interface is blended by a LM
> > pair in quad-pipe case, thus a LM pair should not blend any content
> > that cross the half of screen in this case. Clip plane into pipes per
> > left and right half screen ROI if topology is quad pipe case.
> >
> > The clipped rectangle on every half of screen is futher handled by two
> > pipes if its width exceeds a limit for a single pipe.
> >
> > Signed-off-by: Jun Nie <jun.nie@xxxxxxxxxx>
> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
>
> I've run several CI workloads. This is the first commit which breaks IGT
> tests. See https://gitlab.freedesktop.org/drm/msm/-/pipelines/1503075

Thanks for the work! I did test locally with all my patch set on sm8650 and
did not reproduce the issue. The baseline is not aligned with msm-next-lumag
yet because I need to port my platform patches to it. I will do it tomorrow.
 Do you think this issue is related to difference of sm8650 and sm8350?
Regarding all failure cases are write back related.

- Jun

>
> > ---
> >  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c  |  11 +++
> >  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h  |   2 +
> >  drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 137 +++++++++++++++++++++---------
> >  3 files changed, 110 insertions(+), 40 deletions(-)
>
> --
> With best wishes
> Dmitry