From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Fri, 11 Dec 2015 16:46:40 +0100 Subject: [U-Boot] [PATCH 1/2] spl: dm: Add SPL_DM_SEQ_ALIAS config option In-Reply-To: <201512111607.56816.marex@denx.de> References: <1449845290-31114-1-git-send-email-nathan@nathanrossi.com> <1449845290-31114-2-git-send-email-nathan@nathanrossi.com> <201512111607.56816.marex@denx.de> Message-ID: <566AEFE0.5010708@xilinx.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 >> Cc: Simon Glass >> Cc: Masahiro Yamada >> Cc: Linus Walleij >> Cc: Marek Vasut >> Cc: Michal Simek >> --- >> 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. That's probably not the best solution. But of course we can use it. IRC Stefan had the same problem. Thanks, Michal