-
Notifications
You must be signed in to change notification settings - Fork 522
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.
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
[portsorch]: Add support of cable breakout feature #320
Conversation
Signed-off-by: Volodymyr Samotiy <volodymyrs@mellanox.com>
@nikos-li do you have some concerns about this pull request? Let's discuss them here. |
p.m_index = static_cast<int32_t>(m_portList.size()); // TODO: Assume no deletion of physical port | ||
p.m_port_id = id; | ||
|
||
/* Initialize the port and create router interface and host interface */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
create router interface [](start = 39, length = 23)
this comment here is actualy incorrect, the initializePort no longer crate router interface.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct. let me modify it.
* msft_github/master: [portsorch]: Use sai_serialize api to write to DB (sonic-net#331) [portsorch]: Update comments (sonic-net#333) [switchorch]: Add support of ECMP and LAG hash seed attribute (sonic-net#324) [portsorch]: Add support of cable breakout feature (sonic-net#320)
Signed-off-by: Qi Luo <qiluo-msft@users.noreply.github.com>
- Description For cmis cable, tx power and rx power is not rounding as other eeprom fields - Motivation and Context Fix issue: rounding float value for txpower and rxpower - How Has This Been Tested? Manual test
Signed-off-by: Volodymyr Samotiy volodymyrs@mellanox.com