[U-Boot] [PATCH v2] bootvx_fdt: fix missing 'fdt_fixup_ethernet(...)' on vxWorks boot

[Bin Meng]

On Fri, Aug 25, 2017 at 5:32 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(...).
>
> 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>
>
> ---
>
> Changes in v2:
> - restore original behaviour from
>   b3b522f247d3f19c4f2309ab0e6a02b3dd82de45 with moving the
>   fdt_fixup_ethernet(...) call after boot_relocate_fdt(...)
>
>  common/bootm_os.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/common/bootm_os.c b/common/bootm_os.c
> index 1feea8a..934019f 100644
> --- a/common/bootm_os.c
> +++ b/common/bootm_os.c
> @@ -275,6 +275,8 @@ void do_bootvx_fdt(bootm_headers_t *images)
>                 ret = boot_relocate_fdt(lmb, of_flat_tree, &of_size);
>                 if (ret)
>                         return;

nits: please add one blank line here.

> +               /* Update ethernet nodes */
> +               fdt_fixup_ethernet(*of_flat_tree);
>
>                 ret = fdt_add_subnode(*of_flat_tree, 0, "chosen");
>                 if ((ret >= 0 || ret == -FDT_ERR_EXISTS)) {
> --

Reviewed-by: Bin Meng <bmeng.cn at gmail.com>