From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Wed, 15 Nov 2017 10:34:07 +0100 Subject: [U-Boot] [PATCH] test/py: Add dependency on HUSH parser In-Reply-To: <81bb13dd-f8e4-bd96-93ff-78e371a07b30@wwwdotorg.org> References: <81bb13dd-f8e4-bd96-93ff-78e371a07b30@wwwdotorg.org> Message-ID: <836cf295-8166-5bf3-d074-d0e80a24e229@monstr.eu> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 -------------- next part -------------- A non-text attachment was scrubbed... Name: multiplexed_log.css Type: text/css Size: 1316 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: OpenPGP digital signature URL: