Skip to content

Commit 854043d

Browse files
committed
bug fix in set_low_power()
1 parent 068a4dd commit 854043d

2 files changed

Lines changed: 6 additions & 4 deletions

File tree

sonic_platform_base/sonic_xcvr/api/public/c_cmis.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -346,7 +346,10 @@ def get_transceiver_bulk_status(self):
346346
trans_dom['cfo'] = self.vdm_dict['CFO [MHz]'][1][0]
347347
trans_dom['tx_curr_power'] = self.vdm_dict['Tx Power [dBm]'][1][0]
348348
trans_dom['rx_tot_power'] = self.vdm_dict['Rx Total Power [dBm]'][1][0]
349-
trans_dom['rx_sig_power'] = self.vdm_dict['Rx Signal Power [dBm]'][1][0]
349+
try:
350+
trans_dom['rx_sig_power'] = self.vdm_dict['Rx Signal Power [dBm]'][1][0]
351+
except KeyError:
352+
pass
350353
trans_dom['laser_config_freq'] = self.get_laser_config_freq()
351354
trans_dom['laser_curr_freq'] = self.get_current_laser_freq()
352355
trans_dom['tx_config_power'] = self.get_tx_config_power()

sonic_platform_base/sonic_xcvr/api/public/cmis.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -824,9 +824,8 @@ def set_low_power(self, AssertLowPower):
824824
AssertLowPower being 0 means "set to high power"
825825
AssertLowPower being 1 means "set to low power"
826826
'''
827-
if AssertLowPower:
828-
low_power_control = AssertLowPower << 6
829-
self.xcvr_eeprom.write(consts.MODULE_LEVEL_CONTROL, low_power_control)
827+
low_power_control = AssertLowPower << 6
828+
self.xcvr_eeprom.write(consts.MODULE_LEVEL_CONTROL, low_power_control)
830829

831830
def get_loopback_capability(self):
832831
'''

0 commit comments

Comments
 (0)