[U-Boot] [PATCH] bootvx_fdt: fix missing 'fdt_fixup_ethernet(...)' on vxWorks boot
Bin Meng
bmeng.cn at gmail.com
Fri Aug 25 09:21:55 UTC 2017
Hi Hannes,
On Fri, Aug 25, 2017 at 4:44 PM, Hannes Schmelzer <oe5hpm at oevsv.at> wrote:
> Before commit 26d6119 (fdt: Move fdt_fixup_ethernet to a common place)
> the fdt_fixup_ethernet(...) was called during do_bootvx_fdt(...).
>
Thanks for catching this!
> Afterwards the only (common) place for this fixup is during
> image_setup_libfdt(...) and this is only called, at least on ARM
> platform, from image_setup_linux(...).
>
> All this ends up in the fact, that the fdt_fixup_ethernet(...) is only
> called on booting a linux image and not on booting a vxWorks image.
>
> We fix this with adding the fdt_fixup_ethernet(...) call again to
> do_bootvx_fdt(...)
>
> Signed-off-by: Hannes Schmelzer <oe5hpm at oevsv.at>
>
> ---
>
> common/bootm_os.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/common/bootm_os.c b/common/bootm_os.c
> index 1feea8a..56b1c3c 100644
> --- a/common/bootm_os.c
> +++ b/common/bootm_os.c
> @@ -271,6 +271,8 @@ void do_bootvx_fdt(bootm_headers_t *images)
>
> if (*of_flat_tree) {
> boot_fdt_add_mem_rsv_regions(lmb, *of_flat_tree);
> + /* Update ethernet nodes */
> + fdt_fixup_ethernet(*of_flat_tree);
>
Can we insert this call after boot_relocate_fdt(), like the original
commit in b3b522f247d3f19c4f2309ab0e6a02b3dd82de45 did?
> ret = boot_relocate_fdt(lmb, of_flat_tree, &of_size);
> if (ret)
> --
Regards,
Bin
More information about the U-Boot
mailing list