On 12.12.2015 15:08, Nathan Rossi wrote:
On Sat, Dec 12, 2015 at 10:05 PM, Stefan Roese <s...@denx.de> wrote:
On 11.12.2015 18:32, Marek Vasut wrote:
On Friday, December 11, 2015 at 04:46:40 PM, Michal Simek wrote:
On 11.12.2015 16:07, Marek Vasut wrote:
On Friday, December 11, 2015 at 03:48:09 PM, Nathan Rossi wrote:
The Device Model sequence alias feature is required by some Uclasses.
Instead of disabling the feature for all SPL targets allow it to be
configured.
The config option is disabled by default to reduce code size for
targets
that are not interested or do not require this feature.
Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
Cc: Simon Glass <s...@chromium.org>
Cc: Masahiro Yamada <yamada.masah...@socionext.com>
Cc: Linus Walleij <linus.wall...@linaro.org>
Cc: Marek Vasut <ma...@denx.de>
Cc: Michal Simek <michal.si...@xilinx.com>
---
Based on a small amount of inspection for the Zynq platform, enabling
this config option adds ~1KB of code size.
Also on a side note, this might affect the socfpga target as it
forcibly
overrides the #undef from config_uncmd_spl.h in its common header. I
have Cc'd the respective maintainer for this reason.
The fix for SoCFPGA is easy -- enable the SPL_DM_SEQ_ALIAS in
configs/socfpga*. It is needed for booting from QSPI NOR.
I will add a patch to this series, depending on how it is agreed to
enable this option.
That's probably not the best solution. But of course we can use it.
IRC Stefan had the same problem.
So would a better solution be to force the ARCH_ZYNQ config (and
others ARCH_SOCFPGA/ARCH_MVEBU) to 'select SPL_DM_SEQ_ALIAS' instead
of enabling them separately for each defconfig? like is done for
SPL_OF_CONTROL/etc.
Yes, please. Autoselecting this option is definitely better.
So what is the solution ?
I added
#define CONFIG_DM_SEQ_ALIAS 1
to the common config header for MVEBU. If its possible to set this
via Kconfig in a way where its not #undef'ed by config_uncmd_spl.h,
that would be even better.
As with socfpga, would you like me to do the same for MVEBU, add a
patch to this series to enable it for MVEBU?
Yes, please do so.
Thanks,
Stefan
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
