diff options
author | Daisuke Nojiri <dnojiri@chromium.org> | 2017-06-23 17:41:50 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-06-28 21:50:50 -0700 |
commit | de36e33ecf8d7e376d36b68c4be0229d306006ad (patch) | |
tree | f9ae3e404cffeb93f30c93abcd35257923c92483 /cts/i2c/th.c | |
parent | 681b6fdf1f3ed049324403da3f6125b0cf21a2df (diff) | |
download | chrome-ec-de36e33ecf8d7e376d36b68c4be0229d306006ad.tar.gz |
eCTS: Add I2C suite to run_ects.py
This patch adds i2c suite to run_ects.py. It also adds checks for
return values from i2c_read and i2c_write functions.
BUG=chromium:653183
BRANCH=none
TEST=Run run_ects.py and verify all test pass.
Change-Id: Ie3d6e1e6f131235e2b28f39e1546c9cb4c3b92f6
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/547024
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'cts/i2c/th.c')
-rw-r--r-- | cts/i2c/th.c | 38 |
1 files changed, 19 insertions, 19 deletions
diff --git a/cts/i2c/th.c b/cts/i2c/th.c index 17ed64b9f5..78035cb1b2 100644 --- a/cts/i2c/th.c +++ b/cts/i2c/th.c @@ -27,18 +27,18 @@ void i2c_data_received(int port, uint8_t *buf, int len) int i2c_set_response(int port, uint8_t *buf, int len) { switch (buf[0]) { - case READ_8_OFFSET: - buf[0] = READ_8_DATA; + case READ8_OFF: + buf[0] = READ8_DATA; return 1; - case READ_16_OFFSET: - buf[0] = READ_16_DATA & 0xFF; - buf[1] = (READ_16_DATA >> 8) & 0xFF; + case READ16_OFF: + buf[0] = READ16_DATA & 0xFF; + buf[1] = (READ16_DATA >> 8) & 0xFF; return 2; - case READ_32_OFFSET: - buf[0] = READ_32_DATA & 0xFF; - buf[1] = (READ_32_DATA >> 8) & 0xFF; - buf[2] = (READ_32_DATA >> 16) & 0xFF; - buf[3] = (READ_32_DATA >> 24) & 0xFF; + case READ32_OFF: + buf[0] = READ32_DATA & 0xFF; + buf[1] = (READ32_DATA >> 8) & 0xFF; + buf[2] = (READ32_DATA >> 16) & 0xFF; + buf[3] = (READ32_DATA >> 24) & 0xFF; return 4; default: return 0; @@ -73,10 +73,10 @@ enum cts_rc write8_test(void) if (wait_for_in_flag(100)) return CTS_RC_TIMEOUT; - if (inbox[0] != WRITE_8_OFFSET) + if (inbox[0] != WRITE8_OFF) return CTS_RC_FAILURE; in = inbox[1]; - if (in != WRITE_8_DATA) + if (in != WRITE8_DATA) return CTS_RC_FAILURE; return CTS_RC_SUCCESS; @@ -88,10 +88,10 @@ enum cts_rc write16_test(void) if (wait_for_in_flag(100)) return CTS_RC_TIMEOUT; - if (inbox[0] != WRITE_16_OFFSET) + if (inbox[0] != WRITE16_OFF) return CTS_RC_FAILURE; in = inbox[2] << 8 | inbox[1] << 0; - if (in != WRITE_16_DATA) + if (in != WRITE16_DATA) return CTS_RC_FAILURE; return CTS_RC_SUCCESS; @@ -103,10 +103,10 @@ enum cts_rc write32_test(void) if (wait_for_in_flag(100)) return CTS_RC_TIMEOUT; - if (inbox[0] != WRITE_32_OFFSET) + if (inbox[0] != WRITE32_OFF) return CTS_RC_FAILURE; in = inbox[4] << 24 | inbox[3] << 16 | inbox[2] << 8 | inbox[1]; - if (in != WRITE_32_DATA) + if (in != WRITE32_DATA) return CTS_RC_FAILURE; return CTS_RC_SUCCESS; @@ -116,7 +116,7 @@ enum cts_rc read8_test(void) { if (wait_for_in_flag(100)) return CTS_RC_TIMEOUT; - if (inbox[0] != READ_8_OFFSET) + if (inbox[0] != READ8_OFF) return CTS_RC_FAILURE; return CTS_RC_SUCCESS; @@ -126,7 +126,7 @@ enum cts_rc read16_test(void) { if (wait_for_in_flag(100)) return CTS_RC_TIMEOUT; - if (inbox[0] != READ_16_OFFSET) + if (inbox[0] != READ16_OFF) return CTS_RC_FAILURE; return CTS_RC_SUCCESS; @@ -136,7 +136,7 @@ enum cts_rc read32_test(void) { if (wait_for_in_flag(100)) return CTS_RC_TIMEOUT; - if (inbox[0] != READ_32_OFFSET) + if (inbox[0] != READ32_OFF) return CTS_RC_FAILURE; return CTS_RC_SUCCESS; |