[Public]
> -----Original Message-----
> From: Lyude Paul <lyude@xxxxxxxxxx>
> Sent: Saturday, June 29, 2024 1:40 AM
> To: Lin, Wayne <Wayne.Lin@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx; dri-
> devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: jani.nikula@xxxxxxxxx; imre.deak@xxxxxxxxx; daniel@xxxxxxxx; Wentland,
> Harry <Harry.Wentland@xxxxxxx>; Zuo, Jerry <Jerry.Zuo@xxxxxxx>;
> stable@xxxxxxxxxxxxxxx
> 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
