[Public] > -----Original Message----- > From: Lyude Paul <ly...@redhat.com> > Sent: Saturday, June 29, 2024 1:40 AM > To: Lin, Wayne <wayne....@amd.com>; amd-gfx@lists.freedesktop.org; dri- > de...@lists.freedesktop.org > Cc: jani.nik...@intel.com; imre.d...@intel.com; dan...@ffwll.ch; Wentland, > Harry <harry.wentl...@amd.com>; Zuo, Jerry <jerry....@amd.com>; > sta...@vger.kernel.org > Subject: Re: [PATCH 2/3] drm/dp_mst: Skip CSN if topology probing is not > done yet > > On Thu, 2024-06-27 at 09:04 +0000, Lin, Wayne wrote: > > > > I understand your concern. My patch will just check whether mst > > manager starts the probing process or not by confirming whether we > > sent LINK_ADDRESS to the 1st mst branch already. It will drop the CSN > > event only when the event comes earlier than the probing. The CSN > > events occur during topology probing should still have chance to be > > handled after probing process release the > > mgr->probe_lock > > I think. Does this make sense to you please? Thanks! > > Yeah - that seems like the perfect solution :), sounds good to me
Thanks, Lyude! Could you help to merge drm changes - the [PATCH 1/3] and this one [PATCH 2/3] then please? The last one [PATCH 3/3], changes in amd only, I'll ping a gain for review. Appreciate : ) > > > > > > > } else if (up_req->msg.req_type == > > > > DP_RESOURCE_STATUS_NOTIFY) { > > > > const struct drm_dp_resource_status_notify *res_stat = > > > > &up_req->msg.u.resource_stat; > > > > > > -- > > > Cheers, > > > Lyude Paul (she/her) > > > Software Engineer at Red Hat > > > > -- > > Regards, > > Wayne Lin > > > > -- > Cheers, > Lyude Paul (she/her) > Software Engineer at Red Hat -- Regards, Wayne Lin