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

[aurelio remonda]

2015-05-28 18:22 GMT-03:00 Liviu Ionescu <address@hidden>:
> > On 29 May 2015, at 00:09, Aurelio C. Remonda <address@hidden> wrote:
> > 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?

Thanks for the feedback! Yes, we could but I think its outside of the
scope of this particular contribution.

> > 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?

As Peter said, it may not actually be that simple.

Perhaps we could commit this now and add the VFP in the future?