forked from mirrors/linux
		
	drm/amdgpu/gfx9: update pg_flags after determining if gfx off is possible
We need to set certain power gating flags after we determine
if the firmware version is sufficient to support gfxoff.
Previously we set the pg flags in early init, but we later
we might have disabled gfxoff if the firmware versions didn't
support it.  Move adding the additional pg flags after we
determine whether or not to support gfxoff.
Fixes: 005440066f ("drm/amdgpu: enable gfxoff again on raven series (v2)")
Tested-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
Tested-by: Tom St Denis <tom.stdenis@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: Kai-Heng Feng <kai.heng.feng@canonical.com>
Cc: stable@vger.kernel.org
			
			
This commit is contained in:
		
							parent
							
								
									2f62c5d6ed
								
							
						
					
					
						commit
						98f58ada2d
					
				
					 2 changed files with 4 additions and 5 deletions
				
			
		| 
						 | 
				
			
			@ -604,6 +604,10 @@ static void gfx_v9_0_check_if_need_gfxoff(struct amdgpu_device *adev)
 | 
			
		|||
		    (adev->gfx.rlc_feature_version < 1) ||
 | 
			
		||||
		    !adev->gfx.rlc.is_rlc_v2_1)
 | 
			
		||||
			adev->pm.pp_feature &= ~PP_GFXOFF_MASK;
 | 
			
		||||
		if (adev->pm.pp_feature & PP_GFXOFF_MASK)
 | 
			
		||||
			adev->pg_flags |= AMD_PG_SUPPORT_GFX_PG |
 | 
			
		||||
				AMD_PG_SUPPORT_CP |
 | 
			
		||||
				AMD_PG_SUPPORT_RLC_SMU_HS;
 | 
			
		||||
		break;
 | 
			
		||||
	default:
 | 
			
		||||
		break;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -992,11 +992,6 @@ static int soc15_common_early_init(void *handle)
 | 
			
		|||
 | 
			
		||||
			adev->pg_flags = AMD_PG_SUPPORT_SDMA | AMD_PG_SUPPORT_VCN;
 | 
			
		||||
		}
 | 
			
		||||
 | 
			
		||||
		if (adev->pm.pp_feature & PP_GFXOFF_MASK)
 | 
			
		||||
			adev->pg_flags |= AMD_PG_SUPPORT_GFX_PG |
 | 
			
		||||
				AMD_PG_SUPPORT_CP |
 | 
			
		||||
				AMD_PG_SUPPORT_RLC_SMU_HS;
 | 
			
		||||
		break;
 | 
			
		||||
	default:
 | 
			
		||||
		/* FIXME: not supported yet */
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue