Thanks for the quick resolution! Might want to add a Fixes header just
for reference when people are reading commit messages?

I can confirm that this fixed the issue introduced by that previous
patch, though!

Patch is Tested-by: Matt Coffin <mcoffi...@gmail.com>

On 10/20/20 12:48 AM, Yifan Zhang wrote:
> [Why]
> Screen corruption on Navi10 card
> 
> [How]
> Set system context in DCN only on Renoir and Cezanne
> 
> Signed-off-by: Yifan Zhang <yifan1.zh...@amd.com>
> ---
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> 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 3d534a4da20b..6855aad7f312 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -1061,10 +1061,12 @@ static int amdgpu_dm_init(struct amdgpu_device *adev)
>  
>       dc_hardware_init(adev->dm.dc);
>  
> -     mmhub_read_system_context(adev, &pa_config);
> +     if (adev->asic_type == CHIP_RENOIR) {
> +             mmhub_read_system_context(adev, &pa_config);
>  
> -     // Call the DC init_memory func
> -     dc_setup_system_context(adev->dm.dc, &pa_config);
> +             // Call the DC init_memory func
> +             dc_setup_system_context(adev->dm.dc, &pa_config);
> +     }
>  
>       adev->dm.freesync_module = mod_freesync_create(adev->dm.dc);
>       if (!adev->dm.freesync_module) {
> 
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to