CBI: Add MODEL_ID field

MODEL_ID is an optional field containing a numeric value identifying
models. Model IDs are unique within each OEM and <OEM_ID, MODEL_ID>
should form a unique ID within the family.

$ cbi-util create --file ~/cbi_image --board_version 0 --oem_id 6
--sku_id 255 --dram_part_num "012345679abcdef" --model_id 127 --size 256

$ hexdump -C /tmp/cbi.bin
0000  43 42 49 fa 00 00 26 00  00 01 00 01 01 06 02 01 |CBI...&.........|
0010  ff 05 01 7f 03 10 30 31  32 33 34 35 36 37 39 61 |......012345679a|
0020  62 63 64 65 66 00 ff ff  ff ff ff ff ff ff ff ff |bcdef...........|

$ cbi-util show --file /tmp/cbi.bin
CBI image: /tmp/cbi.bin
  TOTAL_SIZE: 38
  Data Field: name: value (hex, tag, size)
    BOARD_VERSION: 0 (0x0, 0, 1)
    OEM_ID: 6 (0x6, 1, 1)
    SKU_ID: 255 (0xff, 2, 1)
    MODEL_ID: 127 (0x7f, 5, 1)
    DRAM_PART_NUM: 012345679abcdef (3, 16)
Data validated successfully

localhost # ectool cbi set 5 127 1
localhost # ectool cbi get 5
As integer: 127 (0x7f)
As binary: 7f

> cbi
[1289.860454 CBI Reading board info]
CBI_VERSION: 0x0000
TOTAL_SIZE: 22
BOARD_VERSION: 513 (0x201)
OEM_ID: 3 (0x3)
MODEL_ID: 127 (0x7f)
SKU_ID: 14951 (0x3a67)
 43 42 49 b0 00 00 16 00 00 02 01 02 01 01 03 02
 02 67 3a 05 01 7f ff ff ff ff ff ff ff ff ff ff

Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>

BUG=b:119522898,b:120105950
BRANCH=none
TEST=See above.

Change-Id: Ifd6f3087f5422bcf4c36d3d981b262653d0c89dc
Reviewed-on: https://chromium-review.googlesource.com/1341099
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Furquan Shaikh <furquan@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>

1 files changed, 7 insertions, 0 deletions

diff --git a/common/cbi.c b/common/cbi.c
index f37d6950b1..8ec4472b24 100644
--- a/common/cbi.c
+++ b/common/cbi.c
@@ -253,6 +253,12 @@ int cbi_get_oem_id(uint32_t *id)
 	return cbi_get_board_info(CBI_TAG_OEM_ID, (uint8_t *)id, &size);
 }
 
+int cbi_get_model_id(uint32_t *id)
+{
+	uint8_t size = sizeof(*id);
+	return cbi_get_board_info(CBI_TAG_MODEL_ID, (uint8_t *)id, &size);
+}
+
 static int hc_cbi_get(struct host_cmd_handler_args *args)
 {
 	const struct __ec_align4 ec_params_get_cbi *p = args->params;
@@ -367,6 +373,7 @@ static void dump_cbi(void)
 
 	print_tag("BOARD_VERSION", cbi_get_board_version(&val), &val);
 	print_tag("OEM_ID", cbi_get_oem_id(&val), &val);
+	print_tag("MODEL_ID", cbi_get_model_id(&val), &val);
 	print_tag("SKU_ID", cbi_get_sku_id(&val), &val);
 }