Update chassis.py

sonic-net · Sep 1, 2023 · c8bf16f · c8bf16f
1 parent 8af5dfd
commit c8bf16f
Showing 1 changed file with 17 additions and 9 deletions.
diff --git a/platform/broadcom/sonic-platform-modules-cel/silverstone-v2/pddf/sonic_platform/chassis.py b/platform/broadcom/sonic-platform-modules-cel/silverstone-v2/pddf/sonic_platform/chassis.py
@@ -20,7 +20,7 @@
 
 REBOOT_CAUSE_PATH = "/sys/devices/platform/cpld_wdt/reason"
 SET_SYS_STATUS_LED = "0x3A 0x39 0x2 0x0 {}"
-SET_LED_MODE_Manual = "0x3a 0x42 0x02 0x00"
+GET_LED_MODE = "0x3a 0x42 0x01"
 
 
 class Chassis(PddfChassis):
@@ -65,15 +65,23 @@ def set_status_led(self, color):
         if BMC_EXIST:
             if color == self.get_status_led():
                 return False
-            status, res = self.helper.ipmi_raw(SET_LED_MODE_Manual)
-            if status != 0:
+            status, res = self.helper.ipmi_raw(GET_LED_MODE)
+            if status != 0 or res == "01":
+                print("SYS LED takes automatic ctrl mode!")
+                return False
+            sys_led_color_map = {
+                'off': '00',
+                'green': '01',
+                'amber': '02',
+                'amber_blink_1hz': '03',
+                'amber_blink_4hz': '04',
+                'green_blink_1hz': '05',
+                'green_blink_4hz': '06'
+            }
+            color_val = sys_led_color_map.get(color.lower(), None)
+            if color_val is None:
+                print("SYS LED color %s not support!" % color)
                 return False
-
-            color_val = "0x1"
-            if color == "green":
-                color_val = "0x1"
-            elif color == "amber":
-                color_val = "0x2"
             status, res = self.helper.ipmi_raw(SET_SYS_STATUS_LED.format(color_val))
             return True if status else False
         else: