On 03/21/2014 03:41 AM, Penny Chiu wrote: ... > @@ -940,6 +941,12 @@ t124_bct_get_value(parse_token id, u_int32_t *data, u_int8_t *bct) > + case token_unique_chip_id: > + memcpy(data, > + &(bct_ptr->unique_chip_id), > + sizeof(nvboot_ecid)); > + break; The indentation there is wrong; parameters need to be indented at least one level more than the function they're being passed to, and I think you can wrap at least some of them one fewer lines. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html