On 20/8/25 10:25, Luc Michel wrote:
To align with current branding and ensure coherency with the upcoming
versal2 machine, rename the xlnx-versal-virt machine to amd-versal-virt.
Keep an alias of the old name to the new one for command-line backward
compatibility.

Signed-off-by: Luc Michel <luc.mic...@amd.com>
Reviewed-by: Francisco Iglesias <francisco.igles...@amd.com>
---
  docs/system/arm/xlnx-versal-virt.rst | 26 +++++++++++++++-----------
  hw/arm/xlnx-versal-virt.c            | 11 +++++++----
  2 files changed, 22 insertions(+), 15 deletions(-)


diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx-versal-virt.c
index 0f114ade652..4495b20ee1c 100644
--- a/hw/arm/xlnx-versal-virt.c
+++ b/hw/arm/xlnx-versal-virt.c
@@ -1,7 +1,7 @@
  /*
- * Xilinx Versal Virtual board.
+ * AMD/Xilinx Versal Virtual board.
   *
   * Copyright (c) 2018 Xilinx Inc.
   * Copyright (c) 2025 Advanced Micro Devices, Inc.
   * Written by Edgar E. Iglesias
   *
@@ -24,11 +24,11 @@
  #include "hw/arm/boot.h"
  #include "target/arm/multiprocessing.h"
  #include "qom/object.h"
  #include "target/arm/cpu.h"
-#define TYPE_XLNX_VERSAL_VIRT_MACHINE MACHINE_TYPE_NAME("xlnx-versal-virt")
+#define TYPE_XLNX_VERSAL_VIRT_MACHINE MACHINE_TYPE_NAME("amd-versal-virt")

While the alias works, can you also change the machine name in
tests/functional/test_aarch64_xlnx_versal.py in this commit for
clarity?

Reply via email to