Skip to content

Supermicro_arm64_1G_Switch_G3748_Added: Added new platform Supermicroâ€Ķ - #17708

Open
SelvaSupermicro wants to merge 3 commits into
sonic-net:masterfrom
SelvaSupermicro:Supermicro_arm64_1G_Switch_G3748_Added
Open

SelvaSupermicro wants to merge 3 commits into
sonic-net:masterfrom
SelvaSupermicro:Supermicro_arm64_1G_Switch_G3748_Added

Conversation

@SelvaSupermicro

@SelvaSupermicro SelvaSupermicro commented Jan 9, 2024 •

Copy link
Copy Markdown

â€Ķ 1G switch. The device and platform folders are included in this commit

Why I did it

Added support for new platform, arm64-supermicro_sse_g3748-r0. This new platform enables SONiC on Supermicro G3748 which is a 1G/10G/25G Switch with 10G/25G uplink connectivity.
This PR is dependent on sonic-net/sonic-linux-kernel#394

Work item tracking
  • Microsoft ADO (number only):

How I did it

As per Azure SONiC porting guidelines https://github.com/sonic-net/SONiC/wiki/Porting-Guide

How to verify it

Build SONiC for arm64 platform as below and install in Supermicro G3748 switch.
make init
make configure PLATFORM=marvell-arm64 PLATFORM_ARCH=arm64
make target/sonic-marvell-arm64.bin

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

This is not fix, hence I am assuming this may not qualify for backport. Otherwise, I would want to backport to 202211 since we are already shipping the switch with SONiC 202211 release branch and many of our customers would like to build from that branch.

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

  • <202211>

Description for the changelog

Added support for new platform Supermicro G3748 1G/10G/25G switch

This code is under branch name "Supermicro_arm64_1G_Switch_G3748_Added"

Link to config_db schema for YANG module changes

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

â€Ķ 1G switch. The device and platform folders are included in this commit
@brholmes1

Copy link
Copy Markdown
Contributor

Hi Team,

Please review the following line as it looks like there is a typo.

lif [ $PLATFORM_AC5X -eq 2 ]; then

Thanks!

Brent

Signed-off-by: Selvaraj RAJAN <selvarajr@supermicro.com>
@prgeor

prgeor commented Mar 22, 2024

Copy link
Copy Markdown
Contributor

@SelvaSupermicro please fix the build failure

Signed-off-by: Selvaraj RAJAN <selvarajr@supermicro.com>
@prgeor

prgeor commented May 25, 2024

Copy link
Copy Markdown
Contributor

@SelvaSupermicro please resolve the merge conflict

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.

3 participants