drm/amd/powerplay: expose max engine and memory clock info for powerplay enabled case

[Quan, Evan]

Hi Alex,

Thanks for your information.
I updated the patch using the amdgpu_dpm_get_sclk/mclk APIs which make the patch much simpler.

Regards,
Evan

From: Deucher, Alexander
Sent: Thursday, October 13, 2016 6:04 AM
To: Quan, Evan <Evan.Quan at amd.com>; amd-gfx at lists.freedesktop.org
Cc: Zhang, Jerry <Jerry.Zhang at amd.com>; Huan, Alvin <Alvin.Huan at amd.com>; Quan, Evan <Evan.Quan at amd.com>; Huang, Ray <Ray.Huang at amd.com>
Subject: RE: drm/amd/powerplay: expose max engine and memory clock info for powerplay enabled case

I don't know if we need to add a cgs call to fill in the max_clock_voltage_limits structure.  That's mostly just leftover from radeon.  A better solution would be use the existing amdgpu_dpm_get_sclk and amdgpu_dpm_get_mclk macros.  Maybe something like the attached patch? Unless there is a specific need for the max clocks on AC.  I think in most cases the results will be the same.

Alex

From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf Of Quan, Evan
Sent: Wednesday, October 12, 2016 1:53 AM
To: amd-gfx at lists.freedesktop.org<mailto:amd-gfx at lists.freedesktop.org>
Cc: Zhang, Jerry; Huan, Alvin; Quan, Evan; Huang, Ray
Subject: drm/amd/powerplay: expose max engine and memory clock info for powerplay enabled case

Hi All,

We found the max engine clock and max memory clock are both wrongly reported as 0(by AMDGPU_INFO_DEV_INFO ioctl).
The attached patch tries to fix it.
Please help to review it.
Any comment is welcomed.

Regards,
Evan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161013/0af5d4f8/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-drm-amd-amdgpu-expose-max-engine-and-memory-clock-fo.patch
Type: application/octet-stream
Size: 1621 bytes
Desc: 0001-drm-amd-amdgpu-expose-max-engine-and-memory-clock-fo.patch
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161013/0af5d4f8/attachment.obj>