Re: [PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-04 Thread Abhinav Kumar
On 3/4/2025 7:09 PM, Dmitry Baryshkov wrote: On Wed, 5 Mar 2025 at 00:43, Abhinav Kumar wrote: On 3/4/2025 12:42 PM, Dmitry Baryshkov wrote: On Tue, Mar 04, 2025 at 11:38:24AM -0800, Abhinav Kumar wrote: On 3/3/2025 9:32 PM, Dmitry Baryshkov wrote: On Tue, 4 Mar 2025 at 03:44, Jessic

Re: [PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-04 Thread Dmitry Baryshkov
On Wed, 5 Mar 2025 at 00:43, Abhinav Kumar wrote: > > > > On 3/4/2025 12:42 PM, Dmitry Baryshkov wrote: > > On Tue, Mar 04, 2025 at 11:38:24AM -0800, Abhinav Kumar wrote: > >> > >> > >> On 3/3/2025 9:32 PM, Dmitry Baryshkov wrote: > >>> On Tue, 4 Mar 2025 at 03:44, Jessica Zhang > >>> wrote: > >

Re: [PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-04 Thread Abhinav Kumar
On 3/4/2025 12:42 PM, Dmitry Baryshkov wrote: On Tue, Mar 04, 2025 at 11:38:24AM -0800, Abhinav Kumar wrote: On 3/3/2025 9:32 PM, Dmitry Baryshkov wrote: On Tue, 4 Mar 2025 at 03:44, Jessica Zhang wrote: On 3/3/2025 3:49 PM, Dmitry Baryshkov wrote: On Mon, Mar 03, 2025 at 10:28:00AM

Re: [PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-04 Thread Dmitry Baryshkov
On Tue, Mar 04, 2025 at 11:38:24AM -0800, Abhinav Kumar wrote: > > > On 3/3/2025 9:32 PM, Dmitry Baryshkov wrote: > > On Tue, 4 Mar 2025 at 03:44, Jessica Zhang > > wrote: > > > > > > > > > > > > On 3/3/2025 3:49 PM, Dmitry Baryshkov wrote: > > > > On Mon, Mar 03, 2025 at 10:28:00AM -0800, J

Re: [PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-04 Thread Abhinav Kumar
On 3/3/2025 9:32 PM, Dmitry Baryshkov wrote: On Tue, 4 Mar 2025 at 03:44, Jessica Zhang wrote: On 3/3/2025 3:49 PM, Dmitry Baryshkov wrote: On Mon, Mar 03, 2025 at 10:28:00AM -0800, Jessica Zhang wrote: If new CTLs are reserved by CRTC but atomic_enable() is skipped, the encoders will c

Re: [PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-03 Thread Dmitry Baryshkov
On Tue, 4 Mar 2025 at 03:44, Jessica Zhang wrote: > > > > On 3/3/2025 3:49 PM, Dmitry Baryshkov wrote: > > On Mon, Mar 03, 2025 at 10:28:00AM -0800, Jessica Zhang wrote: > >> If new CTLs are reserved by CRTC but atomic_enable() is skipped, the > >> encoders will configure the stale CTL instead of

Re: [PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-03 Thread Jessica Zhang
On 3/3/2025 3:49 PM, Dmitry Baryshkov wrote: On Mon, Mar 03, 2025 at 10:28:00AM -0800, Jessica Zhang wrote: If new CTLs are reserved by CRTC but atomic_enable() is skipped, the encoders will configure the stale CTL instead of the newly reserved one. The CTLs are propagates in .atomic_mode_s

Re: [PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-03 Thread Dmitry Baryshkov
On Mon, Mar 03, 2025 at 10:28:00AM -0800, Jessica Zhang wrote: > If new CTLs are reserved by CRTC but atomic_enable() is skipped, the > encoders will configure the stale CTL instead of the newly reserved one. The CTLs are propagates in .atomic_mode_set(), not in .atomic_enable(). > > Avoid this

[PATCH RFC] drm/msm/dpu: Force modeset if new CTLs have been reserved

2025-03-03 Thread Jessica Zhang
If new CTLs are reserved by CRTC but atomic_enable() is skipped, the encoders will configure the stale CTL instead of the newly reserved one. Avoid this by setting mode_changed to true if new CTLs have been reserved by CRTC. Note: This patch only adds tracking for the CTL reservation, but eventua