Add API to check if module is Linear Pluggable Optics (LPO) #599
Conversation
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
longhuan-cisco
force-pushed
the
is_lpo
branch
from
789e40a to
ae64feb
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
| LowPwrRequestSW = 4 | ||
| LowPwrAllowRequestHW = 6 | ||
|
|
||
| LPO_HOST_ELECTRICAL_INTERFACE_IDS = [ |
There was a problem hiding this comment.
@longhuan-cisco which SFF spec version are you using?
There was a problem hiding this comment.
@prgeor
The code here is based on today's sonic_platform_base/sonic_xcvr/codes/public/sff8024.py:
sonic-platform-common/sonic_platform_base/sonic_xcvr/codes/public/sff8024.py
Lines 221 to 224 in 1ccb062
sonic-platform-common/sonic_platform_base/sonic_xcvr/codes/public/sff8024.py
Lines 415 to 418 in 1ccb062
But according to latest rev4.13 July 11, 2025 version of SFF-8024, MEDIA_INTERFACE_IDS for LPO starting from 143:
I think we need to update sonic_platform_base/sonic_xcvr/codes/public/sff8024.py, will raise a PR about this
There was a problem hiding this comment.
There's already a PR #596 to update LPO media codes in sonic_platform_base/sonic_xcvr/codes/public/sff8024.py based on latest SFF-8024.
Will update lpo API accordingly.
There was a problem hiding this comment.
@longhuan-cisco @prgeor I corrected the LPO related IDs since this change was causing test failures for #596
4 participants
Description
Add
is_lpoAPI to check if the module is Linear Pluggable Optics (LPO)Motivation and Context
This API will be consumed by sonic-net/sonic-platform-daemons#687
How Has This Been Tested?
Additional Information (Optional)