Anthony Liguori <aligu...@us.ibm.com> wrote:
> 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),
VMSTATE_UNUSED(1) instead?
Later, Juan.
