diff options
author | Mary Ruthven <mruthven@chromium.org> | 2021-11-12 11:40:44 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-11-23 01:12:54 +0000 |
commit | 43fe43d36146df823d07a7bc159d5ecd0880550f (patch) | |
tree | 8cb0e342912079027b05dda4c1e8af533f3f0f15 | |
parent | 171d2142419cf7215aac67d22a0d56a3c0e04c49 (diff) | |
download | chrome-ec-43fe43d36146df823d07a7bc159d5ecd0880550f.tar.gz |
Revert "i2c: add i2clookup host command"
This reverts commit 45434aed20e695e08fcbb3f74c43e03f6fa19bf2.
BUG=b:200823466
TEST=make buildall -j
Change-Id: I353994692fd6b02ff1460234f448ac813cad04cb
Signed-off-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3285752
Reviewed-by: Vadim Sukhomlinov <sukhomlinov@chromium.org>
(cherry picked from commit a340a64dc9cc1ee0b8302ee4f9fa27375487bd94)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3296998
-rw-r--r-- | common/i2c_controller.c | 28 | ||||
-rw-r--r-- | include/ec_commands.h | 34 |
2 files changed, 0 insertions, 62 deletions
diff --git a/common/i2c_controller.c b/common/i2c_controller.c index 9c710e9f06..c12627c085 100644 --- a/common/i2c_controller.c +++ b/common/i2c_controller.c @@ -1054,34 +1054,6 @@ static enum ec_status i2c_command_passthru(struct host_cmd_handler_args *args) } DECLARE_HOST_COMMAND(EC_CMD_I2C_PASSTHRU, i2c_command_passthru, EC_VER_MASK(0)); -static enum ec_status i2c_command_lookup(struct host_cmd_handler_args *args) -{ - const struct ec_params_i2c_lookup *params = args->params; - struct ec_response_i2c_lookup *resp = args->response; - - switch (params->type) { - case I2C_LOOKUP_TYPE_CBI_EEPROM: -#ifdef CONFIG_CROS_BOARD_INFO - resp->i2c_port = I2C_PORT_EEPROM; - /* Convert from 8-bit address to 7-bit address */ - resp->i2c_addr = I2C_ADDR_EEPROM >> 1; -#else - /* Lookup type is supported, but not present on system. */ - return EC_RES_UNAVAILABLE; -#endif /* CONFIG_CROS_BOARD_INFO */ - break; - default: - /* The type was unrecognized */ - return EC_RES_INVALID_PARAM; - } - - args->response_size = sizeof(*resp); - return EC_RES_SUCCESS; -} -DECLARE_HOST_COMMAND(EC_CMD_I2C_LOOKUP, i2c_command_lookup, EC_VER_MASK(0)); -/* If the params union expands in the future, need to bump EC_VER_MASK */ -BUILD_ASSERT(sizeof(struct ec_params_i2c_lookup) == 4); - static enum ec_status i2c_command_passthru_protect(struct host_cmd_handler_args *args) { diff --git a/include/ec_commands.h b/include/ec_commands.h index e2c114d671..c3f86b97d5 100644 --- a/include/ec_commands.h +++ b/include/ec_commands.h @@ -5606,40 +5606,6 @@ struct ec_response_rollback_info { #define EC_CMD_AP_RESET 0x0125 /*****************************************************************************/ -/* I2C lookup command - * - * Return values: - * EC_RES_UNAVAILABLE: Lookup type is supported but not present on system. - * EC_RES_INVALID_PARAM: The type was unrecognized. - */ - -#define EC_CMD_I2C_LOOKUP 0x0126 - -struct ec_i2c_info { - uint16_t port; /* Physical port for device */ - uint16_t addr_flags; /* 7-bit (or 10-bit) address */ -}; - -enum i2c_device_type { - I2C_LOOKUP_TYPE_CBI_EEPROM = 1, - I2C_LOOKUP_TYPE_COUNT, - I2C_LOOKUP_TYPE_MAX = 0xFFFF, -}; - -struct ec_params_i2c_lookup { - uint16_t type; /* enum i2c_device_type */ - /* Used for type specific parameters in future */ - union { - uint16_t reseved; - }; -} __ec_align2; - -struct ec_response_i2c_lookup { - uint16_t i2c_port; /* Physical port for device */ - uint16_t i2c_addr; /* 7-bit (or 10-bit) address */ -} __ec_align1; - -/*****************************************************************************/ /* The command range 0x200-0x2FF is reserved for Rotor. */ /*****************************************************************************/ |