On 20 April 2012 15:43, Andreas Färber <[email protected]> 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
