Skip to content
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

[Netberg][Barefoot] Added support for Aurora 710 #11952

Closed

Conversation

andrewsapronov
Copy link
Contributor

Why I did it

Support Intel Tofino based platforms Netberg Aurora 710
ASIC: Intel Tofino BFN-T10-032D
Pors: 32x 100G

How I did it

Added specification to device/netberg directory
Added platform/barefoot/sonic-platform-modules-netberg contains kernel modules, scripts and sonic_platform packages.
Modified the platform/barefoot/platform-modules-netberg.mk to include Aurora 710 related ID.

How to verify it

Build SONiC

Install the image on the device and verify the related components are installed and shown correctly.

Verification output - nba710-showplatform-master-010822.txt

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

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

Description for the changelog

Link to config_db schema for YANG module changes

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

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 2, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2022

This pull request introduces 11 alerts when merging 5d661b0 into 030de9f - view on LGTM.com

new alerts:

  • 7 for Wrong number of arguments in a class instantiation
  • 2 for Unused local variable
  • 1 for Unused import
  • 1 for Wrong name for an argument in a class instantiation

Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2022

This pull request introduces 8 alerts when merging d564307 into 030de9f - view on LGTM.com

new alerts:

  • 7 for Wrong number of arguments in a class instantiation
  • 1 for Wrong name for an argument in a class instantiation

Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
@andrewsapronov
Copy link
Contributor Author

LGTM warnings related to our code seems to be fixed.

@sujinmkang @prgeor could you please review this PR.
@lguohan please approve and merge.

@lgtm-com
Copy link

lgtm-com bot commented Sep 7, 2022

This pull request introduces 8 alerts when merging 20e6acb into 38cc35f - view on LGTM.com

new alerts:

  • 7 for Wrong number of arguments in a class instantiation
  • 1 for Wrong name for an argument in a class instantiation

Signed-off-by: Andrew Sapronov <andrew.sapronov@gmail.com>
@lgtm-com
Copy link

lgtm-com bot commented Sep 16, 2022

This pull request introduces 8 alerts when merging 4afd47a into a1b50ca - view on LGTM.com

new alerts:

  • 7 for Wrong number of arguments in a class instantiation
  • 1 for Wrong name for an argument in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Oct 5, 2022

This pull request introduces 8 alerts when merging cc43db4 into 1f0699f - view on LGTM.com

new alerts:

  • 7 for Wrong number of arguments in a class instantiation
  • 1 for Wrong name for an argument in a class instantiation

@andrewsapronov
Copy link
Contributor Author

Hi,

merged recent master changes.

@sujinmkang @prgeor please review this PR.
@lguohan please approve and merge

@andrewsapronov
Copy link
Contributor Author

@akokhan could you please verify?

@lgtm-com
Copy link

lgtm-com bot commented Nov 22, 2022

This pull request introduces 8 alerts when merging 1295215 into 283de9a - view on LGTM.com

new alerts:

  • 7 for Wrong number of arguments in a class instantiation
  • 1 for Wrong name for an argument in a class instantiation

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

akokhan
akokhan previously approved these changes Nov 22, 2022
@andrewsapronov
Copy link
Contributor Author

Hi,

@sujinmkang @prgeor please review this PR.
@lguohan please approve and merge

All checks have passed.

@andrewsapronov
Copy link
Contributor Author

Hi,

@sujinmkang @prgeor please review this PR.
@lguohan please approve and merge

All checks have passed.

1 similar comment
@andrewsapronov
Copy link
Contributor Author

Hi,

@sujinmkang @prgeor please review this PR.
@lguohan please approve and merge

All checks have passed.

@andrewsapronov
Copy link
Contributor Author

Hi,

@lguohan please approve and merge

@andrewsapronov
Copy link
Contributor Author

Hi,

@prgeor please review.
@lguohan please approve and merge

All checks have passed and shine green.

BR,
Andrew

@andrewsapronov
Copy link
Contributor Author

Hi,

@prgeor please review.
@lguohan please approve and merge

BR,
Andrew

@andrewsapronov
Copy link
Contributor Author

Hi,

@prgeor please review.
@lguohan please approve and merge

BR,
Andrew

@andrewsapronov andrewsapronov deleted the nba710-initial-support branch May 19, 2023 09:37
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