[U-Boot] test/py main_signon

[Stephen Warren]

On 02/16/2016 12:08 PM, Michal Simek wrote:
> Hi Stephen,
>
> 2016-02-16 17:39 GMT+01:00 Stephen Warren <swarren at wwwdotorg.org
> <mailto:swarren at wwwdotorg.org>>:
>
>     On 02/16/2016 09:04 AM, Michal Simek wrote:
>
>         Hi Heiko,
>
>         On 16.2.2016 14:32, Heiko Schocher wrote:
>
>             Hello Michal,
>
>             Am 16.02.2016 um 13:12 schrieb Michal Simek:
>
>                 Hi Stephen,
>
>                 trying to run the latest testing on zynq board and
>                 getting this
>                 main_signon error.
>
>                 This is what I am running
>                 ./test/py/test.py --bd zynq_zc702  --build
>                 --board-identity zc702
>                 and getting below.
>
>
>             Does this board has SPL support without SPL serial output?
>
>
>         I do load u-boot via jtag that's why SPL logs are not visible.
>
>             If so, can you try my patch:
>             http://patchwork.ozlabs.org/patch/583348/
>
>
>         I have applied your patch but it is still not working.
>
>         If I run full flow with SPL then I can't see any issue.
>
>
>     I assume this is resolved then?
>
>
> Unfortunately both cases should work because SPL is not only one first
> stage bootloader
> which can be used. I didn't test zynqmp but there is no SPL and the same
> problem is
> probably there too. Or is there any dependency that if SPL is not build
> than testing system
> is not expecting it?
>
> I will look tmr at jtag boot mode with SPL if I can get it work.

The test scripts currently assume that if SPL is enabled in the U-Boot 
configuration file, then U-Boot SPL will run and print its signon 
message to the serial console. If you're starting U-Boot using a 
technique that invalidates this assumption, it won't work currently, as 
you found.

It's pretty easy to add a flag in the boardenv file to tell the system 
not to expect SPL signon. If you take a look at my second suggestion in:

http://lists.denx.de/pipermail/u-boot/2016-February/245759.html
[U-Boot] [PATCH] test/py: only check for SPL signature if SPL uses 
serial output

... then it's probably a simple as adding a third variable into the if 
condition, where that variable gets looks up from the "board 
environment", similar to:

spl_skipped = u_boot_console.config.env.get('env__spl_skipped', False)