Skip to content

[Celestica Seastone2] Add 10G SFP1 front port #8267

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

Merged
merged 1 commit into from
Aug 17, 2021

Conversation

bluecmd
Copy link
Contributor

@bluecmd bluecmd commented Jul 27, 2021

Why I did it

The SFP1 port was disabled in the default configuration.

How I did it

This commit enables the 10G front port for usage.
This was done using a DX030 together with an Arista switch using bcmsh and phy diag xe0 dsc to figure out what lane mappings make sense.

Validated on Celestica Seastone2 DX030.

How to verify it

  1. Own a Celestica DX030
  2. Connect the front SFP1 port to something.
  3. It works :-)

You can do tcpdump -i Ethernet128 -n and you will see both incoming and outgoing LLDP.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Reason: If 202106 is still open for these additions it would be nice for it to be in.
If not, master is fine as well.

Description for the changelog

[Celestica Seastone2] Add SFP1 front port

A picture of a cute animal (not mandatory but encouraged)

image

@bluecmd bluecmd marked this pull request as draft July 27, 2021 18:31
@bluecmd bluecmd force-pushed the bluecmd/dx030-10g-sfp1 branch from 2862f43 to ffbbc9d Compare July 27, 2021 18:47
@bluecmd bluecmd marked this pull request as ready for review July 27, 2021 18:47
@bluecmd bluecmd force-pushed the bluecmd/dx030-10g-sfp1 branch from ffbbc9d to 5be0184 Compare July 29, 2021 14:45
The SFP1 port was disabled in the default configuration.
This commit enables the 10G front port for usage.

Validated on Celestica Seastone2 DX030
@bluecmd bluecmd force-pushed the bluecmd/dx030-10g-sfp1 branch from 5be0184 to 7602317 Compare July 29, 2021 15:15
@bluecmd
Copy link
Contributor Author

bluecmd commented Aug 7, 2021

Ping @mudsut4ke, @sujinmkang, and/or @lguohan - you folks have reviewed Celestica PRs recently, would you care to take this on? It seems nobody has been assigned to it.

@sujinmkang sujinmkang merged commit c64b08f into sonic-net:master Aug 17, 2021
@bluecmd bluecmd deleted the bluecmd/dx030-10g-sfp1 branch August 17, 2021 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants