Re: [Qemu-devel] [PATCH] Target-arm: Add the Cortex-M4 CPU

[Liviu Ionescu]

> On 29 May 2015, at 00:09, Aurelio C. Remonda <address@hidden> 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.
> The DSP instructions are already implemented in Qemu, as the A and R profiles
> use them.

great!

Peter mentioned some differences in exception processing, did you check if they 
require changes in emulation?

> The optional FPU in the M4 could be added in the future as a "Cortex-M4F" CPU.

in my implementation I had a single name ("cortex-m4") and some flags, but a 
separate name is probably better. can we reserve 

        { .name = "cortex-m4f", ... } 

for this purpose?

> All we'd have to do is add the ARM_FEATURE_VFP4 to the initfn.

if it is that simple, why don't we add it in for now?


regards,

Liviu