If it is verified by SMU team, I am OK with it. Regards, Eric On 2018-04-11 01:19 PM, Alex Deucher wrote: > On Wed, Apr 11, 2018 at 2:31 AM, Rex Zhu <Rex.Zhu at amd.com> wrote: >> we don't have limit of [50ms, 4sec] sampling period. >> smu calculate average gpu power in real time. >> we can read average gpu power through smu message or >> read special register. >> >> This reverts commit 462d8dcc9fec0d89f1ff6a1f93f1d4f670878c71. >> >> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com> > If Eric is ok with this, > > Acked-by: Alex Deucher <alexander.deucher at amd.com> > >> --- >> drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >> index 21c021b..388184e 100644 >> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >> @@ -3364,8 +3364,7 @@ static int smu7_get_gpu_power(struct pp_hwmgr *hwmgr, >> "Failed to start pm status log!", >> return -1); >> >> - /* Sampling period from 50ms to 4sec */ >> - msleep_interruptible(200); >> + msleep_interruptible(20); >> >> PP_ASSERT_WITH_CODE(!smum_send_msg_to_smc(hwmgr, >> PPSMC_MSG_PmStatusLogSample), >> -- >> 1.9.1 >> >> _______________________________________________ >> amd-gfx mailing list >> amd-gfx at lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
