On 20 April 2012 15:43, Andreas Färber <afaer...@suse.de> wrote: > Am 14.04.2012 18:42, schrieb Peter Maydell: >> static void pxa270c5_initfn(Object *obj) >> { >> ARMCPU *cpu = ARM_CPU(obj); >> + set_feature(&cpu->env, ARM_FEATURE_V7); >> + set_feature(&cpu->env, ARM_FEATURE_VFP4); >> + set_feature(&cpu->env, ARM_FEATURE_VFP_FP16); >> + set_feature(&cpu->env, ARM_FEATURE_NEON); >> + set_feature(&cpu->env, ARM_FEATURE_THUMB2EE); >> + set_feature(&cpu->env, ARM_FEATURE_ARM_DIV); >> + set_feature(&cpu->env, ARM_FEATURE_V7MP); >> cpu->midr = ARM_CPUID_PXA270_C5; >> } > > Beep! Glad I took the time to compare each model... :) > > Once this is fixed, Acked-by.
Oops. Fixed version: static void pxa270c5_initfn(Object *obj) { ARMCPU *cpu = ARM_CPU(obj); set_feature(&cpu->env, ARM_FEATURE_V5); set_feature(&cpu->env, ARM_FEATURE_XSCALE); set_feature(&cpu->env, ARM_FEATURE_IWMMXT); cpu->midr = ARM_CPUID_PXA270_C5; } -- PMM