Fix Silverstone fan issue

sonic-net · Jun 24, 2024 · 6b50843 · 6b50843
1 parent c55c098
commit 6b50843
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 28 deletions.
diff --git a/platform/broadcom/sonic-platform-modules-cel/silverstone/sonic_platform/chassis.py b/platform/broadcom/sonic-platform-modules-cel/silverstone/sonic_platform/chassis.py
@@ -335,10 +335,6 @@ def get_watchdog(self):
 
         return self._watchdog
 
-    def get_thermal_manager(self):
-        from .thermal_manager import ThermalManager
-        return ThermalManager
-
     ##############################################################
     ###################### Device methods ########################
     ##############################################################

diff --git a/platform/broadcom/sonic-platform-modules-cel/silverstone/sonic_platform/helper.py b/platform/broadcom/sonic-platform-modules-cel/silverstone/sonic_platform/helper.py
@@ -94,7 +94,7 @@ def ipmi_raw(self, netfn, cmd):
         if ret != 0:
             status = False
         else:
-            result = data
+            result = data.strip()
 
         return status, result
 
@@ -107,7 +107,7 @@ def ipmi_fru_id(self, id, key=None):
             if ret != 0:
                 status = False
             else:
-                result = data
+                result = data.strip()
         else:
             cmd2_args = ["grep", str(key)]
             status, result = self.run_command(cmd1_args, cmd2_args)
@@ -122,7 +122,7 @@ def ipmi_set_ss_thres(self, id, threshold_key, value):
         if ret != 0:
             status = False
         else:
-            result = data
+            result = data.strip()
 
         return status, result
 

diff --git a/platform/broadcom/sonic-platform-modules-cel/silverstone/sonic_platform/thermal_manager.py b/platform/broadcom/sonic-platform-modules-cel/silverstone/sonic_platform/thermal_manager.py