Add support for read secure_jtag_control and unique_chip_id from
cfg file and write them into BCT structure, and bct_dump can also
parse the two fields and show the data.

Signed-off-by: Penny Chiu <[email protected]>
---
 src/bct_dump.c           | 43 ++++++++++++++++++++++++-----
 src/cbootimage.h         |  2 ++
 src/parse.c              | 72 ++++++++++++++++++++++++++++++++++++++++++++++++
 src/parse.h              |  2 ++
 src/set.c                | 12 ++++++++
 src/t124/nvbctlib_t124.c | 13 ++++++++-
 6 files changed, 136 insertions(+), 8 deletions(-)

diff --git a/src/bct_dump.c b/src/bct_dump.c
index a8e3479..1af2c07 100644
--- a/src/bct_dump.c
+++ b/src/bct_dump.c
@@ -29,6 +29,7 @@ cbootimage_soc_config * g_soc_config;
 
 static void format_u32_hex8(char const * message, void * data);
 static void format_u32(char const * message, void * data);
+static void format_chipuid(char const * message, void * data);
 
 typedef void (*format_function)(char const * message, void * data);
 
@@ -44,6 +45,8 @@ static value_data const values[] = {
        { token_page_size_log2,      "PageSize      = ", format_u32_hex8 },
        { token_partition_size,      "PartitionSize = ", format_u32_hex8 },
        { token_odm_data,            "OdmData       = ", format_u32_hex8 },
+       { token_secure_jtag_control, "JtagCtrl      = ", format_u32_hex8 },
+       { token_unique_chip_id,      "ChipUid       = ", format_chipuid },
        { token_bootloader_used,     "# Bootloader used       = ", format_u32 },
        { token_bootloaders_max,     "# Bootloaders max       = ", format_u32 },
        { token_bct_size,            "# BCT size              = ", format_u32 },
@@ -74,6 +77,19 @@ static void format_u32(char const * message, void * data)
        printf("%s%d;\n", message, *((u_int32_t *) data));
 }
 
+static void format_chipuid(char const * message, void * data)
+{
+       u_int8_t *uid = (u_int8_t *)data;
+       int byte_index;
+       char uid_str[35] = "0x";
+       char *s = &uid_str[2];
+
+       for (byte_index = 15; byte_index >= 0; byte_index--, s += 2)
+               sprintf(s, "%02x", uid[byte_index]);
+
+       printf("%s%s;\n", message, uid_str);
+}
+
 /*****************************************************************************/
 static void usage(void)
 {
@@ -174,17 +190,30 @@ int main(int argc, char *argv[])
                if (!g_soc_config->token_supported(values[i].id))
                        continue;
 
-               e = g_soc_config->get_value(values[i].id,
+               if (values[i].id == token_unique_chip_id) {
+                       u_int8_t uid[16];
+
+                       e = g_soc_config->get_value(values[i].id,
+                                               uid,
+                                               context.bct);
+                       if (e != 0)
+                               for (j = 0; j < 16; j++)
+                                       uid[j] = -1;
+
+                       values[i].format(values[i].message, uid);
+               } else {
+                       e = g_soc_config->get_value(values[i].id,
                                                &data,
                                                context.bct);
 
-               if (e != 0)
-                       data = -1;
-               else if (values[i].id == token_block_size_log2 ||
-                        values[i].id == token_page_size_log2)
-                       data = 1 << data;
+                       if (e != 0)
+                               data = -1;
+                       else if (values[i].id == token_block_size_log2 ||
+                                values[i].id == token_page_size_log2)
+                               data = 1 << data;
 
-               values[i].format(values[i].message, &data);
+                       values[i].format(values[i].message, &data);
+               }
        }
 
        /* Display bootloader values */
diff --git a/src/cbootimage.h b/src/cbootimage.h
index 46e3b8b..252c41e 100644
--- a/src/cbootimage.h
+++ b/src/cbootimage.h
@@ -96,6 +96,8 @@ typedef struct build_image_context_rec
        u_int32_t boot_data_version; /* The boot data version of BCT */
        u_int8_t bct_init; /* The flag for the memory allocation of bct */
        u_int32_t odm_data; /* The odm data value */
+       u_int8_t unique_chip_id[16]; /* The unique chip uid */
+       u_int8_t secure_jtag_control; /* The flag for enabling jtag control */
 } build_image_context;
 
 /* Function prototypes */
diff --git a/src/parse.c b/src/parse.c
index f82c008..8e5daed 100644
--- a/src/parse.c
+++ b/src/parse.c
@@ -45,6 +45,7 @@ set_array(build_image_context *context,
                        u_int32_t value);
 static char *parse_u32(char *str, u_int32_t *val);
 static char *parse_u8(char *str, u_int32_t *val);
+static char *parse_chipuid(char *str, u_int8_t *val);
 static char *parse_filename(char *str, char *name, int chars_remaining);
 static char *parse_enum(build_image_context *context,
                        char *str,
@@ -64,6 +65,10 @@ parse_bootloader(build_image_context *context, parse_token 
token, char *rest);
 static int
 parse_value_u32(build_image_context *context, parse_token token, char *rest);
 static int
+parse_value_chipuid(build_image_context *context,
+                       parse_token token,
+                       char *rest);
+static int
 parse_bct_file(build_image_context *context, parse_token token, char *rest);
 static char
 *parse_end_state(char *str, char *uname, int chars_remaining);
@@ -102,6 +107,8 @@ static parse_item s_top_level_items[] = {
        { "Bctcopy=",       token_bct_copy,             parse_value_u32 },
        { "Version=",       token_version,              parse_value_u32 },
        { "OdmData=",       token_odm_data,             parse_value_u32 },
+       { "ChipUid=",       token_unique_chip_id,       parse_value_chipuid },
+       { "JtagCtrl=",      token_secure_jtag_control,  parse_value_u32 },
        { NULL, 0, NULL } /* Must be last */
 };
 
@@ -165,6 +172,46 @@ parse_u8(char *str, u_int32_t *val)
        return retval;
 }
 
