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

Add Wistron SW-TO3200K and 6512-32R platform support #11818

Merged
merged 1 commit into from
Aug 26, 2022

Conversation

RogerX87
Copy link
Contributor

Signed-off-by: RogerX87 [email protected]

Add sw-to3200k and 6512-32r support for 202111

Why I did it

Add sw-to3200k and 6512-32r support for 202111

How I did it

Add code in device and platform folder for 6512-32r
Update sw-to3200k code both in device and platform folder

How to verify it

Install on Wistron device and run command to verify, please check attachment for log
Signed-off-by: RogerX87 [email protected]

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)

3200k_202111.log

@RogerX87 RogerX87 requested a review from lguohan as a code owner August 23, 2022 08:02
@WistronNetwork
Copy link
Contributor

@yxieca , @sujinmkang , @lizhuhuams would you please review, thanks.

@yxieca yxieca self-requested a review August 24, 2022 17:24
@yxieca
Copy link
Contributor

yxieca commented Aug 24, 2022

@RogerX87 why is this PR directly against 202111 branch? Is there a master PR that cannot be cherry-pickec to 202111 branch cleanly?

@WistronNetwork
Copy link
Contributor

@yxieca , actually we indeed request for backport 202111 but it didn't worked, and we also do some modification specific to 202111.

@yxieca
Copy link
Contributor

yxieca commented Aug 25, 2022

@yxieca , actually we indeed request for backport 202111 but it didn't worked, and we also do some modification specific to 202111.

@WistronNetwork thanks for the clarification, in the future, you could state in the PR comment that this PR is cherry-picking master branch PR #xxxx. so that reviewer won't ask this question again :-)

@yxieca
Copy link
Contributor

yxieca commented Aug 25, 2022

@RogerX87 you might need to rebase your PR to the latest 202111 branch head to pick up the build fixes.

@WistronNetwork
Copy link
Contributor

@yxieca , thanks for kindly remind, please review again.

@yxieca yxieca merged commit ad73953 into sonic-net:202111 Aug 26, 2022
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