summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRandall Spangler <rspangler@chromium.org>2014-06-16 16:30:57 -0700
committerchrome-internal-fetch <chrome-internal-fetch@google.com>2014-06-17 12:15:57 +0000
commit88295aca5ac9bb9d13adf086bda66be09bd9f182 (patch)
tree763fa00109cc89309c07b5da50c5b1ddb48b60db
parent21aa3f7a48b0e573e662ad57b181494cf8beb57f (diff)
downloadchrome-ec-88295aca5ac9bb9d13adf086bda66be09bd9f182.tar.gz
i2c: remove (lack of) 10-bit support in i2c passthru
We never implemented this. We have no devices which support it. And we used bit #17 in a 16-bit field to flag it, so it wouldn't have worked even if we did. So, remove this (dead) code. BUG=chromium:382944 BRANCH=none TEST=make -j buildall Change-Id: Id3a4a93612d1078a3239d85921a05cfd7362b84c Signed-off-by: Randall Spangler <rspangler@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/204162 Reviewed-by: Doug Anderson <dianders@chromium.org>
-rw-r--r--common/i2c.c7
-rw-r--r--include/ec_commands.h3
2 files changed, 0 insertions, 10 deletions
diff --git a/common/i2c.c b/common/i2c.c
index fab2aefc67..82e36f6c25 100644
--- a/common/i2c.c
+++ b/common/i2c.c
@@ -443,13 +443,6 @@ static int check_i2c_params(const struct host_cmd_handler_args *args)
msgnum++, msg++) {
unsigned int addr_flags = msg->addr_flags;
- /* Parse slave address if necessary */
- if (addr_flags & EC_I2C_FLAG_10BIT) {
- /* 10-bit addressing not supported yet */
- PTHRUPRINTF("i2c passthru no 10-bit addressing");
- return EC_RES_INVALID_PARAM;
- }
-
PTHRUPRINTF("i2c passthru port=%d, %s, addr=0x%02x, "
"len=0x%02x",
params->port,
diff --git a/include/ec_commands.h b/include/ec_commands.h
index 54f0ed2aa7..55187cdb93 100644
--- a/include/ec_commands.h
+++ b/include/ec_commands.h
@@ -2046,9 +2046,6 @@ struct ec_response_power_info {
#define EC_CMD_I2C_PASSTHRU 0x9e
-/* Slave address is 10 (not 7) bit */
-#define EC_I2C_FLAG_10BIT (1 << 16)
-
/* Read data; if not present, message is a write */
#define EC_I2C_FLAG_READ (1 << 15)