On 03/03/2014 06:44 AM, Przemyslaw Marczak wrote: > Hello Stephen, > Thank you for review. > > On 02/28/2014 05:55 PM, Stephen Warren wrote: >> On 02/28/2014 08:18 AM, Przemyslaw Marczak wrote: >>> lib/uuid.c: >>> Add get_uuid_str() - this function returns 36 character hexadecimal >>> ASCII >>> string representation of a 128-bit (16 octets) UUID (Universally Unique >>> Identifier) version 4 based on RFC4122, which is randomly generated. >>> >>> Source: https://www.ietf.org/rfc/rfc4122.txt >> >>> diff --git a/disk/part_efi.c b/disk/part_efi.c >> >>> @@ -132,9 +113,11 @@ void print_part_efi(block_dev_desc_t * dev_desc) >>> le64_to_cpu(gpt_pte[i].ending_lba), >>> print_efiname(&gpt_pte[i])); >>> printf("\tattrs:\t0x%016llx\n", gpt_pte[i].attributes.raw); >>> - uuid_string(gpt_pte[i].partition_type_guid.b, uuid); >>> + uuid_bin = (unsigned char *)gpt_pte[i].partition_type_guid.b; >>> + uuid_bin_to_str(uuid_bin, uuid); >> >> I don't know why you need the uuid_bin temporary variable; you could >> just as well do the cast as part of the function parameter. Not a big >> deal though. > > Just because the line was too long.
You can wrap the parameter list. >>> +char *get_uuid_str(void) >> >> This function name isn't particularly good; it gives no hint that it's >> generating a random UUID. Perhaps generate_random_uuid_str() would be >> better. > > What about this? > > /* To generate bin uuid */ > void gen_rand_uuid(unsigned char *uuid) I don't think there's a need to shorten the words so much, but at least that's accurate. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot