Peter Maydell <peter.mayd...@linaro.org> wrote: > On 19 March 2012 22:57, Juan Quintela <quint...@redhat.com> wrote: >> +/* 32 bit float */ >> + >> +typedef union { >> + float32 f; >> + uint32_t i; >> +} VMStateFloat32; >> + >> +static int get_float32(QEMUFile *f, void *pv, size_t size) >> +{ >> + float32 *v = pv; >> + VMStateFloat32 u; >> + qemu_get_be32s(f, &u.i); >> + *v = u.f; >> + return 0; >> +} >> + >> +static void put_float32(QEMUFile *f, void *pv, size_t size) >> +{ >> + float32 *v = pv; >> + VMStateFloat32 u; >> + u.f = *v; >> + qemu_put_be32s(f, &u.i); >> +} > > This conversion (float32<->uint32_t) should be done via > float32_val() and make_float32().
you are right. we can even simplify things with this. Thanks.