From c6ab2ffaa06182803e9df59060faa34032fdd62e Mon Sep 17 00:00:00 2001
From: Edward O'Callaghan <quasisec@google.com>
Date: Fri, 21 Feb 2020 14:58:43 +1100
Subject: mainboard/google/octopus: Migrate onto SKU ID helpers

Leverage the common sku id space helper encoders and
set the sku id max to 0xff for legacy to ensure we
behave the same.

BUG=b:149348474
BRANCH=none
TEST=tested on hatch

Change-Id: I60a37a5f9659b8df4018872956f95e07a3506440
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39035
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
---
 src/mainboard/google/octopus/Kconfig               |  1 +
 src/mainboard/google/octopus/Makefile.inc          |  1 -
 src/mainboard/google/octopus/mainboard_misc.c      | 29 +---------------------
 .../baseboard/include/baseboard/variants.h         |  2 --
 .../google/octopus/variants/bloog/variant.c        |  4 +--
 src/mainboard/google/octopus/variants/bobba/gpio.c |  3 ++-
 .../google/octopus/variants/bobba/variant.c        |  5 ++--
 .../google/octopus/variants/casta/variant.c        |  5 ++--
 src/mainboard/google/octopus/variants/dood/gpio.c  |  3 ++-
 .../google/octopus/variants/dood/variant.c         |  3 ++-
 src/mainboard/google/octopus/variants/foob/gpio.c  |  6 ++---
 .../google/octopus/variants/foob/variant.c         |  2 +-
 src/mainboard/google/octopus/variants/garg/gpio.c  |  3 ++-
 .../google/octopus/variants/garg/variant.c         |  4 +--
 src/mainboard/google/octopus/variants/meep/gpio.c  |  3 ++-
 .../google/octopus/variants/meep/variant.c         |  4 +--
 16 files changed, 27 insertions(+), 51 deletions(-)

(limited to 'src/mainboard')

diff --git a/src/mainboard/google/octopus/Kconfig b/src/mainboard/google/octopus/Kconfig
index a4e49824fc..e52e994433 100644
--- a/src/mainboard/google/octopus/Kconfig
+++ b/src/mainboard/google/octopus/Kconfig
@@ -13,6 +13,7 @@ config BOARD_GOOGLE_BASEBOARD_OCTOPUS
 	select DRIVERS_USB_ACPI
 	select EC_GOOGLE_CHROMEEC
 	select EC_GOOGLE_CHROMEEC_BOARDID
+	select EC_GOOGLE_CHROMEEC_SKUID
 	select EC_GOOGLE_CHROMEEC_ESPI
 	select HAVE_ACPI_RESUME
 	select HAVE_ACPI_TABLES
diff --git a/src/mainboard/google/octopus/Makefile.inc b/src/mainboard/google/octopus/Makefile.inc
index d36d5f7dbe..b8a7366f06 100644
--- a/src/mainboard/google/octopus/Makefile.inc
+++ b/src/mainboard/google/octopus/Makefile.inc
@@ -10,7 +10,6 @@ ramstage-y += mainboard.c
 
 verstage-$(CONFIG_CHROMEOS) += chromeos.c
 smm-y += smihandler.c
-smm-y += mainboard_misc.c
 
 subdirs-y += variants/baseboard
 CPPFLAGS_common += -I$(src)/mainboard/$(MAINBOARDDIR)/variants/baseboard/include
diff --git a/src/mainboard/google/octopus/mainboard_misc.c b/src/mainboard/google/octopus/mainboard_misc.c
index 3672f66692..8b281da9d0 100644
--- a/src/mainboard/google/octopus/mainboard_misc.c
+++ b/src/mainboard/google/octopus/mainboard_misc.c
@@ -21,34 +21,7 @@
 #include <smbios.h>
 #include <string.h>
 
