[coreboot-gerrit] Change in coreboot[master]: ec/google/chromeec: Introduce SKU_ID helpers

[Jett Rink (Code Review)]

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

Change subject: ec/google/chromeec: Introduce SKU_ID helpers
......................................................................

Patch Set 3:

(2 comments)

https://review.coreboot.org/c/coreboot/+/39018/3/src/ec/google/chromeec/ec_b... 
File src/ec/google/chromeec/ec_boardid.c:

https://review.coreboot.org/c/coreboot/+/39018/3/src/ec/google/chromeec/ec_b... 
PS3, Line 50: sku_str
Also in decimal form (%u) the value value could be

"sku4294967295" which needs 13 character plus a NULL (so 14 total)

https://review.coreboot.org/c/coreboot/+/39018/3/src/ec/google/chromeec/ec_b... 
PS3, Line 53: if ((sku_id == SKU_UNKNOWN) ||
            : 		(sku_id >  CONFIG_EC_GOOGLE_CHROMEEC_SKU_ID_MAX)) {
            : 		printk(BIOS_ERR, "%s: Unexpected SKU ID %u\n",
            : 			__func__, sku_id);
            : 		return "";
            : 	}
> +1 to just passing up the value to SMBIOS since the firmware should no longer care about this value. […]

Nevermind about the firmware configuration comment here...I still like just passing up the value verbatim though