On 8.12.2017 15:47, Tom Rini wrote:
On Fri, Dec 08, 2017 at 03:44:58PM +0100, Michal Simek wrote:
On 8.12.2017 15:34, Tom Rini wrote:
On Fri, Dec 08, 2017 at 02:01:29PM +0100, Michal Simek wrote:
Some QEMUs have a problem with time setup that's why sleep test is failing. Introduce env__sleep_accurate boardenv variable to have an option to skip sleep test.
Signed-off-by: Michal Simek michal.simek@xilinx.com
Changes in v1:
- Change subject line and patch description
- Move from RFC to regular patch
- Remove boardidentity marker and use variable instead
configs/zynq_zc706_defconfig | 1 +
This looks unrelated.
yes it is sorry - I have tested it on local zc706 Will fix in v2.
test/py/tests/test_sleep.py | 13 +++++++++++++ 2 files changed, 14 insertions(+)
diff --git a/configs/zynq_zc706_defconfig b/configs/zynq_zc706_defconfig index 52999debc115..762b892f3fcf 100644 --- a/configs/zynq_zc706_defconfig +++ b/configs/zynq_zc706_defconfig @@ -37,6 +37,7 @@ CONFIG_CMD_EXT4=y CONFIG_CMD_EXT4_WRITE=y CONFIG_CMD_FAT=y CONFIG_CMD_FS_GENERIC=y +CONFIG_OF_EMBED=y CONFIG_ENV_IS_IN_SPI_FLASH=y CONFIG_NET_RANDOM_ETHADDR=y CONFIG_SPL_DM_SEQ_ALIAS=y diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py index 64e057132622..ccef24d7f7b8 100644 --- a/test/py/tests/test_sleep.py +++ b/test/py/tests/test_sleep.py @@ -5,10 +5,23 @@ import pytest import time
+""" +Note: This test doesn't rely on boardenv_* configuration values but they can +change test behavior.
+# Setup env__sleep_accurate to False if time is not accurate on your platform +env__sleep_accurate = False
This should default to true. Then a PR to uboot-test-hooks where you say the file for zynq_zc702 has this false, thanks!
This is just comment in that file. By default you can set it up to True or just not setup at all.
Ah, OK, I was reading too quickly. Thanks!
No worries. Will wait for Stephen and then will send PR to uboot-test-hooks and also I can remove sleep from .travis.yml.
Thanks, Michal