RE: [PATCH 2/3] drm/dp_mst: Skip CSN if topology probing is not done yet

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[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




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux