|
[AMD Public Use]
Ah, yes, indeed.
Thanks,
Alex
From: Quan, Evan <Evan.Quan@xxxxxxx>
Sent: Monday, July 6, 2020 9:31 PM To: Alex Deucher <alexdeucher@xxxxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx> Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx> Subject: RE: [PATCH] drm/amdgpu/swSMU: drop pm_enabled check in set mp1 state [AMD Official Use Only - Internal Distribution Only]
I think you may wrongly treated the "pm_enabled" as "dpm_enabled". pm_enabled should be always true unless user specifies "dpm=0" on module loading. -----Original Message----- From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Alex Deucher Sent: Tuesday, July 7, 2020 3:16 AM To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx> Subject: [PATCH] drm/amdgpu/swSMU: drop pm_enabled check in set mp1 state We need to set the mp1 state in PCI shutdown and certain reset cases which happens after we've already suspended the SMU. SMU suspend sets pm_enabled to false which prevents this code from being run. Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx> --- drivers/gpu/drm/amd/powerplay/amdgpu_smu.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c index fe4948aa662f..0ed75a9897eb 100644 --- a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c +++ b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c @@ -1924,9 +1924,6 @@ int smu_set_mp1_state(struct smu_context *smu, uint16_t msg; int ret; -if (!smu->pm_enabled) -return -EOPNOTSUPP; - mutex_lock(&smu->mutex); switch (mp1_state) { -- 2.25.4 _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://nam11.safelinks.protection.outlook.com/?url=""> |
_______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
