Re: [PATCH] drm/amd/display: Only poll analog connectors

["Timur Kristóf" <timur.kristof@gmail.com>]

On Monday, January 19, 2026 6:21:40 PM Central European Standard Time Harry 
Wentland wrote:
> On 2026-01-18 08:03, Timur Kristóf wrote:
> > Analog connectors may be hot-plugged unlike other connector
> > types that don't support HPD.
> > 
> > Stop DRM from polling other connector types that don't
> > support HPD, such as eDP, LVDS, etc. These were wrongly
> > polled when analog connector support was added,
> > causing issues with the seamless boot process.
> > 
> > Fixes: c4f3f114e73c ("drm/amd/display: Poll analog connectors (v3)")
> > Signed-off-by: Timur Kristóf <timur.kristof@gmail.com>
> > Reported-by: Matthew Schwartz <matthew.schwartz@linux.dev>
> > ---
> > 
> >  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_irq.c | 10 ++++++++--
> >  1 file changed, 8 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_irq.c
> > b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_irq.c index
> > 0a2a3f233a0e..e7b0928bd3db 100644
> > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_irq.c
> > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_irq.c
> > @@ -915,13 +915,19 @@ void amdgpu_dm_hpd_init(struct amdgpu_device *adev)
> > 
> >  		struct amdgpu_dm_connector *amdgpu_dm_connector;
> >  		const struct dc_link *dc_link;
> > 
> > -		use_polling |= connector->polled != 
DRM_CONNECTOR_POLL_HPD;
> > -
> > 
> >  		if (connector->connector_type == 
DRM_MODE_CONNECTOR_WRITEBACK)
> >  		
> >  			continue;
> >  		
> >  		amdgpu_dm_connector = 
to_amdgpu_dm_connector(connector);
> > 
> > +		/*
> > +		 * Analog connectors may be hot-plugged unlike other 
connector
> > +		 * types that don't support HPD. Only poll analog 
connectors.
> > +		 */
> > +		use_polling |=
> > +			amdgpu_dm_connector->dc_link &&
> > +			
dc_connector_supports_analog(amdgpu_dm_connector->dc_link->link_id.id)
> > ;
> 
> Why isn't the aconnector->base.polled assignment in
> amdgpu_dm_connector_init_helper doing the trick?
> 
> Harry

Hi Harry,

The issue is that checking "connector->polled != DRM_CONNECTOR_POLL_HPD" will 
also match connectors that don't support hotplugging, such as eDP, LVDS etc.

amdgpu_dm_connector_init_helper is working correctly, it doesn't set the 
DRM_CONNECTOR_POLL_HPD flag for neither analog connectors, nor eDP, LVDS, etc. 
which is correct. The problem is with this check in amdgpu_dm_hpd_init.

Best regards,
Timur

> 
> > +
> > 
> >  		dc_link = amdgpu_dm_connector->dc_link;
> >  		
> >  		/*