[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 05/11] pxa2xx_pic: update to use qdev and arm-pi
From: |
andrzej zaborowski |
Subject: |
Re: [Qemu-devel] [PATCH 05/11] pxa2xx_pic: update to use qdev and arm-pic |
Date: |
Fri, 11 Feb 2011 23:17:12 +0100 |
Hi,
On 11 February 2011 21:24, Dmitry Eremin-Solenikov <address@hidden> wrote:
> On 2/11/11, Dmitry Eremin-Solenikov <address@hidden> wrote:
>> I just planned to later reuse allocated arm-pic IRQ's (the new one) to
>> be passed to pxa2xx-gpio (to drop usage of cpu-env). I think. I can
>> still allocate
>> arm-pic but use only the last IRQ from it. Will that be suitable for you?
>
> I recollected the reason for this change: there is no clean way to pass
> CPUState
> to qdev. So It's either DEFINE_PROP_PTR() - like hack, or
> encapsulating thins into
> qemu_irq
That's true, but in that patchset pxa2xx_pic is not turned into a qdev
device (separate from pxa2xx core). Do you think there's any value in
doing that? I think it's ok if it doesn't present any
issues/overheads or if there's a clear benefit from doing that.
Cheers