On Thu, Dec 19, 2024 at 03:49:27PM +0800, Jun Nie wrote: > Split PIPES_PER_STAGE definition per plane and mixer pair. Because > there are more than 2 pipes in quad pipe case, while 2 pipes at most > per mixer pair. A stage struct serve a mixer pair, so pipes per > plane is split out as PIPES_PER_PLANE. I can barely understand this. The patch itself LGTM. > > Signed-off-by: Jun Nie <jun.nie@xxxxxxxxxx> > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 4 ++-- > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_mdss.h | 1 + > drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 18 +++++++++--------- > drivers/gpu/drm/msm/disp/dpu1/dpu_plane.h | 4 ++-- > 4 files changed, 14 insertions(+), 13 deletions(-) > -- With best wishes Dmitry