drm/amd/display: Fix swapped dimension calculations

[Why]
The values calculated in optc1_get_otg_active_size are assigned to the
wrong output parameters, vertical blank is being used for horizontal size
and vice versa. This results in DPG test pattern looking wrong during
hardware init, as the DPG dimensions get assigned from this output, and
potentially other issues.

Reviewed-by: Aric Cyr <aric.cyr@amd.com>
Acked-by: Zaeem Mohamed <zaeem.mohamed@amd.com>
Signed-off-by: Joshua Aberback <joshua.aberback@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/display/dc/optc/dcn10/dcn10_optc.c b/drivers/gpu/drm/amd/display/dc/optc/dcn10/dcn10_optc.c
index 03140e7372d96b4badebd5f80981d6af5245b3e0..336488c0574e51b33c20bafced37c793a9dd94bc 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/optc/dcn10/dcn10_optc.c
+++ b/drivers/gpu/drm/amd/display/dc/optc/dcn10/dcn10_optc.c
@@ -1423,8 +1423,8 @@ bool optc1_get_otg_active_size(struct timing_generator *optc,
                        OTG_H_BLANK_START, &h_blank_start,
                        OTG_H_BLANK_END, &h_blank_end);
 
-       *otg_active_width = v_blank_start - v_blank_end;
-       *otg_active_height = h_blank_start - h_blank_end;
+       *otg_active_width = h_blank_start - h_blank_end;
+       *otg_active_height = v_blank_start - v_blank_end;
        return true;
 }