On 9/2/25 08:25, Jamin Lin wrote:
This patch moves the "ast2700-evb" alias from the A0 to A1.
The A0 machine remains available via its explicit name
("ast2700a0-evb"), while functional tests are updated to
target A0 by name instead of relying on the generic alias.

Signed-off-by: Jamin Lin <jamin_...@aspeedtech.com>
---
  hw/arm/aspeed.c                                 | 2 +-
  tests/functional/aarch64/test_aspeed_ast2700.py | 4 ++--
  2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
index c31bbe7701..d21b21965a 100644
--- a/hw/arm/aspeed.c
+++ b/hw/arm/aspeed.c
@@ -1995,7 +1995,6 @@ static void 
aspeed_machine_ast2700a0_evb_class_init(ObjectClass *oc,
      MachineClass *mc = MACHINE_CLASS(oc);
      AspeedMachineClass *amc = ASPEED_MACHINE_CLASS(oc);
- mc->alias = "ast2700-evb";
      mc->desc = "Aspeed AST2700 A0 EVB (Cortex-A35)";
      amc->soc_name  = "ast2700-a0";
      amc->hw_strap1 = AST2700_EVB_HW_STRAP1;
@@ -2018,6 +2017,7 @@ static void 
aspeed_machine_ast2700a1_evb_class_init(ObjectClass *oc,
      MachineClass *mc = MACHINE_CLASS(oc);
      AspeedMachineClass *amc = ASPEED_MACHINE_CLASS(oc);
+ mc->alias = "ast2700-evb";
      mc->desc = "Aspeed AST2700 A1 EVB (Cortex-A35)";
      amc->soc_name  = "ast2700-a1";
      amc->hw_strap1 = AST2700_EVB_HW_STRAP1;
diff --git a/tests/functional/aarch64/test_aspeed_ast2700.py 
b/tests/functional/aarch64/test_aspeed_ast2700.py
index d02dc7991c..e8e06701cf 100755
--- a/tests/functional/aarch64/test_aspeed_ast2700.py
+++ b/tests/functional/aarch64/test_aspeed_ast2700.py
@@ -111,8 +111,8 @@ def start_ast2700_test_vbootrom(self, name):
          self.do_test_aarch64_aspeed_sdk_start(
                  self.scratch_file(name, 'image-bmc'))
- def test_aarch64_ast2700_evb_sdk_v09_06(self):
-        self.set_machine('ast2700-evb')
+    def test_aarch64_ast2700a0_evb_sdk_v09_06(self):
+        self.set_machine('ast2700a0-evb')
self.archive_extract(self.ASSET_SDK_V906_AST2700)
          self.start_ast2700_test('ast2700-a0-default')


Reviewed-by: Cédric Le Goater <c...@redhat.com>

Thanks,

C.



Reply via email to