On 08/27/2012 10:29 AM, Daniel Schwierzeck wrote: > Hi Stefan, > > 2012/8/27 Stefan Roese <s...@denx.de>: >> SPL NOR flash booting support is quite simple. Only copying of the >> images is needed. >> >> On MPC5xxx we need to make sure to only use the standard memcpy() >> implementation and not the MPC5xxx specific one. As the MPC5xxx >> version has some complexity which is not needed for this SPL >> booting. >> >> Signed-off-by: Stefan Roese <s...@denx.de> [snip] >> +/* >> + * Don't use the special MPC5xxx memcpy implementation, only use >> + * the standard one. >> + */ >> +#if defined(CONFIG_MPC512X) || defined(CONFIG_MPC5200) >> +extern void *__memcpy(void *, const void *, size_t); >> +#define memcpy __memcpy >> +#endif > > can't we move this to arch-specifc spl code? > I think arch/CPU/SoC specific definitions should be avoided in new > generic frameworks.
This is a good point. And I think we can further tweak this by modifying arch/powerpc/lib/Makefile to just not build memcpy_5200.c for CONFIG_SPL_BUILD. So lets go with that, and I'll just fixup my local copy. -- Tom _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
