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

enable the emc2305 fan controller and NCP power controller 30ms timeo… #8138

Merged
merged 1 commit into from
Aug 12, 2021
Merged

enable the emc2305 fan controller and NCP power controller 30ms timeo… #8138

merged 1 commit into from
Aug 12, 2021

Conversation

jerseyang
Copy link
Contributor

…ut mechanism

Why I did it

fix the dx010 system eeprom unavailable issue

How I did it

enable the i2c slave 30ms timeout mechanism

How to verify it

  1. i2cstress test in DX010 iSMT controller bus

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

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

image

@jerseyang jerseyang requested a review from lguohan as a code owner July 9, 2021 08:07
@ghost
Copy link

ghost commented Jul 9, 2021

CLA assistant check
All CLA requirements met.

@lguohan
Copy link
Collaborator

lguohan commented Jul 15, 2021

can you have a description of the exact problem in the comments?

@jerseyang
Copy link
Contributor Author

Hi Guohan,

   this PR fix the syseeprom are not available in SLX in the new branches(202012/master),  when issue happens, the kernel message output like this:

[81101.215711] ismt_smbus 0000:00:13.0: completion wait timed out
[81102.239737] ismt_smbus 0000:00:13.0: completion wait timed out
[81102.246302] pca953x 17-0020: failed reading register
[81103.263749] ismt_smbus 0000:00:13.0: completion wait timed out
[81104.291677] ismt_smbus 0000:00:13.0: completion wait timed out
[81105.311692] ismt_smbus 0000:00:13.0: completion wait timed out
[81106.335728] ismt_smbus 0000:00:13.0: completion wait timed out
[81106.342298] pca953x 17-0020: failed reading register
[81107.359716] ismt_smbus 0000:00:13.0: completion wait timed out
[81108.383696] ismt_smbus 0000:00:13.0: completion wait timed out
[81109.407752] ismt_smbus 0000:00:13.0: completion wait timed out
[81110.431712] ismt_smbus 0000:00:13.0: completion wait timed out
[81110.438271] pca953x 17-0020: failed reading register
[81111.455733] ismt_smbus 0000:00:13.0: completion wait timed out
[81112.479714] ismt_smbus 0000:00:13.0: completion wait timed out
[81113.503725] ismt_smbus 0000:00:13.0: completion wait timed out
[81114.527711] ismt_smbus 0000:00:13.0: completion wait timed out
[81114.534273] pca953x 17-0020: failed reading register
[81115.551666] ismt_smbus 0000:00:13.0: completion wait timed out
[81116.575710] ismt_smbus 0000:00:13.0: completion wait timed out

@jerseyang
Copy link
Contributor Author

jerseyang commented Jul 19, 2021 via email

@yxieca yxieca requested a review from sujinmkang July 29, 2021 01:45
@Blueve
Copy link
Contributor

Blueve commented Jul 29, 2021

Is this issue only happens on SLX?

@jerseyang
Copy link
Contributor Author

Yeah, I have only found this issue in SLX so far.

@sujinmkang
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit f2e5eeb into sonic-net:master Aug 12, 2021
qiluo-msft pushed a commit that referenced this pull request Aug 12, 2021
…ut mechanism (#8138)

Why I did it
fix the dx010 system eeprom unavailable issue

How I did it
enable the i2c slave 30ms timeout mechanism

How to verify it
i2cstress test in DX010 iSMT controller bus

Co-authored-by: nicwu-cel <[email protected]>
judyjoseph pushed a commit that referenced this pull request Aug 25, 2021
…ut mechanism (#8138)

Why I did it
fix the dx010 system eeprom unavailable issue

How I did it
enable the i2c slave 30ms timeout mechanism

How to verify it
i2cstress test in DX010 iSMT controller bus

Co-authored-by: nicwu-cel <[email protected]>
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