RE: [PATCH v2 0/3] navi10: Implement overdrive pp_od_clk_voltage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Series is reviewed-by: Evan Quan <evan.quan@xxxxxxx>

> -----Original Message-----
> From: Matt Coffin <mcoffin13@xxxxxxxxx>
> Sent: Saturday, November 9, 2019 5:28 AM
> To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; Quan, Evan <Evan.Quan@xxxxxxx>
> Cc: Matt Coffin <mcoffin13@xxxxxxxxx>; Deucher, Alexander
> <Alexander.Deucher@xxxxxxx>
> Subject: [PATCH v2 0/3] navi10: Implement overdrive pp_od_clk_voltage
> 
> [Why]
> Before this patchset, navi10 users could not utilize the overdrive functionality.
> This prevented them from overclocking, overvolting, or undervolting their cards.
> 
> [How]
> Similar to the vega20 implementation, add the pp_od_clk_voltage interface to
> the navi10 powerplay table.
> 
> [Possible Alternatives]
> This could also be done more generically in smu_v11_0 code, but would require
> more significant changes to the vega20 code, and feature-gating based on chip
> capabilities in the smu to disable overdrive for arcterus.
> I chose this path so as not to completely refactor the vega20 side of things, and
> introduce SMU code which would not play nicely with new ASICs.
> 
> v2: rebase off latest code, and remove an incorrect bounds check
> 
> Matt Coffin (3):
>   drm/amdgpu/navi10: implement sclk/mclk OD via pp_od_clk_voltage
>   drm/amdgpu/navi10: implement GFXCLK_CURVE overdrive
>   drm/amdgpu/navi10: Implement od clk printing
> 
>  drivers/gpu/drm/amd/powerplay/inc/smu_v11_0.h |   2 +
>  drivers/gpu/drm/amd/powerplay/navi10_ppt.c    | 282 ++++++++++++++++++
>  drivers/gpu/drm/amd/powerplay/navi10_ppt.h    |   2 +
>  drivers/gpu/drm/amd/powerplay/smu_v11_0.c     |  27 ++
>  4 files changed, 313 insertions(+)
> 
> --
> 2.23.0

_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux