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.

Sign up for GitHub

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

Jump to bottom

[Micas/Platform]platform support M2-W6510-32C #16094

Closed
wants to merge 26 commits into from
Closed

[Micas/Platform]platform support M2-W6510-32C #16094

wants to merge 26 commits into from

Conversation

philo-micas
Copy link
Contributor

@philo-micas philo-micas commented Aug 10, 2023 •
edited
Loading

Why I did it

Add new platform m2-w6510-32c(Trident 3)
ASIC Vendor: Broadcom
Switch ASIC: Trident 3
Port Config: 32×100G QSFP28

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

philo-micas and others added 5 commits August 10, 2023 17:19
@philo-micas
[Micas/Platform]platform support M2-W6510-32C
830596f 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
updata platform-modules-micas.mk and platform-modules-micas.dep
70ea139 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
update platform-modules- micas
690b2e2 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
update one-image.mk file
9d20283 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
Merge branch 'master' into master-b6510-32c
c809762
@philo-micas
Copy link
Contributor Author

report_m2_32c.zip
@yxieca hi xieying, help assign one person to review code, thanks very much!

@philo-micas philo-micas marked this pull request as ready for review August 18, 2023 09:42
@yxieca yxieca requested a review from prgeor August 18, 2023 19:53
@yxieca
Copy link
Contributor

yxieca commented Aug 18, 2023

@philo-micas please if the semgrep check failure. Prince can review after the checker passed.

@philo-micas
Copy link
Contributor Author

philo-micas commented Aug 21, 2023 •
edited
Loading

@philo-micas please if the semgrep check failure. Prince can review after the checker passed.

@yxieca @prgeor Just ignore the semgrep error about xml since 'defusedxml' has not isntalled in sonic currently, help review, thanks!

@philo-micas
fix semgrep error
dcd3b21 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
fix semgrep
1a5898c 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
Copy link
Contributor Author

@philo-micas please if the semgrep check failure. Prince can review after the checker passed.

done!

philo-micas and others added 10 commits October 26, 2023 10:19
@philo-micas
update device and platform files
6fba0a0 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
update permitted_list
125476e 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
update device/micas/x86_64-micas_m2-w6510-32c-r0/monitor.py
6b5807a 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
delete binary files
1c33af7 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
reserve semgrep fix since will cause show env command fail
a52ef3d 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
delete firmware upgrade func
016a356 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
Update wb_xdpe132g5c_pmbus.c
f147dfb
@philo-micas
update wb_xdpe132g5c_pmbus.c
092cfbe 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
Merge branch 'master' into master-b6510-32c
a493e8c
@philo-micas
Update sonic-linux-kernel to use 6.1.38
f08ef1c 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
Copy link
Contributor Author

@lguohan Have done kernel 6.1 adaptation, as we have reviewed and fixed all reviewed comments month ago, (leave one semgrep error as 'defusedxml' not installed in sonic), help push this pr forward again, thanks.

@philo-micas
update rules.mk
f523e2d 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
Copy link
Contributor Author

report_M2-W6510-32C_0112.zip
Kernel 6.1.38 adaption
Platform test report from sonic-mgmt.

@philo-micas
Copy link
Contributor Author

report_M2-W6510-32C_0112.zip Kernel 6.1.38 adaption Platform test report from sonic-mgmt.

@lguohan Hi guohan, test report has been submitted, please help to push forward this pr. Thanks.

@philo-micas
update device files
383f9a8 
Signed-off-by: philo <philo@micasnetworks.com>
@StephenSJCA
Copy link

report_M2-W6510-32C_0112.zip Kernel 6.1.38 adaption Platform test report from sonic-mgmt.

@lguohan Hi guohan, test report has been submitted, please help to push forward this pr. Thanks.

@lguohan @prgeor @yxieca Hi Guohan, Prince, and Ying, please take the time to push forward this PR, it is urgent for us, many thanks!

@philo-micas
triggle rebuild
c46d7f5 
Signed-off-by: philo <philo@micasnetworks.com>
@philo-micas
Copy link
Contributor Author

philo-micas commented Mar 15, 2024 •
edited
Loading

Leave one semgrep error as 'defusedxml' was not installed in sonic.

@philo-micas
Copy link
Contributor Author

Please use new PR link: #20625

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maipbui maipbui maipbui left review comments

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

@prgeor prgeor Awaiting requested review from prgeor

Assignees
No one assigned
Labels
device Not Ready to Merge 🕑 Platform: Micas
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@philo-micas @yxieca @StephenSJCA @maipbui @lguohan