Move to emit_clk_levels from print_clk_levels
Signed-off-by: Lijo Lazar <[email protected]>
---
.../drm/amd/pm/swsmu/smu13/yellow_carp_ppt.c | 25 +++++++++----------
1 file changed, 12 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/yellow_carp_ppt.c
b/drivers/gpu/drm/amd/pm/swsmu/smu13/yellow_carp_ppt.c
index 5d7e671fa3c3..e261edbf7852 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/yellow_carp_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/yellow_carp_ppt.c
@@ -1038,17 +1038,15 @@ static uint32_t
yellow_carp_get_umd_pstate_clk_default(struct smu_context *smu,
return clk_limit;
}
-static int yellow_carp_print_clk_levels(struct smu_context *smu,
- enum smu_clk_type clk_type, char *buf)
+static int yellow_carp_emit_clk_levels(struct smu_context *smu,
+ enum smu_clk_type clk_type, char *buf,
+ int *offset)
{
- int i, idx, size = 0, ret = 0, start_offset = 0;
+ int i, idx, size = *offset, ret = 0, start_offset = *offset;
uint32_t cur_value = 0, value = 0, count = 0;
uint32_t min, max;
uint32_t clk_limit = 0;
- smu_cmn_get_sysfs_buf(&buf, &size);
- start_offset = size;
-
switch (clk_type) {
case SMU_OD_SCLK:
size += sysfs_emit_at(buf, size, "%s:\n", "OD_SCLK");
@@ -1069,17 +1067,17 @@ static int yellow_carp_print_clk_levels(struct
smu_context *smu,
case SMU_FCLK:
ret = yellow_carp_get_current_clk_freq(smu, clk_type,
&cur_value);
if (ret)
- goto print_clk_out;
+ return ret;
ret = yellow_carp_get_dpm_level_count(smu, clk_type, &count);
if (ret)
- goto print_clk_out;
+ return ret;
for (i = 0; i < count; i++) {
idx = (clk_type == SMU_FCLK || clk_type == SMU_MCLK) ?
(count - i - 1) : i;
ret = yellow_carp_get_dpm_freq_by_index(smu, clk_type,
idx, &value);
if (ret)
- goto print_clk_out;
+ return ret;
size += sysfs_emit_at(buf, size, "%d: %uMhz %s\n", i,
value,
cur_value == value ? "*" : "");
@@ -1090,7 +1088,7 @@ static int yellow_carp_print_clk_levels(struct
smu_context *smu,
clk_limit = yellow_carp_get_umd_pstate_clk_default(smu,
clk_type);
ret = yellow_carp_get_current_clk_freq(smu, clk_type,
&cur_value);
if (ret)
- goto print_clk_out;
+ return ret;
min = (smu->gfx_actual_hard_min_freq > 0) ?
smu->gfx_actual_hard_min_freq : smu->gfx_default_hard_min_freq;
max = (smu->gfx_actual_soft_max_freq > 0) ?
smu->gfx_actual_soft_max_freq : smu->gfx_default_soft_max_freq;
if (cur_value == max)
@@ -1111,8 +1109,9 @@ static int yellow_carp_print_clk_levels(struct
smu_context *smu,
break;
}
-print_clk_out:
- return size - start_offset;
+ *offset += size - start_offset;
+
+ return 0;
}
static int yellow_carp_force_clk_levels(struct smu_context *smu,
@@ -1349,7 +1348,7 @@ static const struct pptable_funcs yellow_carp_ppt_funcs =
{
.mode2_reset = yellow_carp_mode2_reset,
.get_dpm_ultimate_freq = yellow_carp_get_dpm_ultimate_freq,
.od_edit_dpm_table = yellow_carp_od_edit_dpm_table,
- .print_clk_levels = yellow_carp_print_clk_levels,
+ .emit_clk_levels = yellow_carp_emit_clk_levels,
.force_clk_levels = yellow_carp_force_clk_levels,
.set_performance_level = yellow_carp_set_performance_level,
.set_fine_grain_gfx_freq_parameters =
yellow_carp_set_fine_grain_gfx_freq_parameters,
--
2.49.0