Re: [PATCH 04/19] drm/msm/dp: replace ST_DISPLAY_OFF with power_on in msm_dp_hpd_unplug_handle()

[Jessica Zhang]

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_abhinavk@xxxxxxxxxxx>
> >
> > 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_abhinavk@xxxxxxxxxxx>
> > Signed-off-by: Jessica Zhang <jessica.zhang@xxxxxxxxxxxxxxxx>
> > ---
> > 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.

Thanks,

Jessica Zhang

>