Re: [PATCH 07/14] drm/amd/powerplay: remove enable_clock_power_gatings_tasks from initialize and resume events

2016-08-08 Thread Eric Huang
Hi Tom, I am concerned with the Sclk Deep Sleep feature. If removing this task, UVD/VCE will always power on and Sclk deep sleep will not work. And the power consumption will block our client's Energy start test. So where are you going to re-add this task ? Thanks, Eric On 16-07-28 10:19 A

Re: [PATCH 07/14] drm/amd/powerplay: remove enable_clock_power_gatings_tasks from initialize and resume events

2016-07-28 Thread Zhu, Rex
For the series: Reviewed-by: Rex Zhu Best Regards Rex From: amd-gfx on behalf of Tom St Denis Sent: Thursday, July 28, 2016 10:19:05 PM To: amd-gfx@lists.freedesktop.org Cc: StDenis, Tom Subject: [PATCH 07/14] drm/amd/powerplay: remove

[PATCH 07/14] drm/amd/powerplay: remove enable_clock_power_gatings_tasks from initialize and resume events

2016-07-28 Thread Tom St Denis
Setting PG state this early would cause lock ups in the IP block initialized functions. Signed-off-by: Tom St Denis --- drivers/gpu/drm/amd/powerplay/eventmgr/eventactionchains.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/eventmgr/eventactionchains.c b/dr