From: Michal Simek <michal.simek@xilinx.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2] spl: dm: Add SPL_DM_SEQ_ALIAS config option
Date: Fri, 11 Dec 2015 16:46:40 +0100 [thread overview]
Message-ID: <566AEFE0.5010708@xilinx.com> (raw)
In-Reply-To: <201512111607.56816.marex@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 <nathan@nathanrossi.com>
>> Cc: Simon Glass <sjg@chromium.org>
>> Cc: Masahiro Yamada <yamada.masahiro@socionext.com>
>> Cc: Linus Walleij <linus.walleij@linaro.org>
>> Cc: Marek Vasut <marex@denx.de>
>> Cc: Michal Simek <michal.simek@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.
That's probably not the best solution. But of course we can use it.
IRC Stefan had the same problem.
Thanks,
Michal
next prev parent reply other threads:[~2015-12-11 15:46 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-11 14:48 [U-Boot] [PATCH 0/2] Change DM_SEQ_ALIAS to be configurable for SPL Nathan Rossi
2015-12-11 14:48 ` [U-Boot] [PATCH 1/2] spl: dm: Add SPL_DM_SEQ_ALIAS config option Nathan Rossi
2015-12-11 15:07 ` Marek Vasut
2015-12-11 15:46 ` Michal Simek [this message]
2015-12-11 17:32 ` Marek Vasut
2015-12-12 12:05 ` Stefan Roese
2015-12-12 14:08 ` Nathan Rossi
2015-12-12 15:38 ` Marek Vasut
2015-12-14 6:26 ` Stefan Roese
2015-12-14 7:29 ` Michal Simek
2015-12-30 5:52 ` Nathan Rossi
2015-12-30 9:02 ` Marek Vasut
2015-12-11 15:47 ` Simon Glass
2015-12-11 14:48 ` [U-Boot] [PATCH 2/2] arm: zynq: Enable SPL_DM_SEQ_ALIAS for all Zynq configs Nathan Rossi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=566AEFE0.5010708@xilinx.com \
--to=michal.simek@xilinx.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox