[U-Boot] [PATCH] spl: spl_nor: surround Linux-load code with #ifdef CONFIG_SPL_OS_BOOT

Masahiro YAMADA yamada.m at jp.panasonic.com
Wed Jan 7 20:20:36 CET 2015


ping?



2014-12-18 16:13 GMT+09:00 Masahiro Yamada <yamada.m at jp.panasonic.com>:
> Hi Tom,
>
> On Wed, 10 Dec 2014 20:34:03 -0500
> Tom Rini <trini at ti.com> wrote:
>
>> On Thu, Dec 11, 2014 at 10:01:38AM +0900, Masahiro Yamada wrote:
>>
>> > If CONFIG_SPL_NOR_SUPPORT is defined, spl_nor_load_image() requires
>> > spl_start_uboot(), CONFIG_SYS_OS_BASE, CONFIG_SYS_SPL_ARGS_ADDR,
>> > CONFIG_SYS_FDT_BASE to be defined even if users just want to run
>> > U-Boot, not Linux.  This is inconvenient.
>> >
>> > Signed-off-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
>>
>> Good idea, but the function to check on U-Boot or Linux should be called
>> spl_start_uboot to match the other load methods :)
>>
>
> I think I am following this way.
>
>
>
>> +#if defined(CONFIG_SPL_OS_BOOT)
>> +int load_linux(void)
>> +{
>> +     if (spl_start_uboot())
>> +             return -1;
>
>
> Here.
> Any problem?
>
>
>
> Best Regards
> Masahiro Yamada
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot



-- 
Best Regards
Masahiro Yamada


More information about the U-Boot mailing list