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

[Mellanox] fix sfp eeprom unreadable after switching from SW to FW control mode #19563

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

yuazhe
Copy link
Contributor

@yuazhe yuazhe commented Jul 12, 2024

Why I did it

The reading of eeprom instantly after switch the module from sw control to fw control might fail, so add a delay

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
  • 202311
  • 202405

Tested branch (Please provide the tested image version)

  • 202311

Description for the changelog

Link to config_db schema for YANG module changes

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

@yuazhe yuazhe requested a review from lguohan as a code owner July 12, 2024 14:08
@keboliu
Copy link
Collaborator

keboliu commented Jul 14, 2024

/azpw ms_conflict

@yuazhe
Copy link
Contributor Author

yuazhe commented Jul 15, 2024

/azpw ms_conflict

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
@yuazhe yuazhe force-pushed the fix_sfp_eeprom_unreadable branch from 3120e41 to b3f1739 Compare July 15, 2024 06:21
@yuazhe
Copy link
Contributor Author

yuazhe commented Jul 18, 2024

@StormLiangMS,@liushilongbuaa please help to resolve the conflict

@liat-grozovik
Copy link
Collaborator

/azp run ms_conflict

Copy link

No pipelines are associated with this pull request.

@yuazhe
Copy link
Contributor Author

yuazhe commented Jul 29, 2024

/azpw ms_conflict

@liat-grozovik liat-grozovik merged commit 66e491d into sonic-net:master Jul 29, 2024
11 checks passed
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Aug 1, 2024
…ntrol mode (sonic-net#19563)

- Why I did it
The reading of eeprom instantly after switch the module from sw control to fw control might fail, so add a delay

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Aug 2, 2024
…ntrol mode (sonic-net#19563)

- Why I did it
The reading of eeprom instantly after switch the module from sw control to fw control might fail, so add a delay

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #19788

mssonicbld pushed a commit that referenced this pull request Aug 3, 2024
…ntrol mode (#19563)

- Why I did it
The reading of eeprom instantly after switch the module from sw control to fw control might fail, so add a delay

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Aug 22, 2024
…ntrol mode (sonic-net#19563)

- Why I did it
The reading of eeprom instantly after switch the module from sw control to fw control might fail, so add a delay

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #19986

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Aug 22, 2024
…ntrol mode (sonic-net#19563)

- Why I did it
The reading of eeprom instantly after switch the module from sw control to fw control might fail, so add a delay

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Aug 22, 2024
…ntrol mode (sonic-net#19563)

- Why I did it
The reading of eeprom instantly after switch the module from sw control to fw control might fail, so add a delay

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
mssonicbld pushed a commit that referenced this pull request Aug 22, 2024
…ntrol mode (#19563)

- Why I did it
The reading of eeprom instantly after switch the module from sw control to fw control might fail, so add a delay

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.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.

8 participants