Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 0/6] prep: some fixes and Super I/O emulation

[Artyom Tarasenko]

On 4/14/12, Hervé Poussineau <address@hidden> wrote:
> Hi,
>
> Patches 1 to 3 implement the pc87312 Super I/O chip.

Nice. It looks pretty similar to the SuperIO chip used in Ultra-5.
We could re-use it in our sun4u machine.
Blue, what do you think?

> This patch has
> been tested on PReP emulation and on IBM 40p (not yet committed).
> Missing part is dynamic reconfiguration of devices (serial, parallel,
> floppy, IDE).

Would it be difficult to implement it? Proprietary Firmware tries to
reconfigure them during the initialization. Although it might be
satisfied if expected aliases are provided.

>However, it is not strictly needed as initial
> configuration can be changed with a property.
>
> Patch 4 completes the qdev-ification of Raven PCI host, where INTA
> register was mistakenly left in PReP machine.
>
> Patches 5 and 6 fix PC speaker in PReP machine, which had never worked.
>
> This patchset is based on Andreas prep-up branch.
>
> Changes v1 -> v2:
> - removed patches already committed to prep-up branch
> - removed patch which changes the default CPU of prep machine
> - changed patches 2/3 according to remarks of Paolo and Andreas
> - added patches 4 to 6
>
> Hervé Poussineau (6):
>   isa: add isa_bus_from_device() method
>   prep: add pc87312 Super I/O emulation
>   prep: use pc87312 Super I/O chip instead of collection of random ISA
>     devices
>   prep: move int-ack register from PReP to Raven PCI emulation
>   pcspk: initialize PC speaker if compiled in
>   prep: initialize pc speaker
>
>  Makefile.objs                   |    1 +
>  arch_init.c                     |    2 +-
>  default-configs/ppc-softmmu.mak |    2 +
>  hw/isa.h                        |    5 +
>  hw/pc87312.c                    |  405
> +++++++++++++++++++++++++++++++++++++++
>  hw/ppc_prep.c                   |   78 +-------
>  hw/prep_pci.c                   |   14 ++
>  trace-events                    |    8 +
>  8 files changed, 446 insertions(+), 69 deletions(-)
>  create mode 100644 hw/pc87312.c
>
> --
> 1.7.9.1
>
>
>


-- 
Regards,
Artyom Tarasenko

solaris/sparc under qemu blog: http://tyom.blogspot.com/search/label/qemu