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

[Ragile]Adapt kernel 5.10 for broadcom on RA-B6510-48V8C #14809

Merged

Conversation

pettershao-ragilenetworks
Copy link
Contributor

@pettershao-ragilenetworks pettershao-ragilenetworks commented Apr 23, 2023

Why I did it

Adapt kernel 5.10 for broadcom on RA-B6510-48V8C

How I did it

Adapt kernel 5.10 for broadcom on RA-B6510-48V8C

How to verify it

show version
docker ps -a
show platform syseeprom
show platform fan
show platform psustatus
show platform temperature
show platform firmware status
show platform ssdhealth
show interface status
show interface transceiver presence
show interface transceiver eeprom
show environment
sensors
sudo show system-health detail

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

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

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)

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
@pettershao-ragilenetworks pettershao-ragilenetworks changed the title Adapt kernel 5.10 for broadcom on RA-B6510-48V4C [Ragile]Adapt kernel 5.10 for broadcom on RA-B6510-48V4C Apr 24, 2023
Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
@pettershao-ragilenetworks pettershao-ragilenetworks changed the title [Ragile]Adapt kernel 5.10 for broadcom on RA-B6510-48V4C [Ragile]Adapt kernel 5.10 for broadcom on RA-B6510-48V8C Apr 25, 2023
Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
@pettershao-ragilenetworks pettershao-ragilenetworks marked this pull request as ready for review April 26, 2023 07:44
@Stephengzh-Ragilenetworks

@lguohan , please take a time to review it, thanks a lot! It's been three weeks since this PR opened, RA-B6510-48V8C is a very important platform for us to make verification, especially our test team is working on a new contribution of a sonic testbed plugin and verifying on this platform. So please take a time! Cheers. @zhangyanzhao ,please help to push this progress, very grateful.

@pettershao-ragilenetworks
Copy link
Contributor Author

pettershao-ragilenetworks commented Jun 24, 2023

Software & hardware interface test report.docx
@lguohan @yxieca help check test report, thanks!

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
@pettershao-ragilenetworks
Copy link
Contributor Author

report_6510.zip
@lguohan @yxieca newest pytest report with version include platform.json, help check and forward, thanks!

@yxieca yxieca requested a review from prgeor July 6, 2023 20:05
Copy link
Contributor

@prgeor prgeor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pettershao-ragilenetworks why include firmware binary .bin files as part of sonic image?

@pettershao-ragilenetworks
Copy link
Contributor Author

pettershao-ragilenetworks commented Jul 7, 2023

@pettershao-ragilenetworks why include firmware binary .bin files as part of sonic image?

@prgeor
thanks for review, these firmware bins are use to adapt two commands(fix community issue): show platform firmware updates and fwutil update all fw since we can't modify fwutil file, and other vendors seem that have't adapt these two commands at all, and their platform_components.json files are all empty, and we can remove them once community fix this issue, and please check our test report for these functions:
RA-B6510-48V8C_check_log.log
Please check if we need to remove them directly or not. waiting for your replying.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prgeor The reason is that: in order to fix below issue,and it will be compiled in ragile platform only, will not affect other vendor's platform and kernel:
sonic-net/sonic-linux-kernel#317

@prgeor
Copy link
Contributor

prgeor commented Aug 4, 2023

@yxieca please merge

@yxieca yxieca merged commit abccdae into sonic-net:master Aug 4, 2023
17 of 18 checks passed
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
…4809)

* Adapt kernel 5.10 for broadcom on RA-B6510-48V4C

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>

* update

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>

* update

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>

* update

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>

* update

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>

* modify one-image.mk file

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>

* modify debian/rule.mk

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>

* Add platform.json file

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>

---------

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants