Re: [Qemu-arm] [Qemu-devel] [PATCH 00/17] some ARM platform QOM'ify work

[Peter Maydell]

On 7 June 2016 at 22:32, Mark Cave-Ayland <address@hidden> wrote:
> On 07/06/16 11:34, xiaoqiang zhao wrote:
>
>> This patch series QOM'ify ARM platform related devices.
>> Where we drop the sysbus init function if possible and use
>> instance_init and DeviceClass::realize function.
>>
>> xiaoqiang zhao (17):
>>   hw/i2c: QOM'ify bitbang_i2c.c
>>   hw/i2c: QOM'ify exynos4210_i2c.c
>>   hw/i2c: QOM'ify omap_i2c.c
>>   hw/i2c: QOM'ify versatile_i2c.c
>>   hw/gpio: QOM'ify mpc8xxx.c
>>   hw/gpio: QOM'ify omap_gpio.c
>>   hw/gpio: QOM'ify pl061.c
>>   hw/gpio: QOM'ify zaurus.c
>>   hw/misc: QOM'ify arm_l2x0.c
>>   hw/misc: QOM'ify eccmemctl.c
>>   hw/misc: QOM'ify exynos4210_pmu.c
>>   hw/misc: QOM'ify mst_fpga.c
>>   hw/misc: QOM'ify slavio_misc.c
>>   hw/dma: QOM'ify pxa2xx_dma.c
>>   hw/dma: QOM'ify sparc32_dma.c
>>   hw/dma: QOM'ify sun4m_iommu.c
>>   hw/sd: QOM'ify pl181.c
>>
>>  hw/dma/pxa2xx_dma.c      | 38 +++++++++++++++++-------------
>>  hw/dma/sparc32_dma.c     | 25 ++++++++++++--------
>>  hw/dma/sun4m_iommu.c     | 12 ++++------
>>  hw/gpio/mpc8xxx.c        | 20 +++++++++-------
>>  hw/gpio/omap_gpio.c      | 61 
>> ++++++++++++++++++++++++------------------------
>>  hw/gpio/pl061.c          | 24 +++++++------------
>>  hw/gpio/zaurus.c         | 14 +++++------
>>  hw/i2c/bitbang_i2c.c     | 14 +++++------
>>  hw/i2c/exynos4210_i2c.c  | 13 +++++------
>>  hw/i2c/omap_i2c.c        | 44 ++++++++++++++++++++--------------
>>  hw/i2c/versatile_i2c.c   | 19 +++++----------
>>  hw/misc/arm_l2x0.c       | 11 ++++-----
>>  hw/misc/eccmemctl.c      | 25 +++++++++++++-------
>>  hw/misc/exynos4210_pmu.c | 11 ++++-----
>>  hw/misc/mst_fpga.c       | 13 +++++------
>>  hw/misc/slavio_misc.c    | 43 ++++++++++++++--------------------
>>  hw/sd/pl181.c            | 26 +++++++++++++--------
>>  17 files changed, 207 insertions(+), 206 deletions(-)
>
> Patches 16 and 17 for sparc32_dma and sun4m_iommu are actually sun4m
> SPARC rather than ARM devices, so while I don't mind if these go through
> someone else's tree then please ensure that you also test
> qemu-system-sparc thoroughly with these patches.

I don't have a good set of sparc test images, so probably
better if you take those. I think eccmemctl.c is sparc too.
mpc8xxx.c is PPC.

thanks
-- PMM