diff options
author | Mary Ruthven <mruthven@chromium.org> | 2021-11-09 19:53:19 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-11-18 04:53:17 +0000 |
commit | 1cfb5293ebbc854ce997d798cc55158031ccba6b (patch) | |
tree | 23d38030c932209f5acf4656907775fbd2f375ab | |
parent | a9033feb7025646e29e02c20b9f20c02ddcb38e0 (diff) | |
download | chrome-ec-1cfb5293ebbc854ce997d798cc55158031ccba6b.tar.gz |
Revert "coral: Add host command to fetch SKU ID"
This reverts commit f747f70816ca825a4ff4f9c6234f97e93611dbaa.
Nothing uses CONFIG_HOSTCMD_SKUID. Remove support.
BUG=b:200823466
TEST=make buildall -j
Change-Id: Iea545047659ab265025878c50986b74d1b3654b1
Signed-off-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3273455
Reviewed-by: Vadim Sukhomlinov <sukhomlinov@chromium.org>
-rw-r--r-- | common/system.c | 16 | ||||
-rw-r--r-- | include/config.h | 3 | ||||
-rw-r--r-- | include/ec_commands.h | 8 | ||||
-rw-r--r-- | include/system.h | 8 |
4 files changed, 0 insertions, 35 deletions
diff --git a/common/system.c b/common/system.c index 1d4586a8ce..a636813e2b 100644 --- a/common/system.c +++ b/common/system.c @@ -1161,22 +1161,6 @@ DECLARE_HOST_COMMAND(EC_CMD_GET_VERSION, host_command_get_version, EC_VER_MASK(0)); -#ifdef CONFIG_HOSTCMD_SKUID -static enum ec_status -host_command_get_sku_id(struct host_cmd_handler_args *args) -{ - struct ec_response_sku_id *r = args->response; - - r->sku_id = system_get_sku_id(); - args->response_size = sizeof(*r); - - return EC_RES_SUCCESS; -} -DECLARE_HOST_COMMAND(EC_CMD_GET_SKU_ID, - host_command_get_sku_id, - EC_VER_MASK(0)); -#endif - static enum ec_status host_command_build_info(struct host_cmd_handler_args *args) { diff --git a/include/config.h b/include/config.h index 377528992c..98331cb453 100644 --- a/include/config.h +++ b/include/config.h @@ -1985,9 +1985,6 @@ /* For access to VBNV on-EC battery-backed storage */ #undef CONFIG_HOSTCMD_VBNV_CONTEXT -/* EC controls the board's SKU ID and can report that to the AP */ -#undef CONFIG_HOSTCMD_SKUID - /* Command to issue AP reset */ #undef CONFIG_HOSTCMD_AP_RESET diff --git a/include/ec_commands.h b/include/ec_commands.h index b3f9716d6d..f89a3ca77d 100644 --- a/include/ec_commands.h +++ b/include/ec_commands.h @@ -1381,14 +1381,6 @@ struct ec_response_get_features { } __ec_align4; /*****************************************************************************/ -/* Get the board's SKU ID from EC */ -#define EC_CMD_GET_SKU_ID 0x000E - -struct ec_response_sku_id { - uint32_t sku_id; -} __ec_align4; - -/*****************************************************************************/ /* Flash commands */ /* Get flash info */ diff --git a/include/system.h b/include/system.h index 27f6810f02..98b580e19a 100644 --- a/include/system.h +++ b/include/system.h @@ -193,14 +193,6 @@ int32_t system_get_rollback_version(enum system_image_copy_t copy); const char *system_get_version(enum system_image_copy_t copy); /** - * Get the SKU ID for a device - * - * @return A value that identifies the SKU variant of a model. Its meaning and - * the number of bits actually used is opaque outside board specific code. - */ -uint32_t system_get_sku_id(void); - -/** * Return the board version number. The meaning of this number is * board-dependent; boards where the code actually cares about this should * declare enum board_version in board.h. |