diff options
author | Lyude Paul <lyude@redhat.com> | 2019-01-08 16:11:27 -0500 |
---|---|---|
committer | Lyude Paul <lyude@redhat.com> | 2019-01-08 17:58:24 -0500 |
commit | fe7553bef8d676d1d8b40666868b33ec39b9df5d (patch) | |
tree | ddbdf6e4f01282f41b6cb60613d14cc5203fe003 /tools/perf/scripts/python/exported-sql-viewer.py | |
parent | bfeffd155283772bbe78c6a05dec7c0128ee500c (diff) |
drm/amdgpu: Don't ignore rc from drm_dp_mst_topology_mgr_resume()
drm_dp_mst_topology_mgr_resume() returns whether or not it managed to
find the topology in question after a suspend resume cycle, and the
driver is supposed to check this value and disable MST accordingly if
it's gone-in addition to sending a hotplug in order to notify userspace
that something changed during suspend.
Currently, amdgpu just makes the mistake of ignoring the return code
from drm_dp_mst_topology_mgr_resume() which means that if a topology was
removed in suspend, amdgpu never notices and assumes it's still
connected which leads to all sorts of problems.
So, fix this by actually checking the rc from
drm_dp_mst_topology_mgr_resume(). Also, reformat the rest of the
function while we're at it to fix the over-indenting.
Signed-off-by: Lyude Paul <lyude@redhat.com>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Cc: Jerry Zuo <Jerry.Zuo@amd.com>
Cc: <stable@vger.kernel.org> # v4.15+
Link: https://patchwork.freedesktop.org/patch/msgid/20190108211133.32564-2-lyude@redhat.com
Diffstat (limited to 'tools/perf/scripts/python/exported-sql-viewer.py')
0 files changed, 0 insertions, 0 deletions