RE: [PATCH 1/2] drm: distinguish return value of drm_dp_check_and_send_link_address.
Pushed to drm-misc-next with a re-formatted commit message, thanks for your contribution! ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
RE: [PATCH 1/2] drm: distinguish return value of drm_dp_check_and_send_link_address.
On Tuesday, January 12th, 2021 at 7:36 AM, Chen, Xiaogang wrote: > Any comment? > > -Original Message- > From: Xiaogang.Chen > Sent: Monday, January 4, 2021 12:02 AM > To: amd-...@lists.freedesktop.org; Wentland, Harry ; > dri-devel@lists.freedesktop.org; airl...@linux.ie > Cc: Chen, Xiaogang > Subject: [PATCH 1/2] drm: distinguish return value of > drm_dp_check_and_send_link_address. > > From: Xiaogang Chen > > drm_dp_check_and_send_link_address discovers MST device topology. > It can return both positive and negative values. When it returns positive > values there is no error found. If it returns negative values there is error > found, such as get NAK , timeout, etc. Following drm_kms_helper_hotplug_event > should be called when drm_dp_check_and_send_link_address returns positive > value. > > Signed-off-by: Xiaogang Chen To the best of my knowledge, this sounds correct. Please wrap your commit message into 80-character lines so that it's easier to read. Regardless, this is: Acked-by: Simon Ser ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
RE: [PATCH 1/2] drm: distinguish return value of drm_dp_check_and_send_link_address.
[AMD Official Use Only - Internal Distribution Only] Any comment? -Original Message- From: Xiaogang.Chen Sent: Monday, January 4, 2021 12:02 AM To: amd-...@lists.freedesktop.org; Wentland, Harry ; dri-devel@lists.freedesktop.org; airl...@linux.ie Cc: Chen, Xiaogang Subject: [PATCH 1/2] drm: distinguish return value of drm_dp_check_and_send_link_address. From: Xiaogang Chen drm_dp_check_and_send_link_address discovers MST device topology. It can return both positive and negative values. When it returns positive values there is no error found. If it returns negative values there is error found, such as get NAK , timeout, etc. Following drm_kms_helper_hotplug_event should be called when drm_dp_check_and_send_link_address returns positive value. Signed-off-by: Xiaogang Chen --- drivers/gpu/drm/drm_dp_mst_topology.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index 17dbed0..3ef5206 100644 --- a/drivers/gpu/drm/drm_dp_mst_topology.c +++ b/drivers/gpu/drm/drm_dp_mst_topology.c @@ -2650,7 +2650,7 @@ static void drm_dp_mst_link_probe_work(struct work_struct *work) drm_dp_mst_topology_put_mstb(mstb); mutex_unlock(>probe_lock); - if (ret) + if (ret > 0) drm_kms_helper_hotplug_event(dev); } -- 2.7.4 ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel