On 4/2/2024 3:52 PM, Asad Kamal wrote:
> Update max set uclk and sclk reporting format for smu_v_13_0_0
> 

Use aldebaran instead of smu v13.0.0 - both are different. You may also
add the description similar to patch 1.

With those updates,

Reviewed-by: Lijo Lazar <lijo.la...@amd.com>

Thanks,
Lijo

> Signed-off-by: Asad Kamal <asad.ka...@amd.com>
> ---
>  .../gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c   | 16 ++++++++++------
>  1 file changed, 10 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c 
> b/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c
> index f4b31b86e2a4..6f0d7602d2a8 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c
> @@ -759,8 +759,11 @@ static int aldebaran_emit_clk_levels(struct smu_context 
> *smu,
>       switch (type) {
>  
>       case SMU_OD_SCLK:
> -             *offset += sysfs_emit_at(buf, *offset, "%s:\n", "GFXCLK");
> -             fallthrough;
> +             *offset += sysfs_emit_at(buf, *offset, "%s:\n", "OD_SCLK");
> +             *offset += sysfs_emit_at(buf, *offset, "0: %uMhz\n1: %uMhz\n",
> +                                   pstate_table->gfxclk_pstate.curr.min,
> +                                   pstate_table->gfxclk_pstate.curr.max);
> +             return 0;
>       case SMU_SCLK:
>               ret = aldebaran_get_current_clk_freq_by_table(smu, SMU_GFXCLK, 
> &cur_value);
>               if (ret) {
> @@ -788,8 +791,11 @@ static int aldebaran_emit_clk_levels(struct smu_context 
> *smu,
>               break;
>  
>       case SMU_OD_MCLK:
> -             *offset += sysfs_emit_at(buf, *offset, "%s:\n", "MCLK");
> -             fallthrough;
> +             *offset += sysfs_emit_at(buf, *offset, "%s:\n", "OD_MCLK");
> +             *offset += sysfs_emit_at(buf, *offset, "0: %uMhz\n1: %uMhz\n",
> +                                   pstate_table->uclk_pstate.curr.min,
> +                                   pstate_table->uclk_pstate.curr.max);
> +             return 0;
>       case SMU_MCLK:
>               ret = aldebaran_get_current_clk_freq_by_table(smu, SMU_UCLK, 
> &cur_value);
>               if (ret) {
> @@ -850,7 +856,6 @@ static int aldebaran_emit_clk_levels(struct smu_context 
> *smu,
>       }
>  
>       switch (type) {
> -     case SMU_OD_SCLK:
>       case SMU_SCLK:
>               for (i = 0; i < display_levels; i++) {
>                       clock_mhz = freq_values[i];
> @@ -863,7 +868,6 @@ static int aldebaran_emit_clk_levels(struct smu_context 
> *smu,
>               }
>               break;
>  
> -     case SMU_OD_MCLK:
>       case SMU_MCLK:
>       case SMU_SOCCLK:
>       case SMU_FCLK:

Reply via email to