Commit e5490464 authored by SivapiriyanKumarasamy's avatar SivapiriyanKumarasamy Committed by Alex Deucher

drm/amd/display: fix dp_hdmi_max_pixel_clk units

[Why]
We are incorrectly using dp_hdmi_max_pixel_clk because the units are not clear.

[How]
Rename to dp_hdmi_max_pixel_clk_in_khz, and change mode timing validation to use
the value correctly.
Signed-off-by: default avatarSivapiriyanKumarasamy <sivapiriyan.kumarasamy@amd.com>
Reviewed-by: default avatarAric Cyr <Aric.Cyr@amd.com>
Acked-by: default avatarLeo Li <sunpeng.li@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent b62f95d1
...@@ -2150,7 +2150,7 @@ static bool dp_active_dongle_validate_timing( ...@@ -2150,7 +2150,7 @@ static bool dp_active_dongle_validate_timing(
return false; return false;
} }
if (get_timing_pixel_clock_100hz(timing) > (dongle_caps->dp_hdmi_max_pixel_clk * 10)) if (get_timing_pixel_clock_100hz(timing) > (dongle_caps->dp_hdmi_max_pixel_clk_in_khz * 10))
return false; return false;
return true; return true;
......
...@@ -2304,8 +2304,8 @@ static void get_active_converter_info( ...@@ -2304,8 +2304,8 @@ static void get_active_converter_info(
hdmi_caps = {.raw = det_caps[3] }; hdmi_caps = {.raw = det_caps[3] };
union dwnstream_port_caps_byte2 union dwnstream_port_caps_byte2
hdmi_color_caps = {.raw = det_caps[2] }; hdmi_color_caps = {.raw = det_caps[2] };
link->dpcd_caps.dongle_caps.dp_hdmi_max_pixel_clk = link->dpcd_caps.dongle_caps.dp_hdmi_max_pixel_clk_in_khz =
det_caps[1] * 25000; det_caps[1] * 2500;
link->dpcd_caps.dongle_caps.is_dp_hdmi_s3d_converter = link->dpcd_caps.dongle_caps.is_dp_hdmi_s3d_converter =
hdmi_caps.bits.FRAME_SEQ_TO_FRAME_PACK; hdmi_caps.bits.FRAME_SEQ_TO_FRAME_PACK;
...@@ -2322,7 +2322,7 @@ static void get_active_converter_info( ...@@ -2322,7 +2322,7 @@ static void get_active_converter_info(
translate_dpcd_max_bpc( translate_dpcd_max_bpc(
hdmi_color_caps.bits.MAX_BITS_PER_COLOR_COMPONENT); hdmi_color_caps.bits.MAX_BITS_PER_COLOR_COMPONENT);
if (link->dpcd_caps.dongle_caps.dp_hdmi_max_pixel_clk != 0) if (link->dpcd_caps.dongle_caps.dp_hdmi_max_pixel_clk_in_khz != 0)
link->dpcd_caps.dongle_caps.extendedCapValid = true; link->dpcd_caps.dongle_caps.extendedCapValid = true;
} }
......
...@@ -395,7 +395,7 @@ struct dc_dongle_caps { ...@@ -395,7 +395,7 @@ struct dc_dongle_caps {
bool is_dp_hdmi_ycbcr422_converter; bool is_dp_hdmi_ycbcr422_converter;
bool is_dp_hdmi_ycbcr420_converter; bool is_dp_hdmi_ycbcr420_converter;
uint32_t dp_hdmi_max_bpc; uint32_t dp_hdmi_max_bpc;
uint32_t dp_hdmi_max_pixel_clk; uint32_t dp_hdmi_max_pixel_clk_in_khz;
}; };
/* Scaling format */ /* Scaling format */
enum scaling_transformation { enum scaling_transformation {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment