On Tue, Jan 28, 2025 at 07:20:35PM -0800, Jessica Zhang wrote: > From: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > > Up to now the driver has been using encoder to allocate hardware > resources. Switch it to use CRTC id in preparation for the next step. > > Reviewed-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > Signed-off-by: Jessica Zhang <quic_jesszhan@xxxxxxxxxxx> > --- > Changes in v5: > - Reordered to prevent breaking CI and upon partial application > > Changes in v4 (due to rebase): > - moved *_get_assigned_resources() changes for DSPP and LM from > encoder *_virt_atomic_mode_set() to *_assign_crtc_resources() > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 20 ++- > drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 10 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 12 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 189 ++++++++++++++-------------- > drivers/gpu/drm/msm/disp/dpu1/dpu_rm.h | 7 +- > 5 files changed, 113 insertions(+), 125 deletions(-) This patch seems to make kms_color@ctm-max and kms_writeback@writeback-check-output fail on sc7180-trogdor-kingoftown (limozeen didn't start job execution for this commit at this moment). - https://gitlab.freedesktop.org/drm/msm/-/jobs/70213449 - https://gitlab.freedesktop.org/drm/msm/-/jobs/70213446 I've restarted the jobs, but it would be nice to understand the cause. -- With best wishes Dmitry
