On 05/01/2012 09:00 AM, Peter Maydell wrote:
On 1 May 2012 14:50, Anthony Liguori<anth...@codemonkey.ws> wrote:
On 05/01/2012 03:49 PM, Peter Maydell wrote:
When QOM supports (1) exporting gpio signals and
This is trivial. It'll come in as soon as 1.2 opens up.
You need to get it through code review first...
(just as a random example, this commit:
is going down the wrong path because it's incorrectly introducing
knowledge of device internals to the utility i8259_init()
Do you mean:
- qdev_connect_gpio_out(&dev->qdev, 0, irq_set);
+ pin_connect_qemu_irq(&s->int_out, irq_set);
There are three ways to do this:
1) pin_connect_qemu_irq(i8259_get_int_out(s), irq_set);
2) pin_connect_qemu_irq(&s->int_out, irq_set);
3) pin_connect_qemu_irq(PIN(object_get_child(s, "int_out")), irq_set);
Having converted a lot of devices by hand, I prefer (2). I'd like to move to a
typedef struct PICDevice
/*< private >*/
This let's us use (2), while still embedding devices and keeping the gory state
details private to the implementation.