On Mon, Jul 14, 2025 at 01:38:55PM -0700, Jessica Zhang wrote: > > > On 7/14/2025 5:27 AM, Dmitry Baryshkov wrote: > > On Fri, Jul 11, 2025 at 05:58:09PM -0700, Jessica Zhang wrote: > > > From: Abhinav Kumar <quic_abhin...@quicinc.com> > > > > > > msm_dp_hpd_unplug_handle() checks if the display was already disabled and > > > if so does not transition to ST_DISCONNECT_PENDING state and goes directly > > > to ST_DISCONNECTED. The same result can be achieved with the !power_on > > > check. > > > > > > Replace ST_DISPLAY_OFF with !power_on to achieve the same outcome. > > > > > > Signed-off-by: Abhinav Kumar <quic_abhin...@quicinc.com> > > > Signed-off-by: Jessica Zhang <jessica.zh...@oss.qualcomm.com> > > > --- > > > Note: Taken from https://patchwork.freedesktop.org/series/142010/ > > > --- > > > drivers/gpu/drm/msm/dp/dp_display.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Please squash all state-related patches into a single one. It would make > > it easier to review and more logical. > > Hi Dmitry, > > Ack -- I'd wanted to keep all the patches small, but I can squash patches > 4-16 into 1 patch if that makes it easier for you.
I think it's easier this way: we get rid of the HPD state machine and use flags all over the place. -- With best wishes Dmitry
