On Wed, 21 Jun 2023, Simon Ser <cont...@emersion.fr> wrote: > On Wednesday, June 21st, 2023 at 14:11, Jani Nikula <jani.nik...@intel.com> > wrote: > >> On Wed, 21 Jun 2023, Simon Ser cont...@emersion.fr wrote: >> >> > On Wednesday, June 21st, 2023 at 14:05, Jani Nikula jani.nik...@intel.com >> > wrote: >> > >> > > > - if (changed) >> > > > + if (hweight32(changed) == 1) >> > > > + drm_kms_helper_connector_hotplug_event(first_changed_connector); >> > > >> > > What if more than one connector share the same hpd pin? >> > >> > Ah, I did not believe this could happen. I'll rework the patch to >> > count the number of changed connectors instead. >> >> A DP++ port is probably the prime example of this, with both DP and HDMI >> connectors. > > Wouldn't that be handled by the separate DP logic though? (The > drm_dp_mst family of helpers.)
It's not DP MST, though. See intel_ddi_init() initializing both DP and HDMI. One encoder, one HPD pin, two connectors. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center
