Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Improve the way to check port type of RJ45 port (#2249)
* Update the presence state of RJ45 port Present/Not present => Link Up/Link Down Use the new platform API to test whether the port is an RJ45 port Signed-off-by: Stephen Sun <stephens@nvidia.com> * Use new platform API to check whether a port is RJ45 and represent present status accordingly Signed-off-by: Stephen Sun <stephens@nvidia.com> * Adjust sfputil and testcases Signed-off-by: Stephen Sun <stephens@nvidia.com> * Adjust sfpshow Signed-off-by: Stephen Sun <stephens@nvidia.com> * Exact is_rj45_port to a common module shared between sfpshow and intfutil Signed-off-by: Stephen Sun <stephens@nvidia.com> * Fall back to old way for checking RJ45 port Signed-off-by: Stephen Sun <stephens@nvidia.com> * Move RJ45 part to platform_sfputil_helper Signed-off-by: Stephen Sun <stephens@nvidia.com> * Remove fallback mechanism in is_rj45_port Signed-off-by: Stephen Sun <stephens@nvidia.com> * Remove get_child_ports which is not used Signed-off-by: Stephen Sun <stephens@nvidia.com> * Temporarily commit Signed-off-by: Stephen Sun <stephens@nvidia.com> * Update unit test Signed-off-by: stephens <stephens@contoso.com> * Adjust unit test Signed-off-by: Stephen Sun <stephens@nvidia.com> * Commit missed files Signed-off-by: Stephen Sun <stephens@nvidia.com> * Add missing files Signed-off-by: stephens <stephens@contoso.com> * Fix typo Signed-off-by: Stephen Sun <stephens@nvidia.com> * Remove code that was committed by mistake. Signed-off-by: Stephen Sun <stephens@nvidia.com> * Fix an issue: the ports should be in nature order in sfputil show presence Signed-off-by: Stephen Sun <stephens@nvidia.com> * Fix present state for RJ45: Link Up/Down => Port Up/Down Signed-off-by: Stephen Sun <stephens@nvidia.com> * LGTM warning supression Signed-off-by: Stephen Sun <stephens@nvidia.com> * LGTM warning supression Signed-off-by: Stephen Sun <stephens@nvidia.com> * Move present state part into another PR Signed-off-by: Stephen Sun <stephens@nvidia.com> * Fix review comments Signed-off-by: Stephen Sun <stephens@nvidia.com> Co-authored-by: stephens <stephens@contoso.com>
- Loading branch information