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 <[email protected]>; [email protected]
Cc: Zhang, Jerry <[email protected]>; Huan, Alvin <[email protected]>; Quan, 
Evan <[email protected]>; Huang, Ray <[email protected]>
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:[email protected]] On Behalf Of Quan, 
Evan
Sent: Wednesday, October 12, 2016 1:53 AM
To: [email protected]<mailto:[email protected]>
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

Attachment: 0001-drm-amd-amdgpu-expose-max-engine-and-memory-clock-fo.patch
Description: 0001-drm-amd-amdgpu-expose-max-engine-and-memory-clock-fo.patch

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to