On 29 November 2013 11:01, Peter Maydell <peter.mayd...@linaro.org> wrote:
> On 27 November 2013 08:23, liguang <lig.f...@cn.fujitsu.com> wrote:
>> Signed-off-by: liguang <lig.f...@cn.fujitsu.com>
>> ---
>> include/hw/ptimer.h | 3 +++
>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/include/hw/ptimer.h b/include/hw/ptimer.h
>> index 28fcaf1..a9b8f1d 100644
>> --- a/include/hw/ptimer.h
>> +++ b/include/hw/ptimer.h
>> @@ -27,6 +27,9 @@ void ptimer_stop(ptimer_state *s);
>>
>> extern const VMStateDescription vmstate_ptimer;
>>
>> +#define VMSTATE_PTIMER_ARRAY(_f, _s, _n) \
>> + VMSTATE_STRUCT_ARRAY(_f, _s, _n, 0, vmstate_ptimer, ptimer_state*)
>
> The final parameter here should just be the struct type
> ("ptimer_state"), not a pointer-to-it.
Actually, looking at the patch where you've used this,
you do actually want to deal with an
array-of-pointers-to-ptimers. For that you need
#define VMSTATE_PTIMER_ARRAY(_f, _s, _n) \
VMSTATE_ARRAY_OF_POINTER(_f, _s, _n, 0, vmstate_ptimer, ptimer_state *)
(compare VMSTATE_TIMER_ARRAY in vmstate.h)
thanks
-- PMM
