diff options
author | Philippe Mathieu-Daudé <philmd@linaro.org> | 2024-01-29 16:18:24 +0100 |
---|---|---|
committer | Peter Maydell <peter.maydell@linaro.org> | 2024-02-02 13:51:58 +0000 |
commit | 04f4915424743671ca70c405ab69bc0a3ed8d937 (patch) | |
tree | f287a937c46ce22ef33565d87e5280da94b41b2d /hw/arm/msf2-soc.c | |
parent | 104395bc661b41055aaae1d8eb95bd2d99bdd20f (diff) | |
download | qemu-04f4915424743671ca70c405ab69bc0a3ed8d937.zip qemu-04f4915424743671ca70c405ab69bc0a3ed8d937.tar.gz qemu-04f4915424743671ca70c405ab69bc0a3ed8d937.tar.bz2 |
hw/arm/msf2: Simplify setting MachineClass::valid_cpu_types[]
The M2Sxxx SoC family can only be used with Cortex-M3.
Propagating the CPU type from the board level is pointless.
Hard-code the CPU type at the SoC level.
Remove the now ignored MachineClass::default_cpu_type field.
Use the common code introduced in commit c9cf636d48 ("machine: Add
a valid_cpu_types property") to check for valid CPU type at the
board level.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-id: 20240129151828.59544-6-philmd@linaro.org
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'hw/arm/msf2-soc.c')
-rw-r--r-- | hw/arm/msf2-soc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/hw/arm/msf2-soc.c b/hw/arm/msf2-soc.c index b5fe9f3..d6eb9ec 100644 --- a/hw/arm/msf2-soc.c +++ b/hw/arm/msf2-soc.c @@ -134,7 +134,7 @@ static void m2sxxx_soc_realize(DeviceState *dev_soc, Error **errp) armv7m = DEVICE(&s->armv7m); qdev_prop_set_uint32(armv7m, "num-irq", 81); - qdev_prop_set_string(armv7m, "cpu-type", s->cpu_type); + qdev_prop_set_string(armv7m, "cpu-type", ARM_CPU_TYPE_NAME("cortex-m3")); qdev_prop_set_bit(armv7m, "enable-bitband", true); qdev_connect_clock_in(armv7m, "cpuclk", s->m3clk); qdev_connect_clock_in(armv7m, "refclk", s->refclk); @@ -231,7 +231,6 @@ static Property m2sxxx_soc_properties[] = { * part name specifies the type of SmartFusion2 device variant(this * property is for information purpose only. */ - DEFINE_PROP_STRING("cpu-type", MSF2State, cpu_type), DEFINE_PROP_STRING("part-name", MSF2State, part_name), DEFINE_PROP_UINT64("eNVM-size", MSF2State, envm_size, MSF2_ENVM_MAX_SIZE), DEFINE_PROP_UINT64("eSRAM-size", MSF2State, esram_size, |