On Thu, Feb 24, 2011 at 7:57 PM, Juan Quintela <quint...@redhat.com> wrote:
> Signed-off-by: Juan Quintela <quint...@redhat.com>
> ---
> hw/max111x.c | 49 +++++++++++++++++--------------------------------
> 1 files changed, 17 insertions(+), 32 deletions(-)
>
> diff --git a/hw/max111x.c b/hw/max111x.c
> index 3adc3e4..eca117e 100644
> --- a/hw/max111x.c
> +++ b/hw/max111x.c
> @@ -94,36 +94,22 @@ static uint32_t max111x_transfer(SSISlave *dev, uint32_t
> value)
> return max111x_read(s);
> }
>
> -static void max111x_save(QEMUFile *f, void *opaque)
> -{
> - MAX111xState *s = (MAX111xState *) opaque;
> - int i;
> -
> - qemu_put_8s(f, &s->tb1);
> - qemu_put_8s(f, &s->rb2);
> - qemu_put_8s(f, &s->rb3);
> - qemu_put_be32(f, s->inputs);
> - qemu_put_be32(f, s->com);
> - for (i = 0; i < s->inputs; i ++)
> - qemu_put_byte(f, s->input[i]);
> -}
> -
> -static int max111x_load(QEMUFile *f, void *opaque, int version_id)
> -{
> - MAX111xState *s = (MAX111xState *) opaque;
> - int i;
> -
> - qemu_get_8s(f, &s->tb1);
> - qemu_get_8s(f, &s->rb2);
> - qemu_get_8s(f, &s->rb3);
> - if (s->inputs != qemu_get_be32(f))
> - return -EINVAL;
> - s->com = qemu_get_be32(f);
> - for (i = 0; i < s->inputs; i ++)
> - s->input[i] = qemu_get_byte(f);
> -
> - return 0;
> -}
> +static const VMStateDescription vmstate_max111x = {
> + .name = "max111x",
> + .version_id = 0,
> + .minimum_version_id = 0,
> + .minimum_version_id_old = 0,
> + .fields = (VMStateField[]) {
> + VMSTATE_UINT8(tb1, MAX111xState),
> + VMSTATE_UINT8(rb2, MAX111xState),
> + VMSTATE_UINT8(rb3, MAX111xState),
> + VMSTATE_INT32(inputs, MAX111xState),
Shouldn't this be VMSTATE_EQUAL_INT32(inputs, MAX111xState)?
