disp_dpp_voltage_level_khz is declared and printed, but never assigned.
So when using clock tracepoint the value of this clock is just garbage.
Assign clk->disp_dpp_voltage_level_khz since this is stored in dc_clocks
as other entries.

Fixes: 13b5ca42ca9c ("drm/amd/display: Add tracepoint for capturing clocks 
state")
Signed-off-by: Melissa Wen <[email protected]>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_trace.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_trace.h 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_trace.h
index f33a2c1e0da5..cc70aeb75798 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_trace.h
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_trace.h
@@ -513,6 +513,7 @@ TRACE_EVENT(amdgpu_dm_dc_clocks_state,
            TP_fast_assign(
                           __entry->dispclk_khz = clk->dispclk_khz;
                           __entry->dppclk_khz = clk->dppclk_khz;
+                          __entry->disp_dpp_voltage_level_khz = 
clk->disp_dpp_voltage_level_khz;
                           __entry->dcfclk_khz = clk->dcfclk_khz;
                           __entry->socclk_khz = clk->socclk_khz;
                           __entry->dcfclk_deep_sleep_khz = 
clk->dcfclk_deep_sleep_khz;
-- 
2.53.0

Reply via email to