From: Jamin Lin <jamin_lin@aspeedtech.com>
To: "Cédric Le Goater" <clg@kaod.org>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Steven Lee" <steven_lee@aspeedtech.com>,
"Troy Lee" <leetroy@gmail.com>,
"Kane Chen" <kane_chen@aspeedtech.com>,
"Andrew Jeffery" <andrew@codeconstruct.com.au>,
"Joel Stanley" <joel@jms.id.au>,
"open list:ASPEED BMCs" <qemu-arm@nongnu.org>,
"open list:All patches CC here" <qemu-devel@nongnu.org>
Cc: Jamin Lin <jamin_lin@aspeedtech.com>, Troy Lee <troy_lee@aspeedtech.com>
Subject: [PATCH v2 11/11] tests/functional/arm/test_aspeed_ast1030: Test the usbredir server transport
Date: Mon, 5 Oct 2026 06:44:32 +0000 [thread overview]
Message-ID: <20261005064418.3060881-12-jamin_lin@aspeedtech.com> (raw)
In-Reply-To: <20261005064418.3060881-1-jamin_lin@aspeedtech.com>
Export the AST1030 UDC gadget with usb-redir-server and import it on an
AST2600 with usb-redir, then check that the AST2600 enumerates it. That
covers the announce, control and enumeration paths of the transport, and
the UDC model behind it.
The two machines have to come from the same QEMU binary, because a
functional test gets only one. ast1030-evb and ast2600-evb are both in
qemu-system-arm.
Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
tests/functional/arm/meson.build | 1 +
tests/functional/arm/test_aspeed_ast1030.py | 52 +++++++++++++++++++++
2 files changed, 53 insertions(+)
diff --git a/tests/functional/arm/meson.build b/tests/functional/arm/meson.build
index 61b00932db..cfbd376b0d 100644
--- a/tests/functional/arm/meson.build
+++ b/tests/functional/arm/meson.build
@@ -1,6 +1,7 @@
# SPDX-License-Identifier: GPL-2.0-or-later
test_arm_timeouts = {
+ 'aspeed_ast1030' : 720,
'aspeed_palmetto' : 120,
'aspeed_romulus' : 120,
'aspeed_witherspoon' : 120,
diff --git a/tests/functional/arm/test_aspeed_ast1030.py b/tests/functional/arm/test_aspeed_ast1030.py
index 83a96ec322..d1d059c83e 100755
--- a/tests/functional/arm/test_aspeed_ast1030.py
+++ b/tests/functional/arm/test_aspeed_ast1030.py
@@ -6,6 +6,8 @@
#
# SPDX-License-Identifier: GPL-2.0-or-later
+import os
+
from aspeed import AspeedTest
from qemu_test import Asset, exec_command_and_wait_for_pattern
@@ -68,6 +70,56 @@ def test_arm_ast1030_zephyros_1_07(self):
'kernel uptime',
]: exec_command_and_wait_for_pattern(self, shell_cmd, "uart:~$")
+ ASSET_SDK_V1103_AST2600 = Asset(
+ 'https://github.com/AspeedTech-BMC/openbmc/releases/download/v11.03/ast2600-default-image.tar.gz',
+ '47e3656a14bf7a4de28d3dfbf48bc2325443bc42d270f3bc82646f92f6dea165')
+
+ def test_arm_ast1030_usbredir_to_ast2600(self):
+ self.require_device('usb-redir-server')
+ self.require_device('usb-redir')
+ self.set_machine('ast2600-evb')
+ self.set_machine('ast1030-evb')
+
+ ast1030_kernel_file = self.archive_extract(
+ self.ASSET_ZEPHYR_3_08, member="ast1030-evb-demo/zephyr.elf")
+
+ ast2600_image_file = self.archive_extract(
+ self.ASSET_SDK_V1103_AST2600,
+ member="ast2600-default-image/image-bmc")
+ sock = os.path.join(self.socket_dir().name, 'usbredir.sock')
+
+ udc = self.get_vm(name='udc')
+ udc.set_console()
+ udc.add_args('-kernel', ast1030_kernel_file, '-nographic',
+ '-chardev',
+ f'socket,id=usbredir0,path={sock},server=on,wait=off',
+ '-device',
+ 'usb-redir-server,id=udcredir,chardev=usbredir0',
+ '-device', 'aspeed.udc-gadget,bus=udcredir.0,'
+ 'udc=/machine/soc/udc')
+ udc.launch()
+ self.wait_for_console_pattern('Booting Zephyr OS', vm=udc)
+ exec_command_and_wait_for_pattern(self, 'usb enable', 'uart:~$',
+ vm=udc)
+
+ host = self.get_vm(name='host')
+ host.set_machine('ast2600-evb')
+ host.set_console()
+ host.add_args('-drive',
+ f'file={ast2600_image_file},if=mtd,format=raw',
+ '-snapshot',
+ '-chardev', f'socket,id=usbredir0,path={sock}',
+ '-device', 'usb-redir,chardev=usbredir0,bus=usb-bus.1')
+ host.launch()
+ self.wait_for_console_pattern('Starting kernel ...', vm=host)
+ self.wait_for_console_pattern('login:', vm=host)
+ exec_command_and_wait_for_pattern(self, 'root', 'Password:', vm=host)
+ exec_command_and_wait_for_pattern(self, '0penBmc',
+ 'root@ast2600-default:~#', vm=host)
+ exec_command_and_wait_for_pattern(self, 'lsusb',
+ 'ZEPHYR Zephyr DFU sample',
+ vm=host)
+
def test_arm_ast1030_otp_blockdev_device(self):
self.vm.set_machine("ast1030-evb")
--
2.43.0
prev parent reply other threads:[~2026-10-05 6:47 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-05 6:44 [PATCH v2 00/11] hw/usb/aspeed-udc: Add AST1030 UDC support Jamin Lin
2026-10-05 6:44 ` [PATCH v2 01/11] hw/usb/aspeed-udc: Make the SETUP buffer byte addressable Jamin Lin
2026-10-12 6:27 ` Cédric Le Goater
2026-10-05 6:44 ` [PATCH v2 02/11] hw/usb/aspeed-udc: Mask the reserved bits on register writes Jamin Lin
2026-10-12 6:27 ` Cédric Le Goater
2026-10-05 6:44 ` [PATCH v2 03/11] hw/usb/aspeed-udc: Clear EP_POOL_ACK when no endpoint ack is left Jamin Lin
2026-10-12 6:28 ` Cédric Le Goater
2026-10-05 6:44 ` [PATCH v2 04/11] hw/usb/aspeed-udc: Rename the transfer helpers after their DMA mode Jamin Lin
2026-10-12 6:28 ` Cédric Le Goater
2026-10-05 6:44 ` [PATCH v2 05/11] hw/usb/aspeed-udc: Store the EP_DMA_STS write first Jamin Lin
2026-10-12 6:28 ` Cédric Le Goater
2026-10-05 6:44 ` [PATCH v2 06/11] hw/usb/aspeed-udc: Pick the EP_DMA_STS path by direction Jamin Lin
2026-10-12 6:29 ` Cédric Le Goater
2026-10-05 6:44 ` [PATCH v2 07/11] hw/usb/aspeed-udc: Support single-stage IN transfers Jamin Lin
2026-10-05 6:44 ` [PATCH v2 08/11] hw/usb/aspeed-udc: Use the endpoint max packet size for OUT transfers Jamin Lin
2026-10-05 6:44 ` [PATCH v2 09/11] hw/usb/aspeed-udc: Add trace events for the transfer paths Jamin Lin
2026-10-12 6:29 ` Cédric Le Goater
2026-10-05 6:44 ` [PATCH v2 10/11] hw/arm/aspeed_ast10x0: Add the USB device controller Jamin Lin
2026-10-12 6:29 ` Cédric Le Goater
2026-10-05 6:44 ` 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=20261005064418.3060881-12-jamin_lin@aspeedtech.com \
--to=jamin_lin@aspeedtech.com \
--cc=andrew@codeconstruct.com.au \
--cc=clg@kaod.org \
--cc=joel@jms.id.au \
--cc=kane_chen@aspeedtech.com \
--cc=leetroy@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=steven_lee@aspeedtech.com \
--cc=troy_lee@aspeedtech.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.