On Mon, Sep 11, 2017 at 4:56 AM, KONRAD Frederic
<frederic.kon...@adacore.com> wrote:
>
>
> On 09/06/2017 02:12 AM, Alistair Francis wrote:
>>
>> Signed-off-by: Alistair Francis <alistair.fran...@xilinx.com>
>> ---
>>
>>   hw/arm/netduino2.c | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/hw/arm/netduino2.c b/hw/arm/netduino2.c
>> index 3cfe332dd1..accca344fd 100644
>> --- a/hw/arm/netduino2.c
>> +++ b/hw/arm/netduino2.c
>> @@ -43,8 +43,13 @@ static void netduino2_init(MachineState *machine)
>>     static void netduino2_machine_init(MachineClass *mc)
>>   {
>> +    const char *val = "cortex-m3";
>> +
>>       mc->desc = "Netduino 2 Machine";
>>       mc->init = netduino2_init;
>> +
>> +    mc->valid_cpu_types = g_array_new(false, false, sizeof(char *));
>> +    g_array_append_val(mc->valid_cpu_types, val);
>>   }
>
>
> Hi Alistair,
>
> Nice idea! Why not adding an helper function which take a const
> char** in argument?

Thanks Fred.

I'm going to send a v2 with just that :)

Thanks,
Alistair

>
> Fred
>
>>     DEFINE_MACHINE("netduino2", netduino2_machine_init)
>>
>

Reply via email to