diff options
author | Lyude Paul <lyude@redhat.com> | 2022-11-14 17:17:53 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2022-11-17 00:19:18 -0500 |
commit | a3ae99598b9f89dd5ac9d2900fb7ffb3dcfa3d35 (patch) | |
tree | 3f660a8f8c59725aebfa69c2ae844590af78ac80 /drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | |
parent | 7cce4cd628bee0d0caff7518c377cf8f599aa38f (diff) |
drm/display/dp_mst: Fix drm_dp_mst_add_affected_dsc_crtcs() return code
Looks like that we're accidentally dropping a pretty important return code
here. For some reason, we just return -EINVAL if we fail to get the MST
topology state. This is wrong: error codes are important and should never
be squashed without being handled, which here seems to have the potential
to cause a deadlock.
Signed-off-by: Lyude Paul <lyude@redhat.com>
Reviewed-by: Wayne Lin <Wayne.Lin@amd.com>
Fixes: 8ec046716ca8 ("drm/dp_mst: Add helper to trigger modeset on affected DSC MST CRTCs")
Cc: <stable@vger.kernel.org> # v5.6+
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_object.c')
0 files changed, 0 insertions, 0 deletions