+/*
+ * Parse the given string and transfer to chip uid.
+ *
+ * @param str          String to parse
+ * @param chipuid      Returns chip uid that was parsed
+ * @return the remainder of the string after the number was parsed
+ */
+static char *
+parse_chipuid(char *str, u_int8_t *chipuid)
+{
+       int byte_index = 0;
+       int paddings = 0;
+       char byte_str[3];
+
+       if (*str++ != '0')
+               return NULL;
+
+       if (*str++ != 'x')
+               return NULL;
+
+       paddings = strlen(str) % 2;
+       byte_index = strlen(str) / 2 + paddings;
+
+       while (*str != '\0' && byte_index > 0) {
+               char *endptr;
+
+               strncpy(byte_str, str, 2 - paddings);
+               byte_str[2-paddings] = '\0';
+               str += 2 - paddings;
+
+               chipuid[byte_index - 1] = strtoul(byte_str, &endptr, 16);
+               if (*endptr)
+                       return NULL;
+
+               byte_index--;
+               paddings = 0;
+       }
+
+       return str;
+}
 
 /*
  * Parse the given string and find the file name then
@@ -486,6 +533,31 @@ static int parse_value_u32(build_image_context *context,
 }
 
 /*
+ * General handler for setting chip uid in config files.
+ *
+ * @param context      The main context pointer
+ * @param token                The parse token value
+ * @param rest         String to parse
+ * @return 0 and 1 for success and failure
+ */
+static int parse_value_chipuid(build_image_context *context,
+                       parse_token token,
+                       char *rest)
+{
+       u_int8_t value[16];
+
+       assert(context != NULL);
+       assert(rest != NULL);
+
+       memset(value, 0, sizeof(value));
+       rest = parse_chipuid(rest, value);
+       if (rest == NULL)
+               return 1;
+
+       return context_set_value(context, token, value);
+}
+
+/*
  * Parse the given string and find the bct file name.
  *
  * @param context      The main context pointer
diff --git a/src/parse.h b/src/parse.h
index d9d873f..114168c 100644
--- a/src/parse.h
+++ b/src/parse.h
@@ -108,6 +108,8 @@ typedef enum
        token_dev_type_spi,
        token_num_sdram_sets,
        token_pre_bct_pad_blocks,
+       token_unique_chip_id,
+       token_secure_jtag_control,
 
        token_nand_clock_divider,
        token_nand_nand_timing,
diff --git a/src/set.c b/src/set.c
index 3d62254..4a78b71 100644
--- a/src/set.c
+++ b/src/set.c
@@ -199,6 +199,18 @@ int context_set_value(build_image_context *context,
                context->pre_bct_pad_blocks = *((u_int32_t *)value);
                break;
 
+       case token_secure_jtag_control:
+               context->secure_jtag_control = *((u_int32_t *)value);
+               g_soc_config->set_value(token_secure_jtag_control,
+                       value, context->bct);
+               break;
+
+       case token_unique_chip_id:
+               memcpy(context->unique_chip_id, value, 16);
+               g_soc_config->set_value(token_unique_chip_id,
+                       value, context->bct);
+               break;
+
        DEFAULT();
        }
 
diff --git a/src/t124/nvbctlib_t124.c b/src/t124/nvbctlib_t124.c
index b1ec445..b103ea7 100644
--- a/src/t124/nvbctlib_t124.c
+++ b/src/t124/nvbctlib_t124.c
@@ -111,7 +111,9 @@ parse_token t124_root_token_list[] = {
        token_hash_size,
        token_crypto_offset,
        token_crypto_length,
-       token_max_bct_search_blks
+       token_max_bct_search_blks,
+       token_unique_chip_id,
+       token_secure_jtag_control
 };
 
 int
@@ -941,6 +943,7 @@ t124_bct_get_value(parse_token id, void *data, u_int8_t 
*bct)
        CASE_GET_NVU32(num_sdram_sets);
        CASE_GET_NVU32(bootloader_used);
        CASE_GET_NVU32(odm_data);
+       CASE_GET_NVU32(secure_jtag_control);
 
        /*
         * Constants.
@@ -955,6 +958,10 @@ t124_bct_get_value(parse_token id, void *data, u_int8_t 
*bct)
                sizeof(nvboot_hash));
                break;
 
+       case token_unique_chip_id:
+               memcpy(data, &(bct_ptr->unique_chip_id), sizeof(nvboot_ecid));
+               break;
+
        case token_reserved_offset:
                *((u_int32_t *)data) = (u_int8_t *)&(samplebct.reserved)
                                - (u_int8_t *)&samplebct;
@@ -1020,6 +1027,10 @@ t124_bct_set_value(parse_token id, void *data, u_int8_t 
*bct)
        CASE_SET_NVU32(num_sdram_sets);
        CASE_SET_NVU32(bootloader_used);
        CASE_SET_NVU32(odm_data);
+       CASE_SET_NVU32(secure_jtag_control);
+       case token_unique_chip_id:
+               memcpy(&bct_ptr->unique_chip_id, data, sizeof(nvboot_ecid));
+               break;
 
        default:
                return -ENODATA;
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to