Skip to content

Conversation

@CliffyWong
Copy link
Contributor

@CliffyWong CliffyWong commented Sep 17, 2025

Description

This PR is about the codes and test codes for supporting the OSFP front panel port mode of 8x200G

Motivation

As PRD required, each OSFP-RHS front panel port should include the follow modes:

  • 2x 800GE
  • 4x 400GE
  • 8x 200GE

and each OSFP port shall be able to be configured individually, regardless of the modes of the other ports.

Based on the requirement, the profile PROFILE_200G_1_PAM4_RS544X2N_OPTICAL = 53 should be programmed. This involves :

  • implementing the necessary code to program this profile
  • adding corresponding hardware test codes for validation
  • enable the Hw PortProfile Test of PROFILE_200G_1_PAM4_RS544X2N_OPTICAL

Test Plan

1.Used clang-format command to pretty the format
2.Cross checking passed, built qsfp_service, qsfp_hw_test successfully.
3.QSFP service could be launched successfully
4.Run QSFP hardware T0 tests with Profile 52&53(PROFILE_800G_4_PAM4_RS544X2N_OPTICAL /PROFILE_200G_1_PAM4_RS544X2N_OPTICAL ) and passed

image
  1. Run QSFP hardware 100% tests with Profile 52&53(PROFILE_800G_4_PAM4_RS544X2N_OPTICAL /PROFILE_200G_1_PAM4_RS544X2N_OPTICAL ) and passed
full

Pls refer the QSFP config used in the test :
qsfp_53.txt

As the logs exceed the limitation,pls refer the full logs in GDriver.

@meta-cla meta-cla bot added the CLA Signed label Sep 17, 2025
@facebook-github-bot
Copy link
Contributor

@birdsoup has imported this pull request. If you are a Meta employee, you can view this in D83297193.

@birdsoup
Copy link
Contributor

birdsoup commented Oct 2, 2025

It looks like all of these changes were already included in one of the PRs for another NPI

@CliffyWong can you confirm if it's OK to just close this PR?

@CliffyWong
Copy link
Contributor Author

CliffyWong commented Oct 3, 2025

It looks like all of these changes were already included in one of the PRs for another NPI

@CliffyWong can you confirm if it's OK to just close this PR?

Hi @birdsoup ,
Yes, it is ok to close it. Thanks!

@birdsoup birdsoup closed this Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants