All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <monstr@monstr.eu>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] test: py: Add an option to skip sleep test
Date: Fri, 8 Dec 2017 18:51:02 +0100	[thread overview]
Message-ID: <CAHTX3dJ6_Ma55r_UNLMK14zL+vL4bdbgh5RKT1gCKStAj9qiAw@mail.gmail.com> (raw)
In-Reply-To: <1f30c43b-b75e-9fce-8708-28f79ea26645@wwwdotorg.org>

2017-12-08 17:54 GMT+01:00 Stephen Warren <swarren@wwwdotorg.org>:

> On 12/08/2017 07:47 AM, 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.
>>
>
> Nit: Commit descriptions are typically wrapped around 70-74 characters;
> the above is very narrow.
>
> diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
>>
>
> +"""
>> +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
>> +
>> +"""
>>
>
> Nit: No need for that trailing empty line in the docstring.
>
> Reviewed-by: Stephen Warren <swarren@nvidia.com>
>
> (I was going to contemplate a more generic system implemented in
> conftest.py that could work with any test, but I guess this is simple
> enough and solves the only issue we have right now, so it's fine.)
>


I think also that generic solution would be better. I was checking xilinx
repo where we have some simple i2c
device tests which also needs to be run based on platforms. This test is
broken on qemu and some boards are also broken.
There is not an issue with travis because sleep tests are disabled now.
For my qemu testing it is not big deal and I can use -k not sleep or this
patch as temporary workaround till we have generic solution in place.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform

  reply	other threads:[~2017-12-08 17:51 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-08 14:47 [U-Boot] [PATCH v2] test: py: Add an option to skip sleep test Michal Simek
2017-12-08 14:48 ` Tom Rini
2017-12-08 16:54 ` Stephen Warren
2017-12-08 17:51   ` Michal Simek [this message]
2017-12-13 22:57 ` [U-Boot] [U-Boot, " Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAHTX3dJ6_Ma55r_UNLMK14zL+vL4bdbgh5RKT1gCKStAj9qiAw@mail.gmail.com \
    --to=monstr@monstr.eu \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.