[Accton][wedge800bact] Fix eth1/29/1,eth1/32/4 and eth1/33/3 profileID mapping #808
+321
−1,527
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…D mapping
Description:
This commit fixes incorrect profileIDs mapping for
1. port eth1/29/1(uplink port)
According to the PRD specification, the uplink port should support the following profileIDs:
- PROFILE_800G_8_PAM4_RS544X2N_OPTICAL = 39
- PROFILE_400G_4_PAM4_RS544X2N_OPTICAL = 38
- PROFILE_200G_4_PAM4_RS544X2N_OPTICAL = 25
- PROFILE_100G_4_NRZ_RS528_OPTICAL = 23
2. port eth1/32/4, remove unsupported profileID.
PROFILE_100G_1_PAM4_RS544_OPTICAL = 47,
3. port eth1/33/3, add profileID
PROFILE_50G_2_NRZ_RS528_OPTICAL = 48,
Changes made:
1. Updated eth1/29/1,eth1/32/4 and eth1/33/3 profileIDs mapping in wedge800bact_port_profile_mapping.csv
2. Regenerated platform mapping JSON file using Platform Mapping Configuration Generation tool
3. Update platform mapping CPP with R-JSON format
jcyu-accton
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
This commit fixes incorrect profileIDs mapping for
Changes made:
Pre-submission checklist
pip install -r requirements-dev.txt && pre-commit installpre-commit runpre-commit run --files fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_port_profile_mapping.csv
pre-commit run --files fboss/lib/platform_mapping_v2/generated_platform_mappings/wedge800bact_platform_mapping.json
pre-commit run --files fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.cpp
Summary
Test Plan
Test Plan
The Platform Mapping Configuration Generation (PMCG) Tools used for verification are based on Commit (00fa992)
The changes have been verified to run T0 SAI Test, Agent HW Test, and Link Test by using the overwritten platform mapping JSON file.
The changes also have been verified to run full-EVT+ Link test with optics_one_link_wedge800bact.json,optics_two_link_wedge800bact.json.