Skip to content

Commit 7cf82b2

Browse files
vsyrjalajnikula
authored andcommitted
drm/i915/mst: Fix .mode_valid_ctx() return values
.mode_valid_ctx() returns an errno, not the mode status. Fix the code to do the right thing. Cc: [email protected] Cc: Stanislav Lisovskiy <[email protected]> Fixes: d51f25e ("drm/i915: Add DSC support to MST path") Signed-off-by: Ville Syrjälä <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected] Reviewed-by: Jani Nikula <[email protected]> (cherry picked from commit c179903) Signed-off-by: Jani Nikula <[email protected]>
1 parent 20c2dbf commit 7cf82b2

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

drivers/gpu/drm/i915/display/intel_dp_mst.c

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1031,11 +1031,15 @@ intel_dp_mst_mode_valid_ctx(struct drm_connector *connector,
10311031
* Big joiner configuration needs DSC for TGL which is not true for
10321032
* XE_LPD where uncompressed joiner is supported.
10331033
*/
1034-
if (DISPLAY_VER(dev_priv) < 13 && bigjoiner && !dsc)
1035-
return MODE_CLOCK_HIGH;
1034+
if (DISPLAY_VER(dev_priv) < 13 && bigjoiner && !dsc) {
1035+
*status = MODE_CLOCK_HIGH;
1036+
return 0;
1037+
}
10361038

1037-
if (mode_rate > max_rate && !dsc)
1038-
return MODE_CLOCK_HIGH;
1039+
if (mode_rate > max_rate && !dsc) {
1040+
*status = MODE_CLOCK_HIGH;
1041+
return 0;
1042+
}
10391043

10401044
*status = intel_mode_valid_max_plane_size(dev_priv, mode, false);
10411045
return 0;

0 commit comments

Comments
 (0)