[U-Boot] [PATCH v2 20/28] tpm: Check that parse_byte_string() has data to parse

Simon Glass sjg at chromium.org
Mon Aug 31 00:43:55 CEST 2015


On 25 August 2015 at 12:40, Christophe Ricard
<christophe.ricard at gmail.com> wrote:
> Hi Simon,
> Le 25/08/2015 06:13, Simon Glass a écrit :
>>
>> Hi Christophe,
>>
>> On 24 August 2015 at 14:22, Christophe Ricard
>> <christophe.ricard at gmail.com> wrote:
>>>
>>> Hi Simon,
>>>
>>> Print an error ? Are you sure ? I guess the comment is not accurate ;).
>>>
>> Well returning NULL from parse_byte_string() will cause an error to be
>> printed by the caller...
>
>
> This is just a comment nitpick. I think it is clear enough but from the
> patch the printing is may be not so obvious.
> It is just fine like that.

OK thanks. I fixed up the commit message to be a bit more explanatory.

Applied to u-boot-dm.


More information about the U-Boot mailing list