summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMary Ruthven <mruthven@chromium.org>2021-11-09 19:53:19 -0600
committerCommit Bot <commit-bot@chromium.org>2021-11-18 04:53:17 +0000
commit1cfb5293ebbc854ce997d798cc55158031ccba6b (patch)
tree23d38030c932209f5acf4656907775fbd2f375ab
parenta9033feb7025646e29e02c20b9f20c02ddcb38e0 (diff)
downloadchrome-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.c16
-rw-r--r--include/config.h3
-rw-r--r--include/ec_commands.h8
-rw-r--r--include/system.h8
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.