We seem to migrate the same field twice. It's been this way since Fabrice committed the original file. Since semantically, we basically ignore the first value, make this an unused entry.
Signed-off-by: Anthony Liguori <aligu...@us.ibm.com> --- hw/sb16.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/hw/sb16.c b/hw/sb16.c index c98546a..1c30e4c 100644 --- a/hw/sb16.c +++ b/hw/sb16.c @@ -77,6 +77,7 @@ typedef struct SB16State { int v2x6; + uint8_t csp_param_dummy; uint8_t csp_param; uint8_t csp_value; uint8_t csp_mode; @@ -1313,7 +1314,7 @@ static const VMStateDescription vmstate_sb16 = { VMSTATE_INT32(can_write, SB16State), VMSTATE_INT32(v2x6, SB16State), - VMSTATE_UINT8(csp_param, SB16State), + VMSTATE_UINT8(csp_param_dummy, SB16State), VMSTATE_UINT8(csp_value, SB16State), VMSTATE_UINT8(csp_mode, SB16State), VMSTATE_UINT8(csp_param, SB16State), -- 1.7.0.4