Skip to content

[ycable][credo] Fix the is_link_active API for Credo Ycable #260

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sujinmkang merged 2 commits into from
Feb 15, 2022
Merged

[ycable][credo] Fix the is_link_active API for Credo Ycable #260

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
53f5d9a
[ycable][credo] Fix the is_link_active API for Credo Ycable
vdahiya12 Feb 7, 2022
7a30e52
Merge branch 'master' of https://github.com/Azure/sonic-platform-comm…
vdahiya12 Feb 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 18 additions & 9 deletions sonic_y_cable/credo/y_cable_credo.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -787,15 +787,24 @@ def is_link_active(self, target):

regval_read = struct.unpack(">B", result)

if (regval_read[0] & 0x01):
self.log_info("NIC link is up")
return True
elif ((regval_read[0] >> 2) & 0x01):
self.log_info("TOR A link is up")
return True
elif ((regval_read[0] >> 1) & 0x01):
self.log_info("TOR B link is up")
return True
if target == YCableBase.TARGET_NIC:
if (regval_read[0] & 0x01):
self.log_info("NIC link is up")
return True
else:
return False
elif target == YCableBase.TARGET_TOR_A:
if ((regval_read[0] >> 2) & 0x01):
self.log_info("TOR A link is up")
return True
else:
return False
elif target == YCableBase.TARGET_TOR_B:
if ((regval_read[0] >> 1) & 0x01):
self.log_info("TOR B link is up")
return True
else:
return False
else:
return YCableBase.TARGET_UNKNOWN

Expand Down