[U-Boot] [PATCH 2/2] travis: Wire Xilinx Versal Virt platform

[Tom Rini]

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 at 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 at 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 at xilinx.com> wrote:
> > > >>>>>
> > > >>>>> Test Xilinx Versal Virt platform running on the v3.1.0 Qemu.
> > > >>>>>
> > > >>>>> Signed-off-by: Michal Simek <michal.simek at 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.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190103/37dd51c6/attachment.sig>