On 09/17/2017 10:55 AM, Simon Glass wrote:
> Hi York,
>
> On 14 September 2017 at 13:01, York Sun <york....@nxp.com> wrote:
>> This partially reverts commit 15eb1d43bf470b85e9031c2fce7e0ce7b27dd321
>> which intended to move assignment of board info earlier, into
>> board_init_r(). However, function preload_console_init() is called
>> either from spl_board_init() or from board_init_f(). For the latter
>> case, the board info assignment is much earlier than board_init_r().
>> Moving such assignment to board_init_r() would be moving it later.
>>
>> Signed-off-by: York Sun <york....@nxp.com>
>> CC: Lokesh Vutla <lokeshvu...@ti.com>
>> CC: Ravi Babu <ravib...@ti.com>
>> CC: Lukasz Majewski <lu...@denx.de>
>> CC: Tom Rini <tr...@konsulko.com>
>>
>> ---
>>
>> Changes in v2:
>> New patch to fix spl after rebasing to latest master.
>>
>> common/spl/spl.c | 8 +++++++-
>> 1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/common/spl/spl.c b/common/spl/spl.c
>> index ce9819e..98b0ca0 100644
>> --- a/common/spl/spl.c
>> +++ b/common/spl/spl.c
>> @@ -365,7 +365,10 @@ void board_init_r(gd_t *dummy1, ulong dummy2)
>> struct spl_image_info spl_image;
>>
>> debug(">>spl:board_init_r()\n");
>> - gd->bd = &bdata;
>> +
>> + if (!gd->bd)
>> + gd->bd = &bdata;
>> +
>> #ifdef CONFIG_SPL_OS_BOOT
>> dram_init_banksize();
>> #endif
>> @@ -450,6 +453,9 @@ void board_init_r(gd_t *dummy1, ulong dummy2)
>> */
>> void preloader_console_init(void)
>> {
>> + if (!gd->bd)
>> + gd->bd = &bdata;
>> +
>
> It seems odd that enabling the console sets this data.
>
> What was the impact of moving it later for your board?
>
gd->bd is used to track the dram bank information. With this moved
later, I cannot track the secure memory reserved. I need this available
before memory is initialized.
York
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