-#define SKU_UNKNOWN		0xFFFFFFFF
-#define SKU_MAX			255
-
-uint32_t get_board_sku(void)
-{
-	static uint32_t sku_id = SKU_UNKNOWN;
-
-	if (sku_id != SKU_UNKNOWN)
-		return sku_id;
-
-	if (google_chromeec_cbi_get_sku_id(&sku_id))
-		sku_id = SKU_UNKNOWN;
-
-	return sku_id;
-}
-
 const char *smbios_system_sku(void)
 {
-	static char sku_str[7]; /* sku{0..255} */
-	uint32_t sku_id = get_board_sku();
-
-	if ((sku_id == SKU_UNKNOWN) || (sku_id > SKU_MAX)) {
-		printk(BIOS_ERR, "%s: Unexpected SKU ID %u\n",
-			__func__, sku_id);
-		return "";
-	}
-
-	snprintf(sku_str, sizeof(sku_str), "sku%u", sku_id);
-
-	return sku_str;
+	return google_chromeec_smbios_system_sku();
 }
diff --git a/src/mainboard/google/octopus/variants/baseboard/include/baseboard/variants.h b/src/mainboard/google/octopus/variants/baseboard/include/baseboard/variants.h
index 2132db591d..bf08a8588d 100644
--- a/src/mainboard/google/octopus/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/google/octopus/variants/baseboard/include/baseboard/variants.h
@@ -35,8 +35,6 @@ extern const struct lpddr4_swizzle_cfg baseboard_lpddr4_swizzle;
 const struct lpddr4_cfg *variant_lpddr4_config(void);
 /* Return memory SKU for the board. */
 size_t variant_memory_sku(void);
-/* Return board SKU */
-uint32_t get_board_sku(void);
 /* Return ChromeOS gpio table and fill in number of entries. */
 const struct cros_gpio *variant_cros_gpios(size_t *num);
 
diff --git a/src/mainboard/google/octopus/variants/bloog/variant.c b/src/mainboard/google/octopus/variants/bloog/variant.c
index 6c85e50948..05a1542d9a 100644
--- a/src/mainboard/google/octopus/variants/bloog/variant.c
+++ b/src/mainboard/google/octopus/variants/bloog/variant.c
@@ -42,7 +42,7 @@ enum {
 const char *get_wifi_sar_cbfs_filename(void)
 {
 	const char *filename = NULL;
-	uint32_t sku_id = get_board_sku();
+	uint32_t sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_33_BLOOG:
@@ -67,7 +67,7 @@ const char *mainboard_vbt_filename(void)
 {
 	uint32_t sku_id;
 
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_33_BLOOG:
diff --git a/src/mainboard/google/octopus/variants/bobba/gpio.c b/src/mainboard/google/octopus/variants/bobba/gpio.c
index 7c522c78ef..dd1084099a 100644
--- a/src/mainboard/google/octopus/variants/bobba/gpio.c
+++ b/src/mainboard/google/octopus/variants/bobba/gpio.c
@@ -18,6 +18,7 @@
 #include <boardid.h>
 #include <gpio.h>
 #include <soc/gpio.h>
+#include <ec/google/chromeec/ec.h>
 
 enum {
 	SKU_37_DROID = 37, /* LTE */
@@ -60,7 +61,7 @@ static const struct pad_config lte_override_table[] = {
 const struct pad_config *variant_override_gpio_table(size_t *num)
 {
 	uint32_t sku_id;
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_37_DROID:
diff --git a/src/mainboard/google/octopus/variants/bobba/variant.c b/src/mainboard/google/octopus/variants/bobba/variant.c
index 1f6e80db78..03eacbfe5d 100644
--- a/src/mainboard/google/octopus/variants/bobba/variant.c
+++ b/src/mainboard/google/octopus/variants/bobba/variant.c
@@ -19,6 +19,7 @@
 #include <baseboard/variants.h>
 #include <delay.h>
 #include <gpio.h>
+#include <ec/google/chromeec/ec.h>
 
 enum {
 	SKU_37_DROID = 37, /* LTE */
@@ -58,7 +59,7 @@ static void power_off_lte_module(u8 slp_typ)
 const char *get_wifi_sar_cbfs_filename(void)
 {
 	const char *filename = NULL;
-	uint32_t sku_id = get_board_sku();
+	uint32_t sku_id = google_chromeec_get_board_sku();
 
 	if (sku_id == 33 || sku_id == 34 || sku_id == 35 || sku_id == 36 || sku_id == 41 ||
 		sku_id == 42 || sku_id == 43 || sku_id == 44)
@@ -74,7 +75,7 @@ void variant_smi_sleep(u8 slp_typ)
 	if (slp_typ != ACPI_S5)
 		return;
 
-	switch (get_board_sku()) {
+	switch (google_chromeec_get_board_sku()) {
 	case SKU_37_DROID:
 	case SKU_38_DROID:
 	case SKU_39_DROID:
diff --git a/src/mainboard/google/octopus/variants/casta/variant.c b/src/mainboard/google/octopus/variants/casta/variant.c
index 12c8dd747b..4b1e42d9c3 100644
--- a/src/mainboard/google/octopus/variants/casta/variant.c
+++ b/src/mainboard/google/octopus/variants/casta/variant.c
@@ -13,6 +13,7 @@
  * GNU General Public License for more details.
  */
 
+#include <ec/google/chromeec/ec.h>
 #include <baseboard/variants.h>
 #include <sar.h>
 
@@ -21,7 +22,7 @@
 const char *get_wifi_sar_cbfs_filename(void)
 {
 	const char *filename = NULL;
-	uint32_t sku_id = get_board_sku();
+	uint32_t sku_id = google_chromeec_get_board_sku();
 
 	if (sku_id == 2)
 		filename = "wifi_sar-bluebird.hex";
@@ -31,7 +32,7 @@ const char *get_wifi_sar_cbfs_filename(void)
 
 bool variant_ext_usb_status(unsigned int port_type, unsigned int port_id)
 {
-	uint32_t sku_id = get_board_sku();
+	uint32_t sku_id = google_chromeec_get_board_sku();
 
 	if (sku_id == 2 && port_id == RIGHT_USB_C_PORT_ID)
 		return false;
diff --git a/src/mainboard/google/octopus/variants/dood/gpio.c b/src/mainboard/google/octopus/variants/dood/gpio.c
index 96b8ac02c7..3d09a39efd 100644
--- a/src/mainboard/google/octopus/variants/dood/gpio.c
+++ b/src/mainboard/google/octopus/variants/dood/gpio.c
@@ -18,6 +18,7 @@
 #include <boardid.h>
 #include <gpio.h>
 #include <soc/gpio.h>
+#include <ec/google/chromeec/ec.h>
 
 enum {
 	SKU_1_LTE  = 1, /* Wifi + LTE */
@@ -60,7 +61,7 @@ static const struct pad_config lte_override_table[] = {
 const struct pad_config *variant_override_gpio_table(size_t *num)
 {
 	uint32_t sku_id;
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_1_LTE:
diff --git a/src/mainboard/google/octopus/variants/dood/variant.c b/src/mainboard/google/octopus/variants/dood/variant.c
index 694e190892..b54b9fa084 100644
--- a/src/mainboard/google/octopus/variants/dood/variant.c
+++ b/src/mainboard/google/octopus/variants/dood/variant.c
@@ -19,6 +19,7 @@
 #include <baseboard/variants.h>
 #include <delay.h>
 #include <gpio.h>
+#include <ec/google/chromeec/ec.h>
 
 enum {
 	SKU_1_LTE  = 1, /* Wifi + LTE */
@@ -63,7 +64,7 @@ void variant_smi_sleep(u8 slp_typ)
 	if (slp_typ != ACPI_S5)
 		return;
 
-	switch (get_board_sku()) {
+	switch (google_chromeec_get_board_sku()) {
 	case SKU_1_LTE:
 	case SKU_3_LTE_2CAM:
 		power_off_lte_module(slp_typ);
diff --git a/src/mainboard/google/octopus/variants/foob/gpio.c b/src/mainboard/google/octopus/variants/foob/gpio.c
index dec2ff550d..55f8196dbc 100644
--- a/src/mainboard/google/octopus/variants/foob/gpio.c
+++ b/src/mainboard/google/octopus/variants/foob/gpio.c
@@ -20,8 +20,6 @@
 #include <soc/gpio.h>
 #include <ec/google/chromeec/ec.h>
 
-#define SKU_UNKNOWN 0xFFFFFFFF
-
 static const struct pad_config default_override_table[] = {
 	PAD_NC(GPIO_52, UP_20K),
 	PAD_NC(GPIO_53, UP_20K),
@@ -70,9 +68,9 @@ bool no_touchscreen_sku(uint32_t sku_id)
 const struct pad_config *variant_override_gpio_table(size_t *num)
 {
 	const struct pad_config *c;
-	uint32_t sku_id = SKU_UNKNOWN;
+	uint32_t sku_id;
 
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 	if (no_touchscreen_sku(sku_id)) {
 		c = non_touchscreen_override_table;
 		*num = ARRAY_SIZE(non_touchscreen_override_table);
diff --git a/src/mainboard/google/octopus/variants/foob/variant.c b/src/mainboard/google/octopus/variants/foob/variant.c
index dcc11dd0bb..47639f6345 100644
--- a/src/mainboard/google/octopus/variants/foob/variant.c
+++ b/src/mainboard/google/octopus/variants/foob/variant.c
@@ -30,7 +30,7 @@ void variant_update_devtree(struct device *dev)
 		return;
 
 	/* SKU ID 1 does not have a touchscreen device, hence disable it. */
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 	if (no_touchscreen_sku(sku_id))
 		touchscreen_i2c_host->enabled = 0;
 }
diff --git a/src/mainboard/google/octopus/variants/garg/gpio.c b/src/mainboard/google/octopus/variants/garg/gpio.c
index eeeb4662e3..987c69e7bd 100644
--- a/src/mainboard/google/octopus/variants/garg/gpio.c
+++ b/src/mainboard/google/octopus/variants/garg/gpio.c
@@ -19,6 +19,7 @@
 #include <gpio.h>
 #include <soc/gpio.h>
 #include <variant/sku.h>
+#include <ec/google/chromeec/ec.h>
 
 static const struct pad_config default_override_table[] = {
 	PAD_NC(GPIO_104, UP_20K),
@@ -72,7 +73,7 @@ static const struct pad_config lte_override_table[] = {
 const struct pad_config *variant_override_gpio_table(size_t *num)
 {
 	uint32_t sku_id;
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_9_HDMI:
diff --git a/src/mainboard/google/octopus/variants/garg/variant.c b/src/mainboard/google/octopus/variants/garg/variant.c
index f5f350a8f0..2afceb9b41 100644
--- a/src/mainboard/google/octopus/variants/garg/variant.c
+++ b/src/mainboard/google/octopus/variants/garg/variant.c
@@ -54,7 +54,7 @@ const char *mainboard_vbt_filename(void)
 {
 	uint32_t sku_id;
 
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_9_HDMI:
@@ -72,7 +72,7 @@ void variant_smi_sleep(u8 slp_typ)
 	if (slp_typ != ACPI_S5)
 		return;
 
-	switch (get_board_sku()) {
+	switch (google_chromeec_get_board_sku()) {
 	case SKU_17_LTE:
 	case SKU_18_LTE_TS:
 		power_off_lte_module(slp_typ);
diff --git a/src/mainboard/google/octopus/variants/meep/gpio.c b/src/mainboard/google/octopus/variants/meep/gpio.c
index 44d9fff129..ed4eb059bf 100644
--- a/src/mainboard/google/octopus/variants/meep/gpio.c
+++ b/src/mainboard/google/octopus/variants/meep/gpio.c
@@ -18,6 +18,7 @@
 #include <gpio.h>
 #include <soc/gpio.h>
 #include <variant/sku.h>
+#include <ec/google/chromeec/ec.h>
 
 static const struct pad_config default_override_table[] = {
 	PAD_NC(GPIO_104, UP_20K),
@@ -44,7 +45,7 @@ static const struct pad_config hdmi_sku_override_table[] = {
 const struct pad_config *variant_override_gpio_table(size_t *num)
 {
 	uint32_t sku_id;
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_33_DORP:
diff --git a/src/mainboard/google/octopus/variants/meep/variant.c b/src/mainboard/google/octopus/variants/meep/variant.c
index 20aaa0a1f4..7cd1e472bf 100644
--- a/src/mainboard/google/octopus/variants/meep/variant.c
+++ b/src/mainboard/google/octopus/variants/meep/variant.c
@@ -22,7 +22,7 @@
 const char *get_wifi_sar_cbfs_filename(void)
 {
 	const char *filename = NULL;
-	uint32_t sku_id = get_board_sku();
+	uint32_t sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_1_MEEP:
@@ -45,7 +45,7 @@ const char *mainboard_vbt_filename(void)
 {
 	uint32_t sku_id;
 
-	sku_id = get_board_sku();
+	sku_id = google_chromeec_get_board_sku();
 
 	switch (sku_id) {
 	case SKU_33_DORP:
-- 
cgit v1.2.3