From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 21 Apr 2020 16:12:48 -0400 Subject: [PATCH 11/11] Nokia RX-51: Add automated test for running RX-51 build in qemu In-Reply-To: References: <20200331223518.10936-1-pali@kernel.org> <20200331223518.10936-12-pali@kernel.org> <20200414104002.rwm7ekt7pjkuk56z@pali> Message-ID: <20200421201248.GR4555@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 Tue, Apr 21, 2020 at 11:36:37AM -0600, Simon Glass wrote: > Hi, > > On Tue, 21 Apr 2020 at 08:56, Lokesh Vutla 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 > > > > > > 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: