[U-Boot] [PATCH v2 3/6] FDT: only call boot_get_fdt from generic code
John Rigby
john.rigby at linaro.org
Tue Oct 12 23:33:56 CEST 2010
On Tue, Oct 12, 2010 at 3:24 PM, Wolfgang Denk <wd at denx.de> wrote:
> Dear John Rigby,
>
> In message <1285775292-15060-4-git-send-email-john.rigby at linaro.org> you wrote:
>> All arches except nios2 and microblaze call boot_get_fdt
>> from bootm_start in common/cmd_bootm.c.
>>
....
>> - /* find flattened device tree */
>> - ret = boot_get_fdt (flag, argc, argv, images, &of_flat_tree, &of_size);
>> - if (ret)
>> - return 1;
>> + /* did generic code already find a device tree? */
>> + if (images->ft_len)
>> + of_flat_tree = images->ft_addr;
>> #endif
>
> So what if the generic code did NOT find a device tree?
>
> Before, there was an error return. And now?
I'll fix this.
>> - /* find flattened device tree */
>> - if (boot_get_fdt(flag, argc, argv, images, &of_flat_tree, &of_size))
>> - return 1;
>> + /* did generic code already find a device tree? */
>> + if (images->ft_len)
>> + of_flat_tree = images->ft_addr;
>
> Ditto.
And this.
>
>
> On which nios2 and microblaze has this been tested?
It hasn't that is why the custodians were CC'd.
>
> You may want to put the NIOS and MB custodians on Cc:
I just checked my original message. They were CC'd unless I am very confused.
Thanks,
John
More information about the U-Boot
mailing list