From 69e01afd2d83ad56b3d331cbd2b739b793b84f99 Mon Sep 17 00:00:00 2001 From: "Arif, Maisam" Date: Mon, 3 Mar 2025 14:06:15 -0600 Subject: [PATCH] [SWDEV-518976] Fix amd-smi metric clock checks (#146) Change-Id: I7daf6f87c4c8331662b6b8f543e3e7f966724714 Signed-off-by: Maisam Arif [ROCm/amdsmi commit: 3a02cc6f619f06b4ea0f99ffab3cee717c7f3f09] --- projects/amdsmi/amdsmi_cli/amdsmi_commands.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/projects/amdsmi/amdsmi_cli/amdsmi_commands.py b/projects/amdsmi/amdsmi_cli/amdsmi_commands.py index 8aa2bcd622..4ec06309e2 100644 --- a/projects/amdsmi/amdsmi_cli/amdsmi_commands.py +++ b/projects/amdsmi/amdsmi_cli/amdsmi_commands.py @@ -2000,10 +2000,14 @@ class AMDSMICommands(): # Populate the deep sleep status for each clock for clock in clocks: if clocks[clock]["clk"] != "N/A" and clocks[clock]["min_clk"] != "N/A": - if clocks[clock]["clk"] < clocks[clock]["min_clk"]: - clocks[clock]["deep_sleep"] = "ENABLED" + # Default to Disabled + clocks[clock]["deep_sleep"] = "DISABLED" + if self.logger.is_json_format(): + if clocks[clock]["clk"]["value"] < clocks[clock]["min_clk"]["value"]: + clocks[clock]["deep_sleep"] = "ENABLED" else: - clocks[clock]["deep_sleep"] = "DISABLED" + if clocks[clock]["clk"] < clocks[clock]["min_clk"]: + clocks[clock]["deep_sleep"] = "ENABLED" else: clocks[clock]["deep_sleep"] = "N/A"