[U-Boot] [PATCH v2 2/8] fdt: add "fdt checksign" command

Simon Glass sjg at chromium.org
Sun Feb 16 00:07:04 CET 2014


Hi Heiko,

On 15 February 2014 16:00, Simon Glass <sjg at chromium.org> wrote:
> Hi Heiko,
>
> On 8 February 2014 22:34, Heiko Schocher <hs at denx.de> wrote:
>> check if a fdt is correct signed
>> pass an optional addr value. Contains the addr of the key blob
>>
>> Signed-off-by: Heiko Schocher <hs at denx.de>
>> Cc: Simon Glass <sjg at chromium.org>
>>
>> ---
>> changes vor v2:
>> - add comment from Simon Glass:
>>   - rename "fdt sign" to "fdt checksign"
>>     -> rename patch subject from "fdt: add "fdt sign" command"
>>        to "fdt: add "fdt checksign" command"
>>   - add error message
>>
>>  common/cmd_fdt.c | 42 +++++++++++++++++++++++++++++++++++++++++-
>>  1 file changed, 41 insertions(+), 1 deletion(-)
>>
>> diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
>> index 3a9edd6..a6744ed 100644
>> --- a/common/cmd_fdt.c
>> +++ b/common/cmd_fdt.c
>> @@ -570,7 +570,7 @@ static int do_fdt(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>>                 ft_board_setup(working_fdt, gd->bd);
>>  #endif
>>         /* Create a chosen node */
>> -       else if (argv[1][0] == 'c') {
>> +       else if (strncmp(argv[1], "cho", 3) == 0) {
>
> Sorry, I missed this earlier - I think this should be "che" if the
> command is checksig.

Hmm, ignore this, I was looking in the wrong place.

Regards,
Simon


More information about the U-Boot mailing list