On 04/04/2012 09:21 PM, Julien Grall wrote:
> This patch replaces all register_ioport* with portio_*. It permits to
> use the new Memory stuff like listener.
>
>
>
> +static const MemoryRegionPortio piix4_portio_list[] = {
> + { 0x00, 64, 1, .read = smb_ioport_readb, }, /* s->smb_io_base */
> + { 0x00, 64, 1, .write = smb_ioport_writeb, }, /* s->smb_io_base */
> + PORTIO_END_OF_LIST(),
> +};
> +
> +static const MemoryRegionPortio acpi_portio_list[] = {
> + { 0x00, 4, 4, .write = acpi_dbg_writel, }, /* ACPI_DBG_IO_ADDR */
> + PORTIO_END_OF_LIST(),
> +};
Please replace single-port lists with MemoryRegionOps (and MemoryRegions
in the state structure). I think I mentioned this in the first review,
but perhaps only about a single instance, sorry.
--
error compiling committee.c: too many arguments to function
