From d553fda98c60d530392798eab0dddb9ee2ab2da6 Mon Sep 17 00:00:00 2001 From: chiourung_huang Date: Wed, 23 Oct 2024 02:14:15 +0000 Subject: [PATCH] skip CMIS init if config state is ConfigUndefined For the ET8001-2FR4, if there are two breakout ports, the config status for one would be ConfigSuccess and the other would be ConfigUndefined. --- sonic-xcvrd/xcvrd/xcvrd.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sonic-xcvrd/xcvrd/xcvrd.py b/sonic-xcvrd/xcvrd/xcvrd.py index 8f5ec34..ac753cf 100644 --- a/sonic-xcvrd/xcvrd/xcvrd.py +++ b/sonic-xcvrd/xcvrd/xcvrd.py @@ -1024,10 +1024,10 @@ def is_cmis_application_update_required(self, api, app_new, host_lanes_mask): if dp_state[name] != 'DataPathActivated': skip = False break - #name = "ConfigStatusLane{}".format(lane + 1) - #if conf_state[name] != 'ConfigSuccess': - # skip = False - # break + name = "ConfigStatusLane{}".format(lane + 1) + if conf_state[name] != 'ConfigSuccess' and conf_state[name] != 'ConfigUndefined': + skip = False + break return (not skip) return True