Skip to content

[device/Accton] Fix i2c fault from ir3570a on 3 models#3245

Merged
lguohan merged 4 commits intosonic-net:masterfrom
roylee123:fix_i2c_fault_from_ir3570a_part2
Jul 31, 2019
Merged

[device/Accton] Fix i2c fault from ir3570a on 3 models#3245
lguohan merged 4 commits intosonic-net:masterfrom
roylee123:fix_i2c_fault_from_ir3570a_part2

Conversation

@roylee123
Copy link
Collaborator

@roylee123 roylee123 commented Jul 31, 2019

I2c burst read may failed due to misoperation of ir3570a(A DC-to-DC converter IC).
As #2966, there are 3 more models have this symptom, as7326-56x, as7726-32x, and as9716-32d.
Also correct typo of naming on as7816-64x and as7716-32x.

Signed-off-by: roy_lee [email protected]

  • What I did
    Disabling i2c function of ir3570a which may failed i2c tranfer to others.
    Close channel of mux after data transfered.
  • How I did it

Identify version of ir3570, if it's ir3570a, disable its alias i2c address.
Enable parameter of driver i2c_mux_pca954x to close channel on after every access.

  • How to verify it

Write 08 to offset 0xcf of systom eeprom and execute i2c block read.
It will return error.
plug-in several transceivers and run "show interfaces transceiver presence" and "sfputil show presence". You may see the different result. (But doing this requires updated xcvrd).

  • Description for the changelog
    As the code diff.

@roylee123 roylee123 closed this Jul 31, 2019
@roylee123 roylee123 reopened this Jul 31, 2019
@lguohan lguohan merged commit ff8e344 into sonic-net:master Jul 31, 2019
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.

3 participants