Re: [Qemu-devel] [RFC PATCH v2 00/22] remove i386/pc dependency: generic SuperIO

[Paolo Bonzini <pbonzini@redhat.com>]

On 05/03/2018 22:19, Philippe Mathieu-Daudé wrote:
> Since v1:
> - complete rewrite, split out the PIIX devices for another series
> 
> In this rewrite I extract the common SuperIO code from pc87312.c; use it in
> few MIPS boards; then as example, easily add a new SuperIO chipset (SMC37C669)
> to the dp264 machine.
> 
> More devices are being converted but I'm running out of time for the soft
> freeze (mips_r4k, hppa_dino, and the PC ones).
> 
> This series could be the 'part 2' of my previous "remove i386/pc dependency
> from non-PC world" I started around 2.9.
> 
> Philippe Mathieu-Daudé (22):
>   hw/isa: Move parallel_hds_isa_init() to hw/char/parallel-isa.c
>   hw/dma/i8257: Rename DMA_init() to i8257_dma_init()
>   hw/input/i8042: Extract declarations from i386/pc.h into input/i8042.h
>   hw/isa/pc87312: Rename the device type as TYPE_PC87312_SUPERIO
>   hw/isa/pc87312: Use uint16_t for the ISA I/O base address
>   hw/isa/pc87312: Use 'unsigned int' for the irq value
>   hw/isa/superio: Add a Super I/O template based on the PC87312 device
>   hw/isa/pc87312: Inherit from the abstract TYPE_ISA_SUPERIO
>   hw/isa/superio: Factor out the parallel code from pc87312.c
>   hw/isa/superio: Factor out the serial code from pc87312.c
>   hw/isa/superio: Factor out the floppy disc controller code from pc87312.c
>   hw/isa/superio: Add a keyboard/mouse controller (8042)
>   hw/isa/superio: Factor out the IDE code from pc87312.c
>   hw/mips/malta: Code movement
>   hw/isa/superio: Factor out the FDC37M817 Super I/O from mips_malta.c
>   hw/mips/mips_fulong2e: Factor out vt82c686b_southbridge_init()
>   hw/isa/vt82c686: Rename vt82c686b_init() -> vt82c686b_isa_init()
>   hw/isa/vt82c686: Add the TYPE_VT82C686B_SUPERIO
>   hw/isa/superio: Add the SMC FDC37C669 Super I/O
>   hw/alpha/dp264: Add the ISA DMA controller
>   hw/alpha/dp264: Use the TYPE_SMC37C669_SUPERIO
>   hw/i386/pc: Factor out the superio code
> 
>  default-configs/alpha-softmmu.mak |   5 +
>  include/hw/char/parallel.h        |  14 +++
>  include/hw/{isa => dma}/i8257.h   |   6 ++
>  include/hw/i386/pc.h              |  17 ---
>  include/hw/input/i8042.h          |  24 +++++
>  include/hw/isa/isa.h              |   2 -
>  include/hw/isa/pc87312.h          |  24 ++---
>  include/hw/isa/superio.h          |  60 +++++++++++
>  include/hw/isa/vt82c686.h         |   4 +-
>  hw/alpha/dp264.c                  |  13 ++-
>  hw/char/parallel-isa.c            |  36 +++++++
>  hw/char/parallel.c                |   2 +-
>  hw/dma/i82374.c                   |   3 +-
>  hw/dma/i8257.c                    |   4 +-
>  hw/i386/pc.c                      |  77 ++++++++------
>  hw/i386/vmmouse.c                 |   1 +
>  hw/i386/vmport.c                  |   1 +
>  hw/input/pckbd.c                  |   2 +-
>  hw/isa/isa-bus.c                  |  26 -----
>  hw/isa/isa-superio.c              | 210 ++++++++++++++++++++++++++++++++++++++
>  hw/isa/pc87312.c                  | 178 ++++++++++++++------------------
>  hw/isa/smc37c669-superio.c        | 115 +++++++++++++++++++++
>  hw/isa/vt82c686.c                 |  22 +++-
>  hw/mips/mips_fulong2e.c           |  85 +++++++--------
>  hw/mips/mips_jazz.c               |   5 +-
>  hw/mips/mips_malta.c              |  48 ++++-----
>  hw/mips/mips_r4k.c                |   3 +-
>  hw/ppc/prep.c                     |   6 +-
>  hw/sparc/sun4m.c                  |   4 -
>  hw/sparc64/sun4u.c                |   6 +-
>  hw/unicore32/puv3.c               |   1 +
>  hw/char/Makefile.objs             |   1 +
>  hw/isa/Makefile.objs              |   1 +
>  hw/isa/trace-events               |  10 +-
>  MAINTAINERS                       |   6 +-
>  35 files changed, 721 insertions(+), 301 deletions(-)
>  create mode 100644 include/hw/char/parallel.h
>  rename include/hw/{isa => dma}/i8257.h (86%)
>  create mode 100644 include/hw/input/i8042.h
>  create mode 100644 include/hw/isa/superio.h
>  create mode 100644 hw/char/parallel-isa.c
>  create mode 100644 hw/isa/isa-superio.c
>  create mode 100644 hw/isa/smc37c669-superio.c
> 

Queued, thanks.

Paolo