[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Liviu Ionescu
Subject: Re: [Qemu-devel] [PATCH] Target-arm: Add the Cortex-M4 CPU
Date: Fri, 29 May 2015 22:15:00 +0300

> On 29 May 2015, at 15:55, aurelio remonda <address@hidden> wrote:
> 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.

ok, I already updated my code to use cortex-m4 or cortex-m4f.

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

this is fine with me.

unfortunately I cannot review your patch, since I have not enough experience 
with that part of qemu.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]