[U-Boot] test/py main_signon

Heiko Schocher
2016-02-17T09:05:14Z

Hello Michal,

Am 17.02.2016 um 09:42 schrieb Michal Simek:
> Hi Stephen,
>
> On 16.2.2016 22:38, Stephen Warren wrote:
>> 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)
>
> ok. I see. Anyway I have done this change locally and it is working good.
> Heiko: Are you able to add this one more checking to your patch? Or
> should I wait till you add your patch to the tree?

I just posted a v2 of my patch ... I vote, that you send your patch
based on mine to the ML.

bye,
Heiko
>
> Stephen: Do you have your u-boot-test-hooks for qemu?
>
> Thanks,
> Michal
>

--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany