Skip to content

DellEMC: Z9332f fix platform issue#6953

Closed
aravindmani-1 wants to merge 2 commits intosonic-net:masterfrom
aravindmani-1:fix_z9332f_platform_issues
Closed

DellEMC: Z9332f fix platform issue#6953
aravindmani-1 wants to merge 2 commits intosonic-net:masterfrom
aravindmani-1:fix_z9332f_platform_issues

Conversation

@aravindmani-1
Copy link
Contributor

@aravindmani-1 aravindmani-1 commented Mar 3, 2021

Why I did it

  • Following I2C errors were seen in Z9332f platform during PMON docker startup.

[ 4.211968] i2c i2c-22: new_device: Instantiated device optoe3 at 0x50
[ 63.803230] i2c i2c-22: delete_device: Deleting device optoe3 at 0x50
[ 63.804691] sysfs: cannot create duplicate filename '/devices/pci0000:00/0000:00:1c.0/0000:09:00.0/goodcores-i2c.3/i2c-3/i2c-22/22-0050'
[ 63.818497] i2c i2c-22: Failed to register i2c client optoe1 at 0x50 (-17)

How I did it

  • This occurs during OPTOE driver re-initialization where drivers will be changed based on the transceiver type.
  • Add a small amount of delay after EEPROM driver deletion.

How to verify it

  • Do cold boot/fast boot/warmboot and check kernel logs.

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

@aravindmani-1 aravindmani-1 requested a review from lguohan as a code owner March 3, 2021 11:43
@aravindmani-1 aravindmani-1 requested a review from lguohan March 8, 2021 11:41
@aravindmani-1
Copy link
Contributor Author

@lguohan , @jleveque can you please review this PR?.

@jleveque
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@aravindmani-1
Copy link
Contributor Author

Addressed this PR changes as part of #7457

@aravindmani-1 aravindmani-1 deleted the fix_z9332f_platform_issues branch October 6, 2021 04:05
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