All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jamin Lin <jamin_lin@aspeedtech.com>
To: Thomas Huth <thuth@redhat.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
	"Beraldo Leal" <bleal@redhat.com>,
	"Troy Lee" <troy_lee@aspeedtech.com>,
	"Steven Lee" <steven_lee@aspeedtech.com>,
	"open list:All patches CC here" <qemu-devel@nongnu.org>,
	"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
	"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
	qemu-arm <qemu-arm@nongnu.org>, "Cédric Le Goater" <clg@kaod.org>,
	"Cleber Rosa" <crosa@redhat.com>
Subject: Re: [PATCH v1 1/1] test/avocado/machine_aspeed.py: Add ast1030 test case
Date: Tue, 22 Mar 2022 11:04:57 +0800	[thread overview]
Message-ID: <20220322030455.GA4273@aspeedtech.com> (raw)
In-Reply-To: <1f14533c-1163-f3c0-72ee-b79eba79480c@redhat.com>

The 03/21/2022 17:39, Thomas Huth wrote:
> On 21/03/2022 10.16, Jamin Lin wrote:
> > Add test case to test "ast1030-evb" machine with zephyr os
> > 
> > Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> > ---
> >   tests/avocado/machine_aspeed.py | 36 +++++++++++++++++++++++++++++++++
> >   1 file changed, 36 insertions(+)
> >   create mode 100644 tests/avocado/machine_aspeed.py
> > 
> > diff --git a/tests/avocado/machine_aspeed.py b/tests/avocado/machine_aspeed.py
> > new file mode 100644
> > index 0000000000..33090af199
> > --- /dev/null
> > +++ b/tests/avocado/machine_aspeed.py
> > @@ -0,0 +1,36 @@
> > +# Functional test that boots the ASPEED SoCs with firmware
> > +#
> > +# Copyright (C) 2022 ASPEED Technology Inc
> > +#
> > +# This work is licensed under the terms of the GNU GPL, version 2 or
> > +# later.  See the COPYING file in the top-level directory.
> > +
> > +from avocado_qemu import QemuSystemTest
> > +from avocado_qemu import wait_for_console_pattern
> > +from avocado_qemu import exec_command_and_wait_for_pattern
> > +from avocado.utils import archive
> > +
> > +
> > +class AST1030Machine(QemuSystemTest):
> > +    """Boots the zephyr os and checks that the console is operational"""
> > +
> > +    timeout = 10
> > +
> > +    def test_ast1030_zephyros(self):
> > +        """
> > +        :avocado: tags=arch:arm
> > +        :avocado: tags=machine:ast1030-evb
> > +        """
> > +        tar_url = ('https://github.com/AspeedTech-BMC'
> > +                   '/zephyr/releases/download/v00.01.04/ast1030-evb-demo.zip')
> > +        tar_hash = '4c6a8ce3a8ba76ef1a65dae419ae3409343c4b20'
> > +        tar_path = self.fetch_asset(tar_url, asset_hash=tar_hash)
> > +        archive.extract(tar_path, self.workdir)
> > +        kernel_file = self.workdir + "/ast1030-evb-demo/zephyr.elf"
> > +        self.vm.set_console()
> > +        self.vm.add_args('-kernel', kernel_file,
> > +                         '-nographic')
> > +        self.vm.launch()
> > +        wait_for_console_pattern(self, "Booting Zephyr OS")
> > +        exec_command_and_wait_for_pattern(self, "help",
> > +                                          "Available commands")
> Looks like this belongs to a new machine type that we do not have in QEMU 
> yet? ... in that case, please send it along in a threaded patch series where 
> you add the new machine type, otherwise it does not make much sense to see 
> this as a single patch on the list.
> 
>   Thomas
>
I created a new series patch to support AST1030 model which included
this patch for avocado test.
http://patchwork.ozlabs.org/project/qemu-devel/list/?series=291430
Thanks-Jamin

      reply	other threads:[~2022-03-22  3:06 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-21  9:16 [PATCH v1 0/1] tests/avocado: Add ast1030 test case Jamin Lin
2022-03-21  9:16 ` [PATCH v1 1/1] test/avocado/machine_aspeed.py: " Jamin Lin
2022-03-21 17:39   ` Thomas Huth
2022-03-22  3:04     ` Jamin Lin [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220322030455.GA4273@aspeedtech.com \
    --to=jamin_lin@aspeedtech.com \
    --cc=bleal@redhat.com \
    --cc=clg@kaod.org \
    --cc=crosa@redhat.com \
    --cc=f4bug@amsat.org \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=steven_lee@aspeedtech.com \
    --cc=thuth@redhat.com \
    --cc=troy_lee@aspeedtech.com \
    --cc=wainersm@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.