Re: [PATCH] drm/dp: Clarify that wait_hpd_asserted() is not optional for panels

2024-03-19 Thread Doug Anderson
Hi, On Tue, Mar 19, 2024 at 1:55 PM Dmitry Baryshkov wrote: > > > -* panel to finish powering on. This is an optional function. > > +* panel to finish powering on. It is optional for DP AUX > > controllers > > +* to implement this function but

Re: [PATCH] drm/dp: Clarify that wait_hpd_asserted() is not optional for panels

2024-03-19 Thread Dmitry Baryshkov
On Tue, 19 Mar 2024 at 22:39, Abhinav Kumar wrote: > > > > On 3/19/2024 1:16 PM, Dmitry Baryshkov wrote: > > On Tue, 19 Mar 2024 at 21:02, Abhinav Kumar > > wrote: > >> > >> > >> > >> On 3/19/2024 11:35 AM, Dmitry Baryshkov wrote: > >>> On Tue, 19 Mar 2024 at 20:15, Douglas Anderson > >>>

Re: [PATCH] drm/dp: Clarify that wait_hpd_asserted() is not optional for panels

2024-03-19 Thread Abhinav Kumar
On 3/19/2024 1:16 PM, Dmitry Baryshkov wrote: On Tue, 19 Mar 2024 at 21:02, Abhinav Kumar wrote: On 3/19/2024 11:35 AM, Dmitry Baryshkov wrote: On Tue, 19 Mar 2024 at 20:15, Douglas Anderson wrote: In response to my patch removing the "wait for HPD" logic at the beginning of the MSM

Re: [PATCH] drm/dp: Clarify that wait_hpd_asserted() is not optional for panels

2024-03-19 Thread Dmitry Baryshkov
On Tue, 19 Mar 2024 at 21:02, Abhinav Kumar wrote: > > > > On 3/19/2024 11:35 AM, Dmitry Baryshkov wrote: > > On Tue, 19 Mar 2024 at 20:15, Douglas Anderson > > wrote: > >> > >> In response to my patch removing the "wait for HPD" logic at the > >> beginning of the MSM DP transfer() callback

Re: [PATCH] drm/dp: Clarify that wait_hpd_asserted() is not optional for panels

2024-03-19 Thread Abhinav Kumar
On 3/19/2024 11:35 AM, Dmitry Baryshkov wrote: On Tue, 19 Mar 2024 at 20:15, Douglas Anderson wrote: In response to my patch removing the "wait for HPD" logic at the beginning of the MSM DP transfer() callback [1], we had some debate about what the "This is an optional function" meant in

Re: [PATCH] drm/dp: Clarify that wait_hpd_asserted() is not optional for panels

2024-03-19 Thread Dmitry Baryshkov
On Tue, 19 Mar 2024 at 20:15, Douglas Anderson wrote: > > In response to my patch removing the "wait for HPD" logic at the > beginning of the MSM DP transfer() callback [1], we had some debate > about what the "This is an optional function" meant in the > documentation of the wait_hpd_asserted()

Re: [PATCH] drm/dp: Clarify that wait_hpd_asserted() is not optional for panels

2024-03-19 Thread Abhinav Kumar
On 3/19/2024 11:14 AM, Douglas Anderson wrote: In response to my patch removing the "wait for HPD" logic at the beginning of the MSM DP transfer() callback [1], we had some debate about what the "This is an optional function" meant in the documentation of the wait_hpd_asserted() callback.

[PATCH] drm/dp: Clarify that wait_hpd_asserted() is not optional for panels

2024-03-19 Thread Douglas Anderson
In response to my patch removing the "wait for HPD" logic at the beginning of the MSM DP transfer() callback [1], we had some debate about what the "This is an optional function" meant in the documentation of the wait_hpd_asserted() callback. Let's clarify. As talked about in the MSM DP patch