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

[centec] fix some bugs on centec tsingma bsp and v682 sonic_platform package #10685

Merged

Conversation

LuiSzee
Copy link
Contributor

@LuiSzee LuiSzee commented Apr 26, 2022

Why I did it

Fix some bugs on centec tsingma bsp and v682 sonic_platform package.

How I did it

  1. add module license for centec mars phy driver
  2. Fix i2c function ability setting for tsingma soc i2c controller
  3. Fix eeprom read error on v682 sonic_platform sfp module

How to verify it

Build SONiC image and verify it on centec E530-48T4X and V682-48Y8C board.

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

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

Description for the changelog

Link to config_db schema for YANG module changes

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

@LuiSzee LuiSzee requested a review from lguohan as a code owner April 26, 2022 05:57
@LuiSzee
Copy link
Contributor Author

LuiSzee commented Apr 28, 2022

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@LuiSzee
Copy link
Contributor Author

LuiSzee commented Apr 29, 2022

/azp run Azure.sonic-buildimage (Build marvell_armhf)

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 10685 in repo Azure/sonic-buildimage

@LuiSzee
Copy link
Contributor Author

LuiSzee commented May 5, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 10685 in repo Azure/sonic-buildimage

@LuiSzee
Copy link
Contributor Author

LuiSzee commented May 5, 2022

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xumia
Copy link
Collaborator

xumia commented Jul 5, 2022

LGTM, @qiluo-msft , could you please help take a look?

@xumia xumia requested a review from qiluo-msft July 5, 2022 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Request for 202111 Branch For PRs being requested for 202111 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants