diff mbox

[V4] Target-arm: Add the Cortex-M4 CPU

Message ID 1434461850-4104-1-git-send-email-aurelioremonda@gmail.com
State New
Headers show

Commit Message

Aurelio C. Remonda June 16, 2015, 1:37 p.m. UTC
This patch adds the Cortex-M4 CPU. The M4 is basically the
same as the M3, the main differences being the DSP instructions and an
optional FPU. Created an ARM_FEATURE_THUMB_DSP to be added to any non-M
thumb2-compatible CPU that uses DSP instructions, and manually added it to
the M4 in its initfn. The THUMB_DSP feature was created in a different patch
along with the bit checking on each DSP instruction. Only no-FPU cortex-M4 is
implemented here, cortex-M4F is not because the core target-arm code doesn't
support the M-profile FPU model yet.

Signed-off-by: Aurelio C. Remonda <aurelioremonda@gmail.com>
---
* Changes in V4: Commit message updated. Deleted unnecessary comment and 
added a blank line in cortex-M4 initfn.
 target-arm/cpu.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Peter Maydell June 16, 2015, 2:20 p.m. UTC | #1
On 16 June 2015 at 14:37, Aurelio C. Remonda <aurelioremonda@gmail.com> wrote:
> This patch adds the Cortex-M4 CPU. The M4 is basically the
> same as the M3, the main differences being the DSP instructions and an
> optional FPU. Created an ARM_FEATURE_THUMB_DSP to be added to any non-M
> thumb2-compatible CPU that uses DSP instructions, and manually added it to
> the M4 in its initfn. The THUMB_DSP feature was created in a different patch
> along with the bit checking on each DSP instruction. Only no-FPU cortex-M4 is
> implemented here, cortex-M4F is not because the core target-arm code doesn't
> support the M-profile FPU model yet.
>
> Signed-off-by: Aurelio C. Remonda <aurelioremonda@gmail.com>

Applied to target-arm.next, thanks.

-- PMM
Aurelio C. Remonda June 16, 2015, 2:35 p.m. UTC | #2
Thank you!

2015-06-16 11:20 GMT-03:00 Peter Maydell <peter.maydell@linaro.org>:
> On 16 June 2015 at 14:37, Aurelio C. Remonda <aurelioremonda@gmail.com> wrote:
>> This patch adds the Cortex-M4 CPU. The M4 is basically the
>> same as the M3, the main differences being the DSP instructions and an
>> optional FPU. Created an ARM_FEATURE_THUMB_DSP to be added to any non-M
>> thumb2-compatible CPU that uses DSP instructions, and manually added it to
>> the M4 in its initfn. The THUMB_DSP feature was created in a different patch
>> along with the bit checking on each DSP instruction. Only no-FPU cortex-M4 is
>> implemented here, cortex-M4F is not because the core target-arm code doesn't
>> support the M-profile FPU model yet.
>>
>> Signed-off-by: Aurelio C. Remonda <aurelioremonda@gmail.com>
>
> Applied to target-arm.next, thanks.
>
> -- PMM
diff mbox

Patch

diff --git a/target-arm/cpu.c b/target-arm/cpu.c
index 4a888ab..d25a500 100644
--- a/target-arm/cpu.c
+++ b/target-arm/cpu.c
@@ -783,6 +783,15 @@  static void cortex_m3_initfn(Object *obj)
     cpu->midr = 0x410fc231;
 }
 
+static void cortex_m4_initfn(Object *obj)
+{
+    ARMCPU *cpu = ARM_CPU(obj);
+
+    set_feature(&cpu->env, ARM_FEATURE_V7);
+    set_feature(&cpu->env, ARM_FEATURE_M);
+    set_feature(&cpu->env, ARM_FEATURE_THUMB_DSP);
+    cpu->midr = 0x410fc240; /* r0p0 */
+}
 static void arm_v7m_class_init(ObjectClass *oc, void *data)
 {
     CPUClass *cc = CPU_CLASS(oc);
@@ -1185,6 +1194,8 @@  static const ARMCPUInfo arm_cpus[] = {
     { .name = "arm11mpcore", .initfn = arm11mpcore_initfn },
     { .name = "cortex-m3",   .initfn = cortex_m3_initfn,
                              .class_init = arm_v7m_class_init },
+    { .name = "cortex-m4",   .initfn = cortex_m4_initfn,
+                             .class_init = arm_v7m_class_init },
     { .name = "cortex-a8",   .initfn = cortex_a8_initfn },
     { .name = "cortex-a9",   .initfn = cortex_a9_initfn },
     { .name = "cortex-a15",  .initfn = cortex_a15_initfn },