On Fri, Jan 04, 2019 at 09:29:47AM +0800, Bin Meng wrote:
On Thu, Jan 3, 2019 at 9:39 PM Tom Rini trini@konsulko.com wrote:
On Thu, Jan 03, 2019 at 09:00:41AM +0100, Michal Simek wrote:
On 26. 12. 18 15:29, Tom Rini wrote:
On Thu, Dec 20, 2018 at 04:30:19PM +0800, Bin Meng wrote:
Hi Michal,
On Thu, Dec 20, 2018 at 4:17 PM Michal Simek michal.simek@xilinx.com wrote:
On 20. 12. 18 9:09, Bin Meng wrote: > On Thu, Dec 20, 2018 at 3:55 PM Michal Simek michal.simek@xilinx.com wrote: >> >> Test Xilinx Versal Virt platform running on the v3.1.0 Qemu. >> >> Signed-off-by: Michal Simek michal.simek@xilinx.com >> --- >> >> Patch needs to be applied when this PR is merged. >> https://github.com/swarren/uboot-test-hooks/pull/21 >> >> --- >> .travis.yml | 8 ++++++++ >> 1 file changed, 8 insertions(+) >> >> diff --git a/.travis.yml b/.travis.yml >> index 6e4b4ba0a34a..e14e6987e4cf 100644 >> --- a/.travis.yml >> +++ b/.travis.yml >> @@ -454,6 +454,14 @@ matrix: >> QEMU_TARGET="arm-softmmu" >> TEST_PY_ID="--id qemu" >> BUILDMAN="^zynq_zc702$" >> + - name: "test/py xilinx_versal_virt" >> + env: >> + - TEST_PY_BD="xilinx_versal_virt" >> + TEST_PY_TEST_SPEC="not sleep" >> + QEMU_TARGET="aarch64-softmmu" >> + QEMU_VERSION="v3.1.0" >> + TEST_PY_ID="--id qemu" >> + BUILDMAN="^xilinx_versal_virt$" > > Can we turn on 3.1.0 for all QEMU targets?
I expected this question. First of all I wanted to enable an option to be able to wire whatever version. Even sha1 for not released version.
I think we can try to test v3.1.0 qemu version for all boards but I will let Tom to decide is we can move or not.
Yes, please try to test that. If everything is good, I see no reason not using it.
Yes, I would also like to move everyone up to v3.1.0 and also having the version we use be spelled out will help with future reproducibility.
v3.1 is not working on vexpress_ca9x4 and vexpress_ca15_tc2. take a look at https://travis-ci.org/michalsimek/u-boot/builds/470873127 I am happy to keep them on v3.0 and move the rest to v3.1 and let vexpress guys to figured out what's wrong with v3.1 and configs around.
What do you think?
The error is: ---------------------------- Captured stdout setup ----------------------------- +u-boot-test-reset vexpress_ca15_tc2 qemu qemu-system-arm: -tftp: invalid option
And I guess the problem is that by v3.1.0 -tftp has been removed and we need to use -netdev user,id=net0,tftp=${UBOOT_TRAVIS_BUILD_DIR} like on the other QEMU-based targets. Can you update uboot-test-hooks (change .travis.yml to clone your fork of uboot-test-hooks rather than swarren's) to have the modern syntax? Thanks!
I think the correct approach is to update uboot-test-hooks and send PR to Stephen, then apply this 3.1.0 patch on the U-Boot side. We should still have U-Boot's .travis.yml point to swarren's git repo, no?
Yes, sorry, to be clear, I was just saying how to debug / confirm the changes to uboot-test-hooks.