diff options
author | Tony Zou <zoutao@huaqin.corp-partner.google.com> | 2019-03-01 15:49:30 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-08-23 00:12:18 +0000 |
commit | 09f808f671a92c8c3d444442b7887d68d905e1b1 (patch) | |
tree | 9e6483e70d3b899adc5835e840787f0566095bda /common | |
parent | fb489129498fa59afbcf6e67fbab0ffd0fab724c (diff) | |
download | chrome-ec-09f808f671a92c8c3d444442b7887d68d905e1b1.tar.gz |
i2c:xfer i2cwrite invalid in ec console
In command
xfer i2cwrite port addr offset value
16bit offset and 8bit offset call the wrong i2c write interfacefix it.
BRANCH=none
BUG=b:126820386
TEST=1:) Build flapjack EC and flash to DUT ,
in ec console can read/write i2c device correctly.
2:) Build kukui EC , build pass.
Change-Id: Ib3aa058ae0917fe62f38bc500a8227d6e36dbab1
Reviewed-on: https://chromium-review.googlesource.com/1496676
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Tony Zou <zoutao@huaqin.corp-partner.google.com>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1767509
Reviewed-by: Edward Hill <ecgh@chromium.org>
Commit-Queue: Edward Hill <ecgh@chromium.org>
Tested-by: Edward Hill <ecgh@chromium.org>
Diffstat (limited to 'common')
-rw-r--r-- | common/i2c_master.c | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/common/i2c_master.c b/common/i2c_master.c index 5f84cc8e9f..5c9c8437c5 100644 --- a/common/i2c_master.c +++ b/common/i2c_master.c @@ -1034,19 +1034,19 @@ static int command_i2cxfer(int argc, char **argv) if (strcasecmp(argv[1], "r") == 0) { /* 8-bit read */ - if(offset_size == 2) - rv = i2c_read_offset16(port, slave_addr, offset, &v, 1); - else - rv = i2c_read8(port, slave_addr, offset, &v); + if (offset_size == 2) + rv = i2c_read_offset16(port, slave_addr, offset, &v, 1); + else + rv = i2c_read8(port, slave_addr, offset, &v); if (!rv) ccprintf("0x%02x [%d]\n", v, v); } else if (strcasecmp(argv[1], "r16") == 0) { /* 16-bit read */ - if(offset_size == 2) - rv = i2c_read_offset16(port, slave_addr, offset, &v, 2); - else - rv = i2c_read16(port, slave_addr, offset, &v); + if (offset_size == 2) + rv = i2c_read_offset16(port, slave_addr, offset, &v, 2); + else + rv = i2c_read16(port, slave_addr, offset, &v); if (!rv) ccprintf("0x%04x [%d]\n", v, v); @@ -1064,19 +1064,19 @@ static int command_i2cxfer(int argc, char **argv) /* 8-bit write */ if (argc < 6) return EC_ERROR_PARAM5; - if(offset_size == 2) - rv = i2c_write8(port, slave_addr, offset, v); - else - rv = i2c_write_offset16(port, slave_addr, offset, v, 1); + if (offset_size == 2) + rv = i2c_write_offset16(port, slave_addr, offset, v, 1); + else + rv = i2c_write8(port, slave_addr, offset, v); } else if (strcasecmp(argv[1], "w16") == 0) { /* 16-bit write */ if (argc < 6) return EC_ERROR_PARAM5; - if(offset_size == 2) - rv = i2c_write16(port, slave_addr, offset, v); - else - rv = i2c_write_offset16(port, slave_addr, offset, v, 2); + if (offset_size == 2) + rv = i2c_write_offset16(port, slave_addr, offset, v, 2); + else + rv = i2c_write16(port, slave_addr, offset, v); } else { return EC_ERROR_PARAM1; |