diff options
author | Peter Maydell <peter.maydell@linaro.org> | 2014-01-13 10:26:16 +0000 |
---|---|---|
committer | Edgar E. Iglesias <edgar.iglesias@xilinx.com> | 2014-01-14 10:09:04 +1000 |
commit | 83e6813a93e38976391b8c382c3375e3e188df3e (patch) | |
tree | 648f23820292d5e0fb8cf9142f56cd3c00569e1b /target-arm/cpu64.c | |
parent | b54f18ba3415c731f0b069f6df56f529997fb74e (diff) | |
download | hqemu-83e6813a93e38976391b8c382c3375e3e188df3e.zip hqemu-83e6813a93e38976391b8c382c3375e3e188df3e.tar.gz |
target-arm: Switch ARMCPUInfo arrays to use terminator entries
Switch the ARMCPUInfo arrays in cpu.c and cpu64.c to use a terminator
entry rather than looping based on ARRAY_SIZE. The latter causes
compile warnings on some versions of gcc if the configure options
happen to result in an empty array.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Tested-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Diffstat (limited to 'target-arm/cpu64.c')
-rw-r--r-- | target-arm/cpu64.c | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/target-arm/cpu64.c b/target-arm/cpu64.c index 60acd24..a639c2e 100644 --- a/target-arm/cpu64.c +++ b/target-arm/cpu64.c @@ -58,7 +58,7 @@ static const ARMCPUInfo aarch64_cpus[] = { #ifdef CONFIG_USER_ONLY { .name = "any", .initfn = aarch64_any_initfn }, #endif - { .name = NULL } /* TODO: drop when we support more CPUs */ + { .name = NULL } }; static void aarch64_cpu_initfn(Object *obj) @@ -101,11 +101,6 @@ static void aarch64_cpu_register(const ARMCPUInfo *info) .class_init = info->class_init, }; - /* TODO: drop when we support more CPUs - all entries will have name set */ - if (!info->name) { - return; - } - type_info.name = g_strdup_printf("%s-" TYPE_ARM_CPU, info->name); type_register(&type_info); g_free((void *)type_info.name); @@ -124,11 +119,13 @@ static const TypeInfo aarch64_cpu_type_info = { static void aarch64_cpu_register_types(void) { - int i; + const ARMCPUInfo *info = aarch64_cpus; type_register_static(&aarch64_cpu_type_info); - for (i = 0; i < ARRAY_SIZE(aarch64_cpus); i++) { - aarch64_cpu_register(&aarch64_cpus[i]); + + while (info->name) { + aarch64_cpu_register(info); + info++; } } |