[U-Boot] test/py main_signon

Stephen Warren
2016-02-16T21:38:25Z

On 02/16/2016 12:08 PM, Michal Simek wrote:
> Hi Stephen,
>
> 2016-02-16 17:39 GMT+01:00 Stephen Warren <swarren@wwwdotorg.org
> <mailto:swarren@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)