[U-Boot] [PATCH 1/3] ARM: revive CONFIG_USE_ARCH_MEMCPY/MEMSET for UniPhier and Tegra
Tom Rini
trini at konsulko.com
Mon Dec 19 22:59:40 CET 2016
On Mon, Dec 19, 2016 at 07:31:02PM +0900, Masahiro Yamada wrote:
> Commit be72591bcd64 ("Kconfig: Move USE_ARCH_MEMCPY/MEMSET to
> Kconfig") is misconversion.
>
> The original logic in include/configs/uniphier.h was as follows:
>
> #if !defined(CONFIG_SPL_BUILD) && !defined(CONFIG_ARM64)
> #define CONFIG_USE_ARCH_MEMSET
> #define CONFIG_USE_ARCH_MEMCPY
> #endif
>
> This means those configs were enabled when building U-Boot proper,
> but disabled when building SPL. Likewise for Tegra.
>
> Now "depends on !SPL" prevents any boards with SPL support
> from reaching these options. This changed the behavior for
> UniPhier and Tegra SoC family.
>
> Please notice these two options only control the U-Boot proper
> build. As you see arch/arm/Makefile, ARM-specific memset/memcpy
> are never compiled for SPL. So, __HAVE_ARCH_MEMCPY/MEMSET should
> not set for SPL.
>
> Fixes: be72591bcd64 ("Kconfig: Move USE_ARCH_MEMCPY/MEMSET to Kconfig")
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
Ah, oops, thanks for spotting that one.
> ---
>
> I am restoring the original behavior for now.
> But, I have been wondering if we could remove these options entirely.
We cannot. That was my first attempt and we have a handful of active (I
checked) boards with tiny enough SPL constraints that switching to the
optimized memcpy/memset push them over size limit and they do not have a
"something" to disable to gain the space back. So I went with asking
for asking for a conversion to enable by default these options as widely
as possible as it's a good thing by and (no pun intended) large.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161219/ade17d95/attachment.sig>
More information about the U-Boot
mailing list