On Wed, Oct 11, 2017 at 4:48 AM, Wang, Ken <Ken.Wang at amd.com> wrote: > From: Ken Wang <Ken.Wang at amd.com> > > Change-Id: I377029075af1e2e002f7cfd793ddd58d8610e474 > Signed-off-by: Ken Wang <Ken.Wang at amd.com> NAK. We use 10khz units for all other asics. We already multiply this by 10 in amdgpu_kms.c before sending it to userspace: /* return all clocks in KHz */ dev_info.gpu_counter_freq = amdgpu_asic_get_xclk(adev) * 10; Just return adev->clock.spll.reference_freq. Alex > --- > drivers/gpu/drm/amd/amdgpu/soc15.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/soc15.c > b/drivers/gpu/drm/amd/amdgpu/soc15.c > index 7839677..a510c8c 100644 > --- a/drivers/gpu/drm/amd/amdgpu/soc15.c > +++ b/drivers/gpu/drm/amd/amdgpu/soc15.c > @@ -279,10 +279,8 @@ static void soc15_init_golden_registers(struct > amdgpu_device *adev) > } > static u32 soc15_get_xclk(struct amdgpu_device *adev) > { > - if (adev->asic_type == CHIP_VEGA10) > - return adev->clock.spll.reference_freq/4; > - else > - return adev->clock.spll.reference_freq; > + /* return the value in Khz instead of 10Khz*/ > + return adev->clock.spll.reference_freq * 10; > } > > > -- > 2.7.4 > > > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx >