Ah yep! I thought you had push rights for some reason
Also, just so patchwork picks up on it before I push:
Reviewed-by: Lyude Paul lyude@redhat.com
On Wed, 2024-07-03 at 08:13 +0000, Lin, Wayne wrote:
[Public]
-----Original Message----- From: Lyude Paul lyude@redhat.com Sent: Saturday, June 29, 2024 1:40 AM To: Lin, Wayne Wayne.Lin@amd.com; amd-gfx@lists.freedesktop.org; dri- devel@lists.freedesktop.org Cc: jani.nikula@intel.com; imre.deak@intel.com; daniel@ffwll.ch; Wentland, Harry Harry.Wentland@amd.com; Zuo, Jerry Jerry.Zuo@amd.com; stable@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