]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
drm/amd/display: Add ODM seamless boot support
authorDuncan Ma <duncan.ma@amd.com>
Thu, 31 Mar 2022 19:13:22 +0000 (15:13 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 3 Jun 2022 20:45:01 +0000 (16:45 -0400)
Revised validation logic when marking for seamless boot. Init resources
accordingly when Pre-OS has ODM enabled. Reset ODM when transitioning
Pre-OS odm to Post-OS non-odm to avoid corruption. Apply logic to set
odm accordingly upon commit.

Signed-off-by: Duncan Ma <duncan.ma@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/core/dc_resource.c
drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c
drivers/gpu/drm/amd/display/dc/dcn31/dcn31_optc.c
drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hwseq.c
drivers/gpu/drm/amd/display/dc/inc/hw/timing_generator.h

index b087452e4590831f45ef6984a6550e51609000df..7072b79d1207dae8415497ce55422c24a4c9800d 100644 (file)
@@ -2165,7 +2165,7 @@ static int acquire_resource_from_hw_enabled_state(
 
                if (pipe_ctx->stream_res.tg->funcs->get_optc_source)
                        pipe_ctx->stream_res.tg->funcs->get_optc_source(pipe_ctx->stream_res.tg,
-                                       &numPipes, &id_src[0], &id_src[1]);
+                                               &numPipes, &id_src[0], &id_src[1]);
 
                if (id_src[0] == 0xf && id_src[1] == 0xf) {
                        id_src[0] = tg_inst;
@@ -2177,6 +2177,8 @@ static int acquire_resource_from_hw_enabled_state(
                        if (id_src[i] == 0xf)
                                return -1;
 
+                       pipe_ctx = &res_ctx->pipe_ctx[id_src[i]];
+
                        pipe_ctx->stream_res.tg = pool->timing_generators[tg_inst];
                        pipe_ctx->plane_res.mi = pool->mis[id_src[i]];
                        pipe_ctx->plane_res.hubp = pool->hubps[id_src[i]];
@@ -2190,13 +2192,17 @@ static int acquire_resource_from_hw_enabled_state(
 
                                if (pool->mpc->funcs->read_mpcc_state) {
                                        struct mpcc_state s = {0};
+
                                        pool->mpc->funcs->read_mpcc_state(pool->mpc, pipe_ctx->plane_res.mpcc_inst, &s);
+
                                        if (s.dpp_id < MAX_MPCC)
                                                pool->mpc->mpcc_array[pipe_ctx->plane_res.mpcc_inst].dpp_id =
                                                                s.dpp_id;
+
                                        if (s.bot_mpcc_id < MAX_MPCC)
                                                pool->mpc->mpcc_array[pipe_ctx->plane_res.mpcc_inst].mpcc_bot =
                                                                &pool->mpc->mpcc_array[s.bot_mpcc_id];
+
                                        if (s.opp_id < MAX_OPP)
                                                pipe_ctx->stream_res.opp->mpc_tree_params.opp_id = s.opp_id;
                                }
@@ -2205,6 +2211,7 @@ static int acquire_resource_from_hw_enabled_state(
 
                        if (id_src[i] >= pool->timing_generator_count) {
                                id_src[i] = pool->timing_generator_count - 1;
+
                                pipe_ctx->stream_res.tg = pool->timing_generators[id_src[i]];
                                pipe_ctx->stream_res.opp = pool->opps[id_src[i]];
                        }
index 1cb206dc83526e057081834fe0262c1df7ee6ece..e3a62873c0e70504fe0c6a49f6565b546b3fb5c0 100644 (file)
@@ -1375,6 +1375,11 @@ void dcn10_init_pipes(struct dc *dc, struct dc_state *context)
                pipe_ctx->stream_res.tg = NULL;
                pipe_ctx->plane_res.hubp = NULL;
 
+               if (tg->funcs->is_tg_enabled(tg)) {
+                       if (tg->funcs->init_odm)
+                               tg->funcs->init_odm(tg);
+               }
+
                tg->funcs->tg_init(tg);
        }
 
index 96fac715a77b43e90af65e4cf528f0bc6bc106d2..c4304f25ce9538da4a3d560a665aef452d8a8d0d 100644 (file)
@@ -213,6 +213,26 @@ void optc31_set_drr(
        }
 }
 
+void optc3_init_odm(struct timing_generator *optc)
+{
+       struct optc *optc1 = DCN10TG_FROM_TG(optc);
+
+       REG_SET_5(OPTC_DATA_SOURCE_SELECT, 0,
+                       OPTC_NUM_OF_INPUT_SEGMENT, 0,
+                       OPTC_SEG0_SRC_SEL, optc->inst,
+                       OPTC_SEG1_SRC_SEL, 0xf,
+                       OPTC_SEG2_SRC_SEL, 0xf,
+                       OPTC_SEG3_SRC_SEL, 0xf
+                       );
+
+       REG_SET(OTG_H_TIMING_CNTL, 0,
+                       OTG_H_TIMING_DIV_MODE, 0);
+
+       REG_SET(OPTC_MEMORY_CONFIG, 0,
+                       OPTC_MEM_SEL, 0);
+       optc1->opp_count = 1;
+}
+
 static struct timing_generator_funcs dcn31_tg_funcs = {
                .validate_timing = optc1_validate_timing,
                .program_timing = optc1_program_timing,
@@ -273,6 +293,7 @@ static struct timing_generator_funcs dcn31_tg_funcs = {
                .program_manual_trigger = optc2_program_manual_trigger,
                .setup_manual_trigger = optc2_setup_manual_trigger,
                .get_hw_timing = optc1_get_hw_timing,
+               .init_odm = optc3_init_odm,
 };
 
 void dcn31_timing_generator_init(struct optc *optc1)
index 04360553a43f3303651fca47260faac061facf54..a10ec5919194166ba083ddcf05606643e3fd51ad 100644 (file)
@@ -657,7 +657,7 @@ void dcn32_init_hw(struct dc *dc)
         * Otherwise, if taking control is not possible, we need to power
         * everything down.
         */
-       if (dcb->funcs->is_accelerated_mode(dcb) || dc->config.seamless_boot_edp_requested) {
+       if (dcb->funcs->is_accelerated_mode(dcb) || !dc->config.seamless_boot_edp_requested) {
                hws->funcs.init_pipes(dc, dc->current_state);
                if (dc->res_pool->hubbub->funcs->allow_self_refresh_control)
                        dc->res_pool->hubbub->funcs->allow_self_refresh_control(dc->res_pool->hubbub,
@@ -669,7 +669,7 @@ void dcn32_init_hw(struct dc *dc)
         * To avoid this, power down hardware on boot
         * if DIG is turned on and seamless boot not enabled
         */
-       if (dc->config.seamless_boot_edp_requested) {
+       if (!dc->config.seamless_boot_edp_requested) {
                struct dc_link *edp_links[MAX_NUM_EDP];
                struct dc_link *edp_link;
 
index a89b2230cd2c453c12ba494933dc0da181d4e0cb..62d4683f17a2456beb4f6a26a4bc30c65ecdb207 100644 (file)
@@ -318,6 +318,8 @@ struct timing_generator_funcs {
                        int vmin, int vmax);
        bool (*validate_vtotal_change_limit)(struct timing_generator *optc,
                        uint32_t vtotal_change_limit);
+
+       void (*init_odm)(struct timing_generator *tg);
 };
 
 #endif