Commit b15cde19 authored by Charlene Liu's avatar Charlene Liu Committed by Alex Deucher

drm/amd/display: Add i2c speed arbitration for dc_i2c and hdcp_i2c

[why]
HDCP 1.4 failed on SL8800 SW w/a test driver use.

[how]
Slow down the HW i2c speed when used by HW i2c.
This request: each acquired_i2c_engine setup the i2c speed needed
and sets the I2c engine for HDCP use at release_engine.

This covers SW using HW i2c engine and HDCP using HW i2c engine.
for dmcu using HW i2c engine, needs add similar logic in dmcu fw.
Signed-off-by: default avatarCharlene Liu <Charlene.Liu@amd.com>
Reviewed-by: default avatarChris Park <Chris.Park@amd.com>
Acked-by: default avatarEryk Brol <eryk.brol@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 54e8094a
...@@ -152,6 +152,7 @@ struct dc_caps { ...@@ -152,6 +152,7 @@ struct dc_caps {
uint32_t max_planes; uint32_t max_planes;
uint32_t max_downscale_ratio; uint32_t max_downscale_ratio;
uint32_t i2c_speed_in_khz; uint32_t i2c_speed_in_khz;
uint32_t i2c_speed_in_khz_hdcp;
uint32_t dmdata_alloc_size; uint32_t dmdata_alloc_size;
unsigned int max_cursor_size; unsigned int max_cursor_size;
unsigned int max_video_width; unsigned int max_video_width;
......
...@@ -299,8 +299,12 @@ static bool setup_engine( ...@@ -299,8 +299,12 @@ static bool setup_engine(
/* we have checked I2c not used by DMCU, set SW use I2C REQ to 1 to indicate SW using it*/ /* we have checked I2c not used by DMCU, set SW use I2C REQ to 1 to indicate SW using it*/
REG_UPDATE(DC_I2C_ARBITRATION, DC_I2C_SW_USE_I2C_REG_REQ, 1); REG_UPDATE(DC_I2C_ARBITRATION, DC_I2C_SW_USE_I2C_REG_REQ, 1);
/*set SW requested I2c speed to default, if API calls in it will be override later*/
set_speed(dce_i2c_hw, dce_i2c_hw->ctx->dc->caps.i2c_speed_in_khz);
if (dce_i2c_hw->setup_limit != 0) if (dce_i2c_hw->setup_limit != 0)
i2c_setup_limit = dce_i2c_hw->setup_limit; i2c_setup_limit = dce_i2c_hw->setup_limit;
/* Program pin select */ /* Program pin select */
REG_UPDATE_6(DC_I2C_CONTROL, REG_UPDATE_6(DC_I2C_CONTROL,
DC_I2C_GO, 0, DC_I2C_GO, 0,
...@@ -339,8 +343,6 @@ static void release_engine( ...@@ -339,8 +343,6 @@ static void release_engine(
{ {
bool safe_to_reset; bool safe_to_reset;
/* Restore original HW engine speed */
set_speed(dce_i2c_hw, dce_i2c_hw->default_speed);
/* Reset HW engine */ /* Reset HW engine */
{ {
...@@ -360,6 +362,9 @@ static void release_engine( ...@@ -360,6 +362,9 @@ static void release_engine(
/* HW I2c engine - clock gating feature */ /* HW I2c engine - clock gating feature */
if (!dce_i2c_hw->engine_keep_power_up_count) if (!dce_i2c_hw->engine_keep_power_up_count)
REG_UPDATE_N(SETUP, 1, FN(SETUP, DC_I2C_DDC1_ENABLE), 0); REG_UPDATE_N(SETUP, 1, FN(SETUP, DC_I2C_DDC1_ENABLE), 0);
/*for HW HDCP Ri polling failure w/a test*/
set_speed(dce_i2c_hw, dce_i2c_hw->ctx->dc->caps.i2c_speed_in_khz_hdcp);
/* Release I2C after reset, so HW or DMCU could use it */ /* Release I2C after reset, so HW or DMCU could use it */
REG_UPDATE_2(DC_I2C_ARBITRATION, DC_I2C_SW_DONE_USING_I2C_REG, 1, REG_UPDATE_2(DC_I2C_ARBITRATION, DC_I2C_SW_DONE_USING_I2C_REG, 1,
DC_I2C_SW_USE_I2C_REG_REQ, 0); DC_I2C_SW_USE_I2C_REG_REQ, 0);
......
...@@ -1071,6 +1071,7 @@ static bool dce100_resource_construct( ...@@ -1071,6 +1071,7 @@ static bool dce100_resource_construct(
pool->base.timing_generator_count = pool->base.res_cap->num_timing_generator; pool->base.timing_generator_count = pool->base.res_cap->num_timing_generator;
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 40; dc->caps.i2c_speed_in_khz = 40;
dc->caps.i2c_speed_in_khz = 40;
dc->caps.max_cursor_size = 128; dc->caps.max_cursor_size = 128;
dc->caps.dual_link_dvi = true; dc->caps.dual_link_dvi = true;
dc->caps.disable_dp_clk_share = true; dc->caps.disable_dp_clk_share = true;
......
...@@ -1372,7 +1372,8 @@ static bool dce110_resource_construct( ...@@ -1372,7 +1372,8 @@ static bool dce110_resource_construct(
pool->base.underlay_pipe_index = pool->base.pipe_count; pool->base.underlay_pipe_index = pool->base.pipe_count;
pool->base.timing_generator_count = pool->base.res_cap->num_timing_generator; pool->base.timing_generator_count = pool->base.res_cap->num_timing_generator;
dc->caps.max_downscale_ratio = 150; dc->caps.max_downscale_ratio = 150;
dc->caps.i2c_speed_in_khz = 100; dc->caps.i2c_speed_in_khz = 40;
dc->caps.i2c_speed_in_khz_hdcp = 40;
dc->caps.max_cursor_size = 128; dc->caps.max_cursor_size = 128;
dc->caps.is_apu = true; dc->caps.is_apu = true;
dc->caps.extended_aux_timeout_support = false; dc->caps.extended_aux_timeout_support = false;
......
...@@ -1240,6 +1240,7 @@ static bool dce112_resource_construct( ...@@ -1240,6 +1240,7 @@ static bool dce112_resource_construct(
pool->base.timing_generator_count = pool->base.res_cap->num_timing_generator; pool->base.timing_generator_count = pool->base.res_cap->num_timing_generator;
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 100; dc->caps.i2c_speed_in_khz = 100;
dc->caps.i2c_speed_in_khz_hdcp = 100; /*1.4 w/a not applied by default*/
dc->caps.max_cursor_size = 128; dc->caps.max_cursor_size = 128;
dc->caps.dual_link_dvi = true; dc->caps.dual_link_dvi = true;
dc->caps.extended_aux_timeout_support = false; dc->caps.extended_aux_timeout_support = false;
......
...@@ -1080,6 +1080,7 @@ static bool dce120_resource_construct( ...@@ -1080,6 +1080,7 @@ static bool dce120_resource_construct(
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 100; dc->caps.i2c_speed_in_khz = 100;
dc->caps.i2c_speed_in_khz_hdcp = 100; /*1.4 w/a not applied by default*/
dc->caps.max_cursor_size = 128; dc->caps.max_cursor_size = 128;
dc->caps.dual_link_dvi = true; dc->caps.dual_link_dvi = true;
dc->caps.psp_setup_panel_mode = true; dc->caps.psp_setup_panel_mode = true;
......
...@@ -970,6 +970,7 @@ static bool dce80_construct( ...@@ -970,6 +970,7 @@ static bool dce80_construct(
pool->base.timing_generator_count = res_cap.num_timing_generator; pool->base.timing_generator_count = res_cap.num_timing_generator;
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 40; dc->caps.i2c_speed_in_khz = 40;
dc->caps.i2c_speed_in_khz_hdcp = 40;
dc->caps.max_cursor_size = 128; dc->caps.max_cursor_size = 128;
dc->caps.dual_link_dvi = true; dc->caps.dual_link_dvi = true;
dc->caps.extended_aux_timeout_support = false; dc->caps.extended_aux_timeout_support = false;
...@@ -1168,6 +1169,7 @@ static bool dce81_construct( ...@@ -1168,6 +1169,7 @@ static bool dce81_construct(
pool->base.timing_generator_count = res_cap_81.num_timing_generator; pool->base.timing_generator_count = res_cap_81.num_timing_generator;
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 40; dc->caps.i2c_speed_in_khz = 40;
dc->caps.i2c_speed_in_khz_hdcp = 40;
dc->caps.max_cursor_size = 128; dc->caps.max_cursor_size = 128;
dc->caps.is_apu = true; dc->caps.is_apu = true;
...@@ -1365,6 +1367,7 @@ static bool dce83_construct( ...@@ -1365,6 +1367,7 @@ static bool dce83_construct(
pool->base.timing_generator_count = res_cap_83.num_timing_generator; pool->base.timing_generator_count = res_cap_83.num_timing_generator;
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 40; dc->caps.i2c_speed_in_khz = 40;
dc->caps.i2c_speed_in_khz_hdcp = 40;
dc->caps.max_cursor_size = 128; dc->caps.max_cursor_size = 128;
dc->caps.is_apu = true; dc->caps.is_apu = true;
......
...@@ -1416,6 +1416,7 @@ static bool dcn10_resource_construct( ...@@ -1416,6 +1416,7 @@ static bool dcn10_resource_construct(
dc->caps.max_video_width = 3840; dc->caps.max_video_width = 3840;
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 100; dc->caps.i2c_speed_in_khz = 100;
dc->caps.i2c_speed_in_khz_hdcp = 100; /*1.4 w/a not applied by default*/
dc->caps.max_cursor_size = 256; dc->caps.max_cursor_size = 256;
dc->caps.max_slave_planes = 1; dc->caps.max_slave_planes = 1;
dc->caps.is_apu = true; dc->caps.is_apu = true;
......
...@@ -3801,6 +3801,7 @@ static bool dcn20_resource_construct( ...@@ -3801,6 +3801,7 @@ static bool dcn20_resource_construct(
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 100; dc->caps.i2c_speed_in_khz = 100;
dc->caps.i2c_speed_in_khz_hdcp = 100; /*1.4 w/a not applied by default*/
dc->caps.max_cursor_size = 256; dc->caps.max_cursor_size = 256;
dc->caps.dmdata_alloc_size = 2048; dc->caps.dmdata_alloc_size = 2048;
......
...@@ -1808,6 +1808,7 @@ static bool dcn21_resource_construct( ...@@ -1808,6 +1808,7 @@ static bool dcn21_resource_construct(
dc->caps.max_downscale_ratio = 200; dc->caps.max_downscale_ratio = 200;
dc->caps.i2c_speed_in_khz = 100; dc->caps.i2c_speed_in_khz = 100;
dc->caps.i2c_speed_in_khz_hdcp = 5; /*1.4 w/a applied by default*/
dc->caps.max_cursor_size = 256; dc->caps.max_cursor_size = 256;
dc->caps.dmdata_alloc_size = 2048; dc->caps.dmdata_alloc_size = 2048;
......
...@@ -2593,6 +2593,7 @@ static bool dcn30_resource_construct( ...@@ -2593,6 +2593,7 @@ static bool dcn30_resource_construct(
pool->base.mpcc_count = pool->base.res_cap->num_timing_generator; pool->base.mpcc_count = pool->base.res_cap->num_timing_generator;
dc->caps.max_downscale_ratio = 600; dc->caps.max_downscale_ratio = 600;
dc->caps.i2c_speed_in_khz = 100; dc->caps.i2c_speed_in_khz = 100;
dc->caps.i2c_speed_in_khz_hdcp = 100; /*1.4 w/a not applied by default*/
dc->caps.max_cursor_size = 256; dc->caps.max_cursor_size = 256;
dc->caps.dmdata_alloc_size = 2048; dc->caps.dmdata_alloc_size = 2048;
......
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