[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V5 4/7] nios2: Add IIC interrupt controller emul
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH V5 4/7] nios2: Add IIC interrupt controller emulation |
Date: |
Wed, 22 Feb 2017 19:31:17 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Marek Vasut <address@hidden> writes:
> On 02/22/2017 07:31 AM, Markus Armbruster wrote:
>> I know this has been committed already, but here goes anyway:
>>
>> Marek Vasut <address@hidden> writes:
>>
>>> From: Chris Wulff <address@hidden>
>>>
>>> Add the Altera Nios2 internal interrupt controller model.
>>>
>>> Signed-off-by: Marek Vasut <address@hidden>
>>> Cc: Chris Wulff <address@hidden>
>>> Cc: Jeff Da Silva <address@hidden>
>>> Cc: Ley Foon Tan <address@hidden>
>>> Cc: Sandra Loosemore <address@hidden>
>>> Cc: Yves Vandervennet <address@hidden>
>>> Cc: Alexander Graf <address@hidden>
>>> Cc: Richard Henderson <address@hidden>
>>> Reviewed-by: Alexander Graf <address@hidden>
>>> ---
>>> V3: Checkpatch cleanup
>>> V4: Rebase on top of qemu/master
>>> V5: Rebase on top of qemu/master
>>> ---
>>> hw/intc/Makefile.objs | 1 +
>>> hw/intc/nios2_iic.c | 103
>>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>>> 2 files changed, 104 insertions(+)
>>> create mode 100644 hw/intc/nios2_iic.c
>>>
>>> diff --git a/hw/intc/Makefile.objs b/hw/intc/Makefile.objs
>>> index 2f44a2da26..8948106ac4 100644
>>> --- a/hw/intc/Makefile.objs
>>> +++ b/hw/intc/Makefile.objs
[...]
>>> +static Property altera_iic_properties[] = {
>>> + DEFINE_PROP_PTR("cpu", AlteraIIC, cpu),
>>
>> Could this be a QOM link instead?
>>
>> We want to get rid of pointer properties. Commit d2fe4ec converts one
>> to a link.
>
> By all means, yes. Can you send a patch ? I can help testing it.
> Otherwise I can cook one up, but it might take a few days.
I'm in a bit of a time squeeze right now, and I'd be grateful if you
could do it. A few days sounds great to me.