[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v4 1/5] hw/arm: add very initial support for Canon
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [RFC v4 1/5] hw/arm: add very initial support for Canon DIGIC SoC |
Date: |
Thu, 05 Sep 2013 23:38:26 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130801 Thunderbird/17.0.8 |
Am 05.09.2013 23:23, schrieb Antony Pavlov:
> On Thu, 05 Sep 2013 20:08:34 +0200
> Andreas Färber <address@hidden> wrote:
>> Am 05.09.2013 09:52, schrieb Antony Pavlov:
>>> diff --git a/hw/arm/digic.c b/hw/arm/digic.c
>>> new file mode 100644
>>> index 0000000..95a9fcd
>>> --- /dev/null
>>> +++ b/hw/arm/digic.c
[...]
>>> +static const TypeInfo digic_type_info = {
>>> + .name = TYPE_DIGIC,
>>> + .parent = TYPE_DEVICE,
>>> + .instance_size = sizeof(DigicState),
>>> + .instance_init = digic_init,
>>> + .class_init = digic_class_init,
>>> +};
>>> +
>>> +static void digic_register_types(void)
>>> +{
>>> + type_register_static(&digic_type_info);
>>> +}
>>> +
>>> +type_init(digic_register_types)
>>> diff --git a/include/hw/arm/digic.h b/include/hw/arm/digic.h
>>> new file mode 100644
>>> index 0000000..0ef4723
>>> --- /dev/null
>>> +++ b/include/hw/arm/digic.h
[...]
>>> +typedef struct DigicState {
>>
>> Please add
>> /*< private >*/
>>
>>> + Object parent_obj;
>>
>> /*< private >*/
>
> /*< public >*/ ?
Yes, sorry, copy&paste and then noticing Object. ;)
Or just leave the latter out so that all fields are undocumented.
>> markers for documentation.
>>
>> It needs to be DeviceState parent_obj though.
>
> In your tegra2 support 'Object parent_obj' is used in a similar situation.
>
> http://repo.or.cz/w/qemu/afaerber.git/blob/refs/heads/tegra:/include/hw/arm/tegra2.h#l42
Thanks for spotting, fixed. (It used to be derived from TYPE_OBJECT, but
we decided to provide QOM realize support only for devices.)
Unfortunately Tegra kernel is still stuck after USB init either way...
Cheers,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
[Qemu-devel] [RFC v4 3/5] hw/arm/digic: add timer support, Antony Pavlov, 2013/09/05
[Qemu-devel] [RFC v4 4/5] hw/arm/digic: add UART support, Antony Pavlov, 2013/09/05
- Re: [Qemu-devel] [RFC v4 4/5] hw/arm/digic: add UART support, Peter Maydell, 2013/09/05
- Re: [Qemu-devel] [RFC v4 4/5] hw/arm/digic: add UART support, Antony Pavlov, 2013/09/06
- Re: [Qemu-devel] [RFC v4 4/5] hw/arm/digic: add UART support, Peter Maydell, 2013/09/06
- Re: [Qemu-devel] [RFC v4 4/5] hw/arm/digic: add UART support, Antony Pavlov, 2013/09/06
- Re: [Qemu-devel] [RFC v4 4/5] hw/arm/digic: add UART support, Peter Maydell, 2013/09/06
- Re: [Qemu-devel] [RFC v4 4/5] hw/arm/digic: add UART support, Antony Pavlov, 2013/09/07
- Re: [Qemu-devel] [RFC v4 4/5] hw/arm/digic: add UART support, Peter Maydell, 2013/09/07