[PATCH] drm/amd/display: Fix DC_FP_START/END use on non-DCN builds

Harry Wentland harry.wentland at amd.com
Thu Feb 17 21:16:34 UTC 2022



On 2022-02-17 16:05, sunpeng.li at amd.com wrote:
> From: Leo Li <sunpeng.li at amd.com>
> 
> DC_FP_START/END is only defined for CONFIG_DRM_AMD_DC_DCN enabled
> builds, and update_bw_bounding_box() is only valid for DCN ASICs.
> 
> Therefore, wrap the entire thing in the ifdef guard.
> 
> Acked-by: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Leo Li <sunpeng.li at amd.com>

Reviewed-by: Harry Wentland <harry.wentland at amd.com>

Harry

> ---
>  drivers/gpu/drm/amd/display/dc/core/dc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
> index 4ea114725d61..b1ce3c0cf477 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
> @@ -987,13 +987,13 @@ static bool dc_construct(struct dc *dc,
>  		goto fail;
>  #ifdef CONFIG_DRM_AMD_DC_DCN
>  	dc->clk_mgr->force_smu_not_present = init_params->force_smu_not_present;
> -#endif
>  
>  	if (dc->res_pool->funcs->update_bw_bounding_box) {
>  		DC_FP_START();
>  		dc->res_pool->funcs->update_bw_bounding_box(dc, dc->clk_mgr->bw_params);
>  		DC_FP_END();
>  	}
> +#endif
>  
>  	/* Creation of current_state must occur after dc->dml
>  	 * is initialized in dc_create_resource_pool because



More information about the amd-gfx mailing list