On Wed, Jun 8, 2011 at 9:55 PM, Andreas Färber
<address@hidden> wrote:
I've refined the series to track the state in ISADevice and to
expose it as VMState.
Error handling has been improved, and setting the state multiple
times is no-op now.
To read the state, I'm introducing support for bool qdev properties.
Some more qdev_prop_get_*() helpers are introduced, too.
Still need to do some runtime testing, but I'd like to hear if this
is getting
mergeable now, especially wrt VMState.
Andreas
Andreas Färber (11):
qdev: Add support for property type bool
qdev: Add helpers for reading properties
isa: Provide set_state callback
isa: Allow to un-assign I/O ports
isa: Allow to un-associate an IRQ
I like the patches above.
But I think the set_state() interface could be improved. For example,
cpu_register_io_memory() gives an index which is passed to
sysbus_register_mmio(). Then the board can instantiate the device at
desired location without caring about the device internals. With
set_state(), the device does everything.