On 26 July 2016 at 19:25, Alistair Francis <alistair.fran...@xilinx.com> wrote: > Signed-off-by: Alistair Francis <alistair.fran...@xilinx.com> > --- > > There is a indentation error in this patch in the gem_transmit function. > I have written it like that to make it easier to see the changes. It is > fixed in the next patch.
> @@ -1417,8 +1500,8 @@ static void gem_init(Object *obj) > > static const VMStateDescription vmstate_cadence_gem = { > .name = "cadence_gem", > - .version_id = 3, > - .minimum_version_id = 3, > + .version_id = 4, > + .minimum_version_id = 4, > .fields = (VMStateField[]) { > VMSTATE_UINT32_ARRAY(regs, CadenceGEMState, CADENCE_GEM_MAXREG), > VMSTATE_UINT16_ARRAY(phy_regs, CadenceGEMState, 32), This bit should have moved into the other patch with the change of the CADENCE_GEM_MAXREG define, right? Otherwise Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM