Dear All, Am 30.11.2010 09:47, schrieb Joakim Tjernlund: >> >> Le 30/11/2010 08:06, Andreas Bießmann a écrit : >>> Signed-off-by: Andreas Bießmann<andreas.de...@googlemail.com> >> >>> + cmp r1, #0 /* symbol == NULL ? */ >>> + beq fixnext >> >> Nak. Don't hide a null pointer. NULL pointers are *not* relocated, since >> they are a constant. If a NULL ends up in relocation tables, that is >> because of a corruption *or* because it was to be relocated, and should >> thus never be ignored. > > Depends, if the same routine is used for relocating fixups you need > this test.
> Undefined weaks will generate a NULL fixup entry. As mentioned by Jens and one other some time ago. Albert, please build e.g. at91rm9200ek boards and search for board_reset(), defined in arch/arm/cpu/arm920t/at91/reset.c regards Andreas Bießmann _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
