From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 3 Jan 2019 22:01:52 -0500 Subject: [U-Boot] [PATCH 2/2] travis: Wire Xilinx Versal Virt platform In-Reply-To: References: <4df8cbd1653d51c5acf3a73efefb4dbae48935fa.1545292515.git.michal.simek@xilinx.com> <4ce88f240cd4bf1a58ff8beec12a422a001973c0.1545292515.git.michal.simek@xilinx.com> <4c08ca0f-628f-0b21-192c-4b3a0f4beb4c@xilinx.com> <20181226142928.GV8702@bill-the-cat> <86155dd9-a23d-0d84-2daa-ed8bbbb263e5@xilinx.com> <20190103133907.GO16433@bill-the-cat> Message-ID: <20190104030152.GY16433@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, Jan 04, 2019 at 09:29:47AM +0800, Bin Meng wrote: > On Thu, Jan 3, 2019 at 9:39 PM Tom Rini 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 wrote: > > > >>> > > > >>> On 20. 12. 18 9:09, Bin Meng wrote: > > > >>>> On Thu, Dec 20, 2018 at 3:55 PM Michal Simek wrote: > > > >>>>> > > > >>>>> Test Xilinx Versal Virt platform running on the v3.1.0 Qemu. > > > >>>>> > > > >>>>> Signed-off-by: Michal Simek > > > >>>>> --- > > > >>>>> > > > >>>>> 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. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: