[PATCH 11/11] Nokia RX-51: Add automated test for running RX-51 build in qemu

[Tom Rini]

On Tue, Apr 21, 2020 at 11:36:37AM -0600, Simon Glass wrote:
> Hi,
> 
> On Tue, 21 Apr 2020 at 08:56, Lokesh Vutla <lokeshvutla at ti.com> wrote:
> >
> > Tom,
> >
> > On 14/04/20 4:10 PM, Pali Rohár wrote:
> > > On Wednesday 01 April 2020 00:35:18 Pali Rohár wrote:
> > >> This patch contains a script which automatically download and compile all
> > >> needed tools to build a simple MTD images for booting Maemo kernel image by
> > >> U-Boot from RAM, eMMC and OneNAND. MTD images are then run in virtual n900
> > >> machine provided by qemu-linaro project.
> > >>
> > >> It can be used to check that U-Boot for Nokia N900 is not broken and can be
> > >> successfully booted in emulator.
> > >>
> > >> Script is registered in to .travis.yml so it would be automatically run on
> > >> Travi CI service.
> > >>
> > >> Signed-off-by: Pali Rohár <pali at kernel.org>
> > >
> > > Tom Rini, in past you have asked me for N900 Travis test. So could you
> > > please review this patch (including fixup at the bottom)?
> >
> > Can you ack this patch?
> 
> Please use a pytest for this (test/py). We don't use shell scripts anymore.

Well, this is where it's tricky and I've been debating with myself on
how to move forward here.

Part of the problem here is that much like a Pi, we could emulate this
board in QEMU but would need not-upstream-QEMU to do it.  But unlike Pi,
there's not a lot of these devices around to test with.  It's not a big
deal that Pi isn't tested by CI via QEMU, my lab as a Pi, Simon's lab
has a Pi and other labs could add one fairly easy.  But adding an N900
to a lab is hard.

Looking over the script to do it, there's a lot of other stuff required
too, for it all to work.  Looking over the script again, there's enough
stuff going on that I wouldn't want it done in a persistent
image/container.

The only changes I would ask for I guess are that it should be put in
.travis.yml in the same areas other non-pytest tests, and put in similar
stanzas in .azure-ci.yml and .gitlab-ci.yml.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200421/0a023340/attachment.sig>