On Thu, Oct 26, 2023 at 1:47 PM Hamza Mahfooz <[email protected]> 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 <[email protected]>
> Tested-by: Mark Broadworth <[email protected]>
> Signed-off-by: Hamza Mahfooz <[email protected]>

Acked-by: Alex Deucher <[email protected]>

> ---
>  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;
> --
> 2.42.0
>

Reply via email to