Hi Jagdish, On Wed, Aug 22, 2018 at 2:18 AM, Jagdish Gediya <jagdish.ged...@nxp.com> wrote: > '__init_end' is defined in ld file. define 'board_fdt_blob_setup' > to setup the fdt blob at '&__init_end'. > > Signed-off-by: Jagdish Gediya <jagdish.ged...@nxp.com> > --- > arch/powerpc/cpu/mpc8xxx/fdt.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/arch/powerpc/cpu/mpc8xxx/fdt.c b/arch/powerpc/cpu/mpc8xxx/fdt.c > index 0d877c4..a7f61fe 100644 > --- a/arch/powerpc/cpu/mpc8xxx/fdt.c > +++ b/arch/powerpc/cpu/mpc8xxx/fdt.c > @@ -12,6 +12,7 @@ > #include <fdt_support.h> > #include <asm/mp.h> > #include <asm/fsl_serdes.h> > +#include <asm/sections.h> > #include <phy.h> > #include <hwconfig.h> > > @@ -19,6 +20,17 @@ > #define CONFIG_USB_MAX_CONTROLLER_COUNT 1 > #endif > > +#ifdef CONFIG_OF_SEPARATE > +void *board_fdt_blob_setup(void)
Instead of creating an mpc85xx-specific board_fdt_blob_setup(), can we modify the mpc85xx linker scripts to use common _end symbol instead of the __init_end symbol so that the generic board_fdt_blob_setup() can be used? > +{ > + void *fdt_blob = NULL; > + > + fdt_blob = (ulong *)&__init_end; > + > + return fdt_blob; > +} > +#endif > + > #if defined(CONFIG_MP) && (defined(CONFIG_MPC85xx) || > defined(CONFIG_MPC86xx)) > static int ft_del_cpuhandle(void *blob, int cpuhandle) > { > -- Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot