Commit 4074bc3f authored by Camille Cho's avatar Camille Cho Committed by Alex Deucher
Browse files

drm/amd/display: Correctly respond in psr enablement interface 



[Why]
dc_link_set_psr_allow_active() always returns true, even in the case
that PSR is not supported.

[How]
Hook up the return value of dc_link_set_psr_allow_active().

Signed-off-by: default avatarCamille Cho <Camille.Cho@amd.com>
Reviewed-by: default avatarJosip Pavic <Josip.Pavic@amd.com>
Acked-by: default avatarAnthony Koo <Anthony.Koo@amd.com>
Acked-by: default avatarRodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent d2e0d826

drivers/gpu/drm/amd/display/dc/core/dc_link.c

+4 −2
Original line number Diff line number Diff line
@@ -2557,12 +2557,14 @@ bool dc_link_set_psr_allow_active(struct dc_link *link, bool allow_active, bool 
	struct dmcu *dmcu = dc->res_pool->dmcu;

	struct dmub_psr *psr = dc->res_pool->psr;



	link->psr_settings.psr_allow_active = allow_active;



	if (psr != NULL && link->psr_settings.psr_feature_enabled)

		psr->funcs->psr_enable(psr, allow_active);

	else if ((dmcu != NULL && dmcu->funcs->is_dmcu_initialized(dmcu)) && link->psr_settings.psr_feature_enabled)

		dmcu->funcs->set_psr_enable(dmcu, allow_active, wait);



	link->psr_settings.psr_allow_active = allow_active;

	else

		return false;



	return true;

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE