Hi,

On 10.11.2017 22:34, Stephen Warren wrote:
> On 11/10/2017 04:01 AM, Michal Simek wrote:
>> After adding our small zynq uboot which has hush parser off these 3 tests
>> start to failed. It is probably just coincidence that others are
>> passing without hush parser.
> 
> What was the exact problem here? The set of tests you've disabled all
> seem to rely on setenv/printenv and don't seem to do anything
> complicated shell syntax wise. Are you sure they shouldn't depend on
> setenv being available rather than hush_parser?

These 3 tests. (html attached too.)

Zynq> printenv baudrate
baudrate=115200
Zynq>
test/py/tests/test_env.py .sZynq> printenv test_env_0
## Error: "test_env_0" not defined
Zynq> .sZynq> setenv test_env_0 "foo"
Zynq> printenv test_env_0
test_env_0="foo"
Zynq> F+u-boot-test-reset zynq_cse_qspi zc706



Zynq> Zynq> setenv test_env_0 "bar"
Zynq> printenv test_env_0
test_env_0="bar"
Zynq> Fs+u-boot-test-reset zynq_cse_qspi zc706

Zynq> Zynq> setenv test_env_1 " "
Zynq> setenv test_env_2 " 1${test_env_1}${test_env_1} 2 "
Zynq> printenv test_env_2
test_env_2=" 1" "" " 2 "
Zynq> setenv test_env_1
Zynq> setenv test_env_2
Zynq> F


Here is the branch with zynq_cse_qpsi configs which is minimum full
u-boot configuration running from on chip memory used for qspi programming.
http://git.denx.de/?p=u-boot/u-boot-microblaze.git;a=shortlog;h=refs/heads/for-stephen

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs

/*
 * Copyright (c) 2015 Stephen Warren
 * Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved.
 *
 * SPDX-License-Identifier: GPL-2.0
 */

/*
 * This provides pretty formatting of the HTML log file, e.g.
 * - colored bars beside/above log sections for easily parsed delineation.
 * - color highlighting of various messages.
 */

body {
    background-color: black;
    color: #ffffff;
}

pre {
    margin-top: 0px;
    margin-bottom: 0px;
}

.implicit {
    color: #808080;
}

.block {
    border-style: solid;
    border-color: #303030;
    border-width: 0px 0px 0px 5px;
    padding-left: 5px
}

.block-header {
    background-color: #303030;
    margin-left: -5px;
    margin-top: 5px;
}

.block-header:hover {
    text-decoration: underline;
}

.block-trailer {
    display: none;
}

.error {
    color: #ff0000
}

.warning {
    color: #ffff00
}

.info {
    color: #808080
}

.action {
    color: #8080ff
}

.status-pass {
    color: #00ff00
}

.status-skipped {
    color: #ffff00
}

.status-xfail {
    color: #ff7f00
}

.status-xpass {
    color: #ff7f00
}

.status-fail {
    color: #ff0000
}

.hidden {
    display: none;
}

a:link {
    text-decoration: inherit;
    color: inherit;
}

a:visited {
    text-decoration: inherit;
    color: inherit;
}

a:hover {
    text-decoration: underline;
}

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to