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.q...@amd.com>; email@example.com Cc: Zhang, Jerry <jerry.zh...@amd.com>; Huan, Alvin <alvin.h...@amd.com>; Quan, Evan <evan.q...@amd.com>; Huang, Ray <ray.hu...@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-boun...@lists.freedesktop.org] On Behalf Of Quan, Evan Sent: Wednesday, October 12, 2016 1:53 AM To: firstname.lastname@example.org<mailto:email@example.com> 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
_______________________________________________ amd-gfx mailing list firstname.lastname@example.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx