[PATCH] drm/amd/display: fix S/G display enablement

Harry Wentland harry.wentland at amd.com
Thu Oct 26 19:47:24 UTC 2023


On 2023-10-26 13:29, Hamza Mahfooz wrote:
> An assignment statement was reversed during a refactor which effectively
> disabled S/G display outright. Since, we use
> adev->mode_info.gpu_vm_support to indicate to the rest of the driver
> that S/G display should be enabled and currently it is always set to
> false. So, to fix this set adev->mode_info.gpu_vm_support's value to
> that of init_data.flags.gpu_vm_support.
> 
> Fixes: d059c2ec6a23 ("drm/amd/display: enable S/G display for for recent APUs by default")
> Reported-by: Mark Broadworth <mark.broadworth at amd.com>
> Tested-by: Mark Broadworth <mark.broadworth at amd.com>
> Signed-off-by: Hamza Mahfooz <hamza.mahfooz at amd.com>

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

Harry

> ---
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 781d04eda3d4..adbeb2c897b5 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -1641,7 +1641,7 @@ static int amdgpu_dm_init(struct amdgpu_device *adev)
>  		init_data.flags.gpu_vm_support = (amdgpu_sg_display != 0) && (adev->flags & AMD_IS_APU);
>  	}
>  
> -	init_data.flags.gpu_vm_support = adev->mode_info.gpu_vm_support;
> +	adev->mode_info.gpu_vm_support = init_data.flags.gpu_vm_support;
>  
>  	if (amdgpu_dc_feature_mask & DC_FBC_MASK)
>  		init_data.flags.fbc_support = true;



More information about the amd-gfx mailing list