Skip to content

Commit ed6e278

Browse files
Alvin Leealexdeucher
authored andcommitted
drm/amd/display: For cursor P-State allow for SubVP
[Description] - Similar to FPO, SubVP should also force cursor P-State allow instead of relying on natural assertion - Implement code path to force and unforce cursor P-State allow for SubVP Reviewed-by: Samson Tam <samson.tam@amd.com> Acked-by: Hersen Wu <hersenxs.wu@amd.com> Signed-off-by: Alvin Lee <alvin.lee2@amd.com> Tested-by: Daniel Wheeler <daniel.wheeler@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
1 parent e4c33ff commit ed6e278

1 file changed

Lines changed: 2 additions & 15 deletions

File tree

drivers/gpu/drm/amd/display/dc/hwss/dcn32/dcn32_hwseq.c

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -615,12 +615,6 @@ void dcn32_update_force_pstate(struct dc *dc, struct dc_state *context)
615615
pipe->stream->fpo_in_use)) {
616616
if (hubp && hubp->funcs->hubp_update_force_pstate_disallow)
617617
hubp->funcs->hubp_update_force_pstate_disallow(hubp, false);
618-
}
619-
620-
/* Today only FPO uses cursor P-State force. Only clear cursor P-State force
621-
* if it's not FPO.
622-
*/
623-
if (!pipe->stream || !pipe->stream->fpo_in_use) {
624618
if (hubp && hubp->funcs->hubp_update_force_cursor_pstate_disallow)
625619
hubp->funcs->hubp_update_force_cursor_pstate_disallow(hubp, false);
626620
}
@@ -632,17 +626,10 @@ void dcn32_update_force_pstate(struct dc *dc, struct dc_state *context)
632626
struct pipe_ctx *pipe = &context->res_ctx.pipe_ctx[i];
633627
struct hubp *hubp = pipe->plane_res.hubp;
634628

635-
if (pipe->stream && pipe->plane_state && pipe->stream->mall_stream_config.type == SUBVP_MAIN) {
629+
if (pipe->stream && (pipe->stream->mall_stream_config.type == SUBVP_MAIN ||
630+
pipe->stream->fpo_in_use)) {
636631
if (hubp && hubp->funcs->hubp_update_force_pstate_disallow)
637632
hubp->funcs->hubp_update_force_pstate_disallow(hubp, true);
638-
}
639-
640-
if (pipe->stream && pipe->stream->fpo_in_use) {
641-
if (hubp && hubp->funcs->hubp_update_force_pstate_disallow)
642-
hubp->funcs->hubp_update_force_pstate_disallow(hubp, true);
643-
/* For now only force cursor p-state disallow for FPO
644-
* Needs to be added for subvp once FW side gets updated
645-
*/
646633
if (hubp && hubp->funcs->hubp_update_force_cursor_pstate_disallow)
647634
hubp->funcs->hubp_update_force_cursor_pstate_disallow(hubp, true);
648635
}

0 commit comments

Comments
 (0)