RFC remove i386/pc dependency: generic SuperIO
Status from this message: Queued, thanks.
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 RFC Latest22 patches 11 repliesApplied
- v1 RFC29 patches 2 repliesSuperseded
Changes in v2 from the cover letter
- complete rewrite, split out the PIIX devices for another series
Patches in v2 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/22 | hw/isa: Move parallel_hds_isa_init() to hw/char/parallel-isa.cMAINTAINERS hw/ hw/ +9 more | +59 | 0 replies | |
| 2/22 | hw/dma/i8257: Rename DMA_init() to i8257_dma_init()MAINTAINERS hw/ hw/ +8 more | +19 | 0 replies | |
| 3/22 | hw/input/i8042: Extract declarations from i386/pc.h into input/i8042.hMAINTAINERS hw/ hw/ +12 more | +43 | 1 reply | |
| 4/22 | hw/isa/pc87312: Rename the device type as TYPE_PC87312_SUPERIOhw/ hw/ include/ | +4 | 1 reply | |
| 5/22 | hw/isa/pc87312: Use uint16_t for the ISA I/O base addresshw/ include/ | +8 | 0 replies | |
| 6/22 | hw/isa/pc87312: Use 'unsigned int' for the irq valuehw/ | +3 | 0 replies | |
| 7/22 | hw/isa/superio: Add a Super I/O template based on the PC87312 devicehw/ hw/ include/ | +72 | 0 replies | |
| 8/22 | hw/isa/pc87312: Inherit from the abstract TYPE_ISA_SUPERIOhw/ include/ | +14 | 0 replies | |
| 9/22 | hw/isa/superio: Factor out the parallel code from pc87312.chw/ hw/ hw/ +2 more | +87 | 1 reply | |
| 10/22 | hw/isa/superio: Factor out the serial code from pc87312.chw/ hw/ hw/ +2 more | +54 | 0 replies | |
| 11/22 | hw/isa/superio: Factor out the floppy disc controller code from pc87312.chw/ hw/ hw/ +2 more | +58 | 0 replies | |
| 12/22 | hw/isa/superio: Add a keyboard/mouse controller (8042)hw/ hw/ include/ | +4 | 1 reply | |
| 13/22 | hw/isa/superio: Factor out the IDE code from pc87312.chw/ hw/ hw/ +1 more | +45 | 0 replies | |
| 14/22 | hw/mips/malta: Code movementhw/ | +7 | 0 replies | |
| 15/22 | hw/isa/superio: Factor out the FDC37M817 Super I/O from mips_malta.chw/ hw/ include/ | +32 | 1 reply | |
| 16/22 | hw/mips/mips_fulong2e: Factor out vt82c686b_southbridge_init()hw/ | +41 | 0 replies | |
| 17/22 | hw/isa/vt82c686: Rename vt82c686b_init() -> vt82c686b_isa_init()hw/ hw/ include/ | +3 | 0 replies | |
| 18/22 | hw/isa/vt82c686: Add the TYPE_VT82C686B_SUPERIOhw/ hw/ include/ | +25 | 1 reply | |
| 19/22 | hw/isa/superio: Add the SMC FDC37C669 Super I/Ohw/ hw/ include/ | +117 | 0 replies | |
| 20/22 | hw/alpha/dp264: Add the ISA DMA controllerdefault-configs/ hw/ | +6 | 0 replies | |
| 21/22 | hw/alpha/dp264: Use the TYPE_SMC37C669_SUPERIOdefault-configs/ hw/ | +7 | 2 replies | |
| 22/22 | hw/i386/pc: Factor out the superio codehw/ | +41 | 0 replies |
Reviews trailers per patch
3 of 22 patches have a Reviewed-by or Acked-by.
| Patch | David Gibson |
|---|---|
| 1/22 | |
| 2/22 | |
| 3/22 | A-b |
| 4/22 | A-b |
| 5/22 | |
| 6/22 | |
| 7/22 | |
| 8/22 | |
| 9/22 | |
| 10/22 | |
| 11/22 | |
| 12/22 | A-b |
| 13/22 | |
| 14/22 | |
| 15/22 | |
| 16/22 | |
| 17/22 | |
| 18/22 | |
| 19/22 | |
| 20/22 | |
| 21/22 | |
| 22/22 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [Qemu-devel] [RFC PATCH v2 00/22] remove i386/pc dependency: generic SuperIO
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.cSignature · 1 line
2.16.2