All previous raspi machines can be created using the
generic machine. Deprecate the old names to maintain
a single one. Update the tests.
Signed-off-by: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
---
QOM HMP introspection test fails because without the 'model'
argument set, no machine is created...
$ qemu-system-aarch64 -M raspi
qemu-system-aarch64: Missing model, try -M raspi,model=help
---
docs/about/deprecated.rst | 13 +++++++++++++
hw/arm/raspi.c | 5 +++++
tests/qtest/bcm2835-dma-test.c | 2 +-
tests/qtest/bcm2835-i2c-test.c | 2 +-
tests/qtest/boot-serial-test.c | 3 ++-
tests/functional/test_aarch64_raspi3.py | 5 ++---
tests/functional/test_aarch64_raspi4.py | 4 ++--
tests/functional/test_arm_raspi2.py | 4 ++--
8 files changed, 28 insertions(+), 10 deletions(-)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 4a3c302962a..c9a11a52f78 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -257,6 +257,19 @@ Big-Endian variants of MicroBlaze ``petalogix-ml605`` and ``xlnx-zynqmp-pmu`` ma
Both ``petalogix-ml605`` and ``xlnx-zynqmp-pmu`` were added for little endian
CPUs. Big endian support is not tested.
+ARM ``raspi0``, ``raspi1ap``, ``raspi2b``, ``raspi3ap``, ``raspi3b`` and ``raspi4b`` machines (since 10.0)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+
+The Raspberry Pi machines have been unified under the generic ``raspi`` machine,
+which takes the model as argument.
+
+ - `raspi0`` is now an alias for ``raspi,model=Zero``
+ - `raspi1ap`` is now an alias for ``raspi,model=1A+``
+ - `raspi2b`` is now an alias for ``raspi,model=2B``
+ - `raspi3ap`` is now an alias for ``raspi,model=3A+``
+ - `raspi3b`` is now an alias for ``raspi,model=3B``
+ - `raspi4b`` is now an alias for ``raspi,model=4B``
+
Backend options
---------------
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
index 8cae1ff6f93..86ecc988e06 100644
--- a/hw/arm/raspi.c
+++ b/hw/arm/raspi.c
@@ -637,6 +637,7 @@ static void raspi0_machine_class_init(ObjectClass *oc, void *data)
rmc->board_rev = 0x920092; /* Revision 1.2 */
raspi_machine_class_init(mc, rmc->board_rev);
+ mc->deprecation_reason = "-M raspi,model=Zero";
};
static void raspi1ap_machine_class_init(ObjectClass *oc, void *data)
@@ -646,6 +647,7 @@ static void raspi1ap_machine_class_init(ObjectClass *oc, void *data)
rmc->board_rev = 0x900021; /* Revision 1.1 */
raspi_machine_class_init(mc, rmc->board_rev);
+ mc->deprecation_reason = "-M raspi,model=A+ (-m 512m)";
};
static void raspi2b_machine_class_init(ObjectClass *oc, void *data)
@@ -655,6 +657,7 @@ static void raspi2b_machine_class_init(ObjectClass *oc, void *data)
rmc->board_rev = 0xa21041;
raspi_machine_class_init(mc, rmc->board_rev);
+ mc->deprecation_reason = "-M raspi,model=2B -m 1g";
};
#ifdef TARGET_AARCH64
@@ -665,6 +668,7 @@ static void raspi3ap_machine_class_init(ObjectClass *oc, void *data)
rmc->board_rev = 0x9020e0; /* Revision 1.0 */
raspi_machine_class_init(mc, rmc->board_rev);
+ mc->deprecation_reason = "-M raspi,model=3A+ -m 512m";
};
static void raspi3b_machine_class_init(ObjectClass *oc, void *data)
@@ -674,6 +678,7 @@ static void raspi3b_machine_class_init(ObjectClass *oc, void *data)
rmc->board_rev = 0xa02082;
raspi_machine_class_init(mc, rmc->board_rev);
+ mc->deprecation_reason = "-M raspi,model=3B -m 1g";
};
static void raspi4b_machine_class_init(ObjectClass *oc, void *data)
diff --git a/tests/qtest/bcm2835-dma-test.c b/tests/qtest/bcm2835-dma-test.c
index 18901b76d21..705e6b2362b 100644
--- a/tests/qtest/bcm2835-dma-test.c
+++ b/tests/qtest/bcm2835-dma-test.c
@@ -111,7 +111,7 @@ int main(int argc, char **argv)
g_test_init(&argc, &argv, NULL);
qtest_add_func("/bcm2835/dma/test_interrupts",
bcm2835_dma_test_interrupts);
- qtest_start("-machine raspi3b");
+ qtest_start("-machine raspi,model=3B -m 1g");
ret = g_test_run();
qtest_end();
return ret;
diff --git a/tests/qtest/bcm2835-i2c-test.c b/tests/qtest/bcm2835-i2c-test.c
index 15991949260..15904abf393 100644
--- a/tests/qtest/bcm2835-i2c-test.c
+++ b/tests/qtest/bcm2835-i2c-test.c
@@ -104,7 +104,7 @@ int main(int argc, char **argv)
}
/* Run I2C tests with TMP105 slaves on all three buses */
- qtest_start("-M raspi3b "
+ qtest_start("-M raspi,model=3B -m 1g "
"-device tmp105,address=0x50,bus=i2c-bus.0 "
"-device tmp105,address=0x50,bus=i2c-bus.1 "
"-device tmp105,address=0x50,bus=i2c-bus.2");
diff --git a/tests/qtest/boot-serial-test.c b/tests/qtest/boot-serial-test.c
index a05d26ee996..fbafd73facb 100644
--- a/tests/qtest/boot-serial-test.c
+++ b/tests/qtest/boot-serial-test.c
@@ -188,7 +188,8 @@ static const testdef_t tests[] = {
sizeof(kernel_pls3adsp1800), kernel_pls3adsp1800 },
{ "microblazeel", "petalogix-ml605", "", "TT",
sizeof(kernel_plml605), kernel_plml605 },
- { "arm", "raspi2b", "", "TT", sizeof(bios_raspi2), 0, bios_raspi2 },
+ { "arm", "raspi,model=2B -m 1g", "", "TT",
+ sizeof(bios_raspi2), 0, bios_raspi2 },
{ "aarch64", "virt", "-cpu max", "TT", sizeof(kernel_aarch64),
kernel_aarch64 },
{ "arm", "microbit", "", "T", sizeof(kernel_nrf51), kernel_nrf51 },
diff --git a/tests/functional/test_aarch64_raspi3.py b/tests/functional/test_aarch64_raspi3.py
index 74f6630ed26..05766d93b3f 100755
--- a/tests/functional/test_aarch64_raspi3.py
+++ b/tests/functional/test_aarch64_raspi3.py
@@ -21,10 +21,9 @@ def test_aarch64_raspi3_atf(self):
efi_name = 'RPI_EFI.fd'
efi_fd = self.archive_extract(self.ASSET_RPI3_UEFI, member=efi_name)
- self.set_machine('raspi3b')
+ self.set_machine('raspi,model=3B -m 1g')
self.vm.set_console(console_index=1)
- self.vm.add_args('-cpu', 'cortex-a53',
- '-nodefaults',
+ self.vm.add_args('-nodefaults',
'-device', f'loader,file={efi_fd},force-raw=true')
self.vm.launch()
self.wait_for_console_pattern('version UEFI Firmware v1.15')
diff --git a/tests/functional/test_aarch64_raspi4.py b/tests/functional/test_aarch64_raspi4.py
index 7a4302b0c5a..3becee9333a 100755
--- a/tests/functional/test_aarch64_raspi4.py
+++ b/tests/functional/test_aarch64_raspi4.py
@@ -34,7 +34,7 @@ def test_arm_raspi4(self):
dtb_path = self.archive_extract(self.ASSET_KERNEL_20190215,
member='boot/bcm2711-rpi-4-b.dtb')
- self.set_machine('raspi4b')
+ self.set_machine('raspi,model=4B -m 1g')
self.vm.set_console()
kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
'earlycon=pl011,mmio32,0xfe201000 ' +
@@ -64,7 +64,7 @@ def test_arm_raspi4_initrd(self):
member='boot/bcm2711-rpi-4-b.dtb')
initrd_path = self.uncompress(self.ASSET_INITRD)
- self.set_machine('raspi4b')
+ self.set_machine('raspi,model=4B -m 1g')
self.vm.set_console()
kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
'earlycon=pl011,mmio32,0xfe201000 ' +
diff --git a/tests/functional/test_arm_raspi2.py b/tests/functional/test_arm_raspi2.py
index d3c7aaa39b0..37390a9c360 100755
--- a/tests/functional/test_arm_raspi2.py
+++ b/tests/functional/test_arm_raspi2.py
@@ -39,7 +39,7 @@ def do_test_arm_raspi2(self, uart_id):
dtb_path = self.archive_extract(self.ASSET_KERNEL_20190215,
member='boot/bcm2709-rpi-2-b.dtb')
- self.set_machine('raspi2b')
+ self.set_machine('raspi,model=2B -m 1g')
self.vm.set_console()
kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
serial_kernel_cmdline[uart_id] +
@@ -65,7 +65,7 @@ def test_arm_raspi2_initrd(self):
member='boot/bcm2709-rpi-2-b.dtb')
initrd_path = self.uncompress(self.ASSET_INITRD)
- self.set_machine('raspi2b')
+ self.set_machine('raspi,model=2B -m 1g')
self.vm.set_console()
kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
'earlycon=pl011,0x3f201000 console=ttyAMA0 '
--
2.47.1
