Skip to content

Add patch to correct i2c timeout#211

Closed
Yi-Shum wants to merge 1 commit into
facebook:mainfrom
Yi-Shum:Add_patch_to_correct_i2c_timeout
Closed

Add patch to correct i2c timeout#211
Yi-Shum wants to merge 1 commit into
facebook:mainfrom
Yi-Shum:Add_patch_to_correct_i2c_timeout

Conversation

@Yi-Shum

@Yi-Shum Yi-Shum commented Mar 21, 2022

Copy link
Copy Markdown
Collaborator

Summary:

  • Correct the i2c timeout time as 35ms.
  • Fix BIC keeping SDA low.
    This issue is encountered if BMC stops transmitting while BIC sends ACK.

Test Plan:

  • Build Code: Pass
  • Apply patch: Pass
Summary:
- Correct the i2c timeout time as 35ms.
- Fix the issue that BIC hold SDA if BMC stop transfer when BIC send ACK

Test Plan:
Build Code: PASS
@Yi-Shum Yi-Shum requested a review from GoldenBug March 21, 2022 06:16
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 21, 2022
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@GoldenBug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

SaraSYLin pushed a commit to Wiwynn/OpenBIC that referenced this pull request Mar 30, 2022
Summary:
- Correct the i2c timeout time as 35ms.
- Fix BIC keeping SDA low.
This issue is encountered if BMC stops transmitting while BIC sends ACK.

Pull Request resolved: facebook#211

Test Plan:
- Build Code: Pass
- Apply patch: Pass

Reviewed By: garnermic

Differential Revision: D35015647

Pulled By: GoldenBug

fbshipit-source-id: 74948e557db061eb080f1cb3b8aea9c5b0cb752d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

2 participants