On 03/26/2014 06:01 AM, Przemyslaw Marczak wrote:
> On 03/25/2014 08:51 PM, Stephen Warren wrote:
>> On 03/19/2014 11:58 AM, Przemyslaw Marczak wrote:
>>> Changes:
>>> - randomly generate partition uuid if any is undefined and
>>> CONFIG_RAND_UUID
>>> is defined
>>> - print debug info about set/unset/generated uuid
>>> - update doc/README.gpt
>>>
>>> Update existing code to the new library functions.
>>> diff --git a/common/cmd_gpt.c b/common/cmd_gpt.c
>>> - if ((strncmp(str, "${", 2) == 0) && (str[strlen(str) - 1] == '}')) {
>>> - s = strdup(str);
>>> - if (s == NULL)
>>> - return -1;
>>> - memset(s + strlen(s) - 1, '\0', 1);
>>> - memmove(s, s + 2, strlen(s) - 1);
>>> + if (!((strncmp(str, "${", 2) == 0) && (str[strlen(str) - 1] == '}')))
>>> + return -1;
>>
>> Since you're inverting that test, you need to change && to || too.
>>
> No, because the invertion refers to the result of "if" - not one of
> conditions.
> !(cond1 && cond2) is the same as:
> (!cond1 || !cond2)
> so this change is ok.
Ah yes, right.
>>> diff --git a/doc/README.gpt b/doc/README.gpt
>>> index 5c133f3..51515c8 100644
>>> --- a/doc/README.gpt
>>> +++ b/doc/README.gpt
>>> @@ -101,7 +101,7 @@ Offset Size Description
>>> 40 8 B First usable LBA for partitions (primary partition
>>> table last
>>> LBA + 1)
>>> 48 8 B Last usable LBA (secondary partition table first
>>> LBA - 1)
>>> -56 16 B Disk GUID (also referred as UUID on UNIXes)
>>> +56 16 B Disk GUID (also referred as UUID on UNIXes) in big
>>> endian
>>
>> According to your earlier comment, GUIDs have a mix of LE and BE fields,
>> so I would simply drop this change and the similar change below. Let
>> wikipedia or the comment you added near to top of lib/uuid.c specify the
>> details.
>>
> Actually I think that this is an important info here. The information
> about endianness is also placed in few places in lib/uuid.c
Why isn't the endianness of all the fields in this structure defined in
this comment then?
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
