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

[Accton]Disable IR3507a by "i2cset -y -a" cmd #10385

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: Jostar Yang jostar_yang@accton.com.tw

Why I did it

IR chip will cause i2c block access fail for system eeprom or port eeprom. Old code use below cmd,
"i2cset -y 0 0x4 0xE5 0x01"
"i2cset -y 0 0x4 0x12 0x02"
But in current master code, i2c tools will limit i2c addr range.
Otherwise, it will show below fail.
Error: Chip address out of range (0x08-0x77)!
So add "-a", like "i2cset -y -a 0 0x4 0xE5 0x01" and "i2cset -y -a 0x4 0x12 0x02" to disable IR chip.

How I did it

Add '-a' to below cmd to disable IR chip,
"i2cset -y 0 0x4 0xE5 0x01"
"i2cset -y 0 0x4 0x12 0x02"

How to verify it

Test system eeprom,
In fail case(IR chip cause i2c block access fail),
root@as7326-56x:/home/admin# i2cdump -y -f 0 0x56 i
Error: Block read failed, return code -6

Set cmd to disable IR chip,
"i2cset -y 0 0x4 0xE5 0x01"
"i2cset -y 0 0x4 0x12 0x02"
root@as7326-56x:/home/admin# i2cdump -y -f 0 0x56
No size specified (using byte-data access)
0 1 2 3 4 5 6 7 8 9 a b c d e f 0123456789abcdef
00: 54 6c 76 49 6e 66 6f 00 01 00 a6 25 13 30 36 2f TlvInfo.?.?%?06/
10: 32 32 2f 32 30 31 38 20 31 37 3a 32 33 3a 31 38 22/2018 17:23:18
20: 27 04 52 30 43 42 28 1b 78 38 36 5f 36 34 2d 61 '?R0CB(?x86_64-a
30: 63 63 74 6f 6e 5f 61 73 37 33 32 36 5f 35 36 78 ccton_as7326_56x

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

Signed-off-by: Jostar Yang <jostar_yang@accton.com.tw>
Copy link

CLA Missing ID CLA Not Signed

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.

1 participant