Dear Joakim Tjernlund, In message <1287049904-18917-1-git-send-email-joakim.tjernl...@transmode.se> you wrote: > The fixup routine must not fixup NULL pointers. > Problem can be seen by > char *testfun(void) __attribute__((weak)); > char *(*myfun)(void) = testfun; > > Then add > printf("myfun:%p, &myfun:%p\n", myfun, &myfun); > before relocation and after relocation. > myfun should be NULL in both cases but it is not. > > Signed-off-by: Joakim Tjernlund <joakim.tjernl...@transmode.se> > --- > arch/powerpc/cpu/74xx_7xx/start.S | 4 +++- > arch/powerpc/cpu/mpc512x/start.S | 4 +++- > arch/powerpc/cpu/mpc5xx/start.S | 4 +++- > arch/powerpc/cpu/mpc5xxx/start.S | 4 +++- > arch/powerpc/cpu/mpc8220/start.S | 4 +++- > arch/powerpc/cpu/mpc824x/start.S | 4 +++- > arch/powerpc/cpu/mpc8260/start.S | 4 +++- > arch/powerpc/cpu/mpc83xx/start.S | 4 +++- > arch/powerpc/cpu/mpc85xx/start.S | 4 +++- > arch/powerpc/cpu/mpc86xx/start.S | 4 +++- > arch/powerpc/cpu/mpc8xx/start.S | 4 +++- > arch/powerpc/cpu/ppc4xx/start.S | 4 +++- > 12 files changed, 36 insertions(+), 12 deletions(-)
Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de There is an order of things in this universe. -- Apollo, "Who Mourns for Adonais?" stardate 3468.1 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot