Change in coreboot[master]: ec/google/chromeec: add support to get SSFC from CBI EEPROM

[Sugnan Prabhu S (Code Review)]

Sugnan Prabhu S has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45297 )

Change subject: ec/google/chromeec: add support to get SSFC from CBI EEPROM
......................................................................

ec/google/chromeec: add support to get SSFC from CBI EEPROM

This change adds a support function to get the SSFC data from CBI.

Change-Id: I52e2edc5b7fa6b5f82e88119355c99ff063a9a22
Signed-off-by: Sugnan Prabhu S 
---
M src/ec/google/chromeec/ec.c
M src/ec/google/chromeec/ec.h
M src/ec/google/chromeec/ec_commands.h
3 files changed, 7 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/97/45297/1

diff --git a/src/ec/google/chromeec/ec.c b/src/ec/google/chromeec/ec.c
index 40285dc..d3c4209 100644
--- a/src/ec/google/chromeec/ec.c
+++ b/src/ec/google/chromeec/ec.c
@@ -856,6 +856,11 @@
    return cbi_get_uint32(version, CBI_TAG_BOARD_VERSION);
 }

+int google_chromeec_cbi_get_ssfc(uint32_t *ssfc)
+{
+	return cbi_get_uint32(ssfc, CBI_TAG_SSFC);
+}
+
 static int cbi_get_string(char *buf, size_t bufsize, uint32_t tag)
 {
    struct ec_params_get_cbi params = {
diff --git a/src/ec/google/chromeec/ec.h b/src/ec/google/chromeec/ec.h
index 9d4e588..de63648 100644
--- a/src/ec/google/chromeec/ec.h
+++ b/src/ec/google/chromeec/ec.h
@@ -80,6 +80,7 @@
 /* version may be stored in CBI as a smaller integer width, but the EC code
    handles it correctly. */
 int google_chromeec_cbi_get_board_version(uint32_t *version);
+int google_chromeec_cbi_get_ssfc(uint32_t *ssfc);

#define CROS_SKU_UNKNOWN	0xFFFFFFFF
 #define CROS_SKU_UNPROVISIONED	0x7FFFFFFF
diff --git a/src/ec/google/chromeec/ec_commands.h b/src/ec/google/chromeec/ec_commands.h
index 62761a2..86204b8 100644
--- a/src/ec/google/chromeec/ec_commands.h
+++ b/src/ec/google/chromeec/ec_commands.h
@@ -5727,6 +5727,7 @@
    CBI_TAG_MODEL_ID = 5,      /* uint32_t or smaller */
    CBI_TAG_FW_CONFIG = 6,     /* uint32_t bit field */
    CBI_TAG_PCB_SUPPLIER = 7,  /* uint32_t or smaller */
+	CBI_TAG_SSFC = 8,          /* uint32_t or smaller */
    CBI_TAG_COUNT,
 };

[Sugnan Prabhu S (Code Review)]

Sugnan Prabhu S has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45297 )

Change subject: ec/google/chromeec: add support to get SSFC from CBI EEPROM
......................................................................

Patch Set 2:

This change is ready for review.

[Paul Fagerburg (Code Review)]

Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45297 )

Change subject: ec/google/chromeec: add support to get SSFC from CBI EEPROM
......................................................................

Patch Set 3: Code-Review+1

[Jett Rink (Code Review)]

Jett Rink has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45297 )

Change subject: ec/google/chromeec: add support to get SSFC from CBI EEPROM
......................................................................

Patch Set 3: Code-Review+1

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Sugnan Prabhu S.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45297 )

Change subject: ec/google/chromeec: add support to get SSFC from CBI EEPROM
......................................................................

Patch Set 3:

(1 comment)

File src/ec/google/chromeec/ec_commands.h:

https://review.coreboot.org/c/coreboot/+/45297/comment/1b0373a3_55977ce4 
PS3, Line 5714: 	CBI_TAG_SSFC = 8,          /* uint32_t or smaller */
This file should be synchronized from the Chromium EC repo. See CB:46403.

[Sugnan Prabhu S (Code Review)]

Sugnan Prabhu S has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/45297 )

Change subject: ec/google/chromeec: add support to get SSFC from CBI EEPROM
......................................................................

Abandoned

Handled by #48791