[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu devel v8 PATCH 5/5] msf2: Add Emcraft's Smartfusi
From: |
sundeep subbaraya |
Subject: |
Re: [Qemu-devel] [Qemu devel v8 PATCH 5/5] msf2: Add Emcraft's Smartfusion2 SOM kit |
Date: |
Fri, 15 Sep 2017 12:01:33 +0530 |
Hi Peter,
On Thu, Sep 14, 2017 at 10:44 PM, Peter Maydell <address@hidden>
wrote:
> On 7 September 2017 at 20:24, Subbaraya Sundeep <address@hidden>
> wrote:
> > Emulated Emcraft's Smartfusion2 System On Module starter
> > kit.
> > +static void emcraft_sf2_machine_init(MachineClass *mc)
> > +{
> > + mc->desc = "SmartFusion2 SOM kit from Emcraft (M2S010)";
> > + mc->init = emcraft_sf2_s2s010_init;
> > + mc->ignore_memory_transaction_failures = true;
>
> Please don't set ignore_memory_transaction_failures in new boards.
> This is a legacy-old-code-only flag.
>
> New boards should define enough devices, either properly
> or using create_unimplemented_device(), to make whatever code
> they're being tested against run.
>
> This is a firm requirement for this code to go into master,
> because once we let a board in with the flag set it's almost
> impossible to clear it because we don't know what guest
> code that previously worked on the board will now break.
>
Ok. I am working on it and will send next spin with this fix.
Thanks,
Sundeep
> thanks
> -- PMM
>
- Re: [Qemu-devel] [Qemu devel v8 PATCH 3/5] msf2: Add Smartfusion2 SPI controller, (continued)
[Qemu-devel] [Qemu devel v8 PATCH 5/5] msf2: Add Emcraft's Smartfusion2 SOM kit, Subbaraya Sundeep, 2017/09/07
Re: [Qemu-devel] [Qemu devel v8 PATCH 0/5] Add support for Smartfusion2 SoC, Philippe Mathieu-Daudé, 2017/09/07