From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Tue, 28 Aug 2012 10:32:17 +0200
Subject: [U-Boot] [PATCH v2 04/12] SPL: Add NOR flash booting support
In-Reply-To: <503BB58D.2080508@ti.com>
References: <1346064667-29692-1-git-send-email-sr@denx.de>
	<1346064667-29692-5-git-send-email-sr@denx.de>
	<CACUy__UsTybbi7qMm=nygPNdyK1AivzNMu7VVLEc1-=2=E68OA@mail.gmail.com>
	<503BB58D.2080508@ti.com>
Message-ID: <503C8211.9000907@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 08/27/2012 07:59 PM, Tom Rini wrote:
> On 08/27/2012 10:29 AM, Daniel Schwierzeck wrote:
>> Hi Stefan,
>>
>> 2012/8/27 Stefan Roese <sr@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 <sr@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.

Good idea. I'll change the powerpc Makefile accordingly.

Thanks,
Stefan