Re: [Qemu-devel] [PATCH v2 00/12] another SysBusDevice::init to Device::realize cleanup

[Eduardo Habkost]

On Tue, Oct 09, 2018 at 05:01:49PM +0200, Philippe Mathieu-Daudé wrote:
> Hi Eduardo,
> 
> On 02/10/2018 23:25, Philippe Mathieu-Daudé wrote:
> > Peter suggested [1] another crusade for this merge window,
> > then Cédric jumped on his horse [2]. My turn on my dromedary.
> > 
> > since v1:
> > - let the empty_slot
> > - sh4_pci and bonito use DeviceState::realize instead of PCIDevice::realize
> > - reword niagara-iob commit message
> > 
> > v1: https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg00220.html
> > - convert few devices to DeviceState::realize,
> > - kill the empty_slot device,
> > - remove unuseful class_init() code [RFC, do we want to keep this?]
> > - few other minor fixes catched while editing
> > 
> > Regards,
> > 
> > Phil.
> > 
> > [1] https://lists.gnu.org/archive/html/qemu-devel/2018-09/msg03605.html
> > [2] https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg00001.html
> > 
> > 001/12:[----] [--] 'trace-events: Fix copy/paste typo'
> > 002/12:[----] [--] 'hw/timer/sun4v-rtc: Convert from DPRINTF() macro to 
> > trace events'
> > 003/12:[----] [--] 'hw/timer/sun4v-rtc: Use DeviceState::realize rather 
> > than SysBusDevice::init'
> > 004/12:[----] [--] 'hw/ssi/xilinx_spi: Use DeviceState::realize rather than 
> > SysBusDevice::init'
> > 005/12:[0006] [FC] 'hw/sh4/sh_pci: Use DeviceState::realize rather than 
> > SysBusDevice::init'
> > 006/12:[0006] [FC] 'hw/pci-host/bonito: Use DeviceState::realize rather 
> > than SysBusDevice::init'
> > 007/12:[----] [--] 'hw/mips/gt64xxx_pci: Convert gt64120_reset() function 
> > into Device reset method'
> > 008/12:[----] [--] 'hw/mips/gt64xxx_pci: Mark as bridge device'
> > 009/12:[----] [--] 'hw/sparc64/niagara: Model the I/O Bridge with the 
> > 'unimplemented_device''
> > 010/12:[----] [--] 'hw/alpha/typhoon: Remove unuseful code'
> > 011/12:[----] [--] 'hw/hppa/dino: Remove unuseful code'
> > 012/12:[----] [--] 'hw/mips/malta: Remove unuseful code'
> 
> Can you take this via your machine-next tree?

I'll do it.  Thanks!

-- 
Eduardo