From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Mon, 25 Aug 2014 10:09:13 -0600 Subject: [U-Boot] [PATCH v2] arm: tegra: initial support for apalis t30 In-Reply-To: <1408915084.3785.21.camel@localhost.localdomain> References: <4e5681daf84a08d160751c3da392f654e2b440ef.1408687805.git.marcel@ziswiler.com> <1f0fc67d6f08de8deba78762088fa61e2a195152.1408687805.git.marcel@ziswiler.com> <53F79A91.2060702@wwwdotorg.org> <1408915084.3785.21.camel@localhost.localdomain> Message-ID: <53FB5FA9.4050904@wwwdotorg.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 08/24/2014 03:18 PM, Marcel Ziswiler wrote: > On Fri, 2014-08-22 at 13:31 -0600, Stephen Warren wrote: >>> +#define CONFIG_USE_ARCH_MEMCPY >> >> That last option isn't set on other Tegra boards; what's it for? > > This is my special gift to Thierry to leverage architecture specific > memory copying routine which gives another few percent performance boost > on image download curtsey Stefan initially experimenting with it on > Vybrid (;-p). This sounds like, if we enable it, it should be part of one of include/configs/tegra*.h, not just one board-specific file. I think we should drop it from this patch, and add it to an appropriate common file in a separate patch.