VMStateDescription isn't supposed to be modified.

Philippe Mathieu-Daudé (3):
  hw/arm: Constify VMStateDescription
  hw/display/qxl: Constify VMStateDescription
  hw/usb: Constify VMStateDescription

 hw/arm/highbank.c             | 2 +-
 hw/arm/pxa2xx_pic.c           | 2 +-
 hw/arm/spitz.c                | 4 ++--
 hw/arm/strongarm.c            | 2 +-
 hw/arm/z2.c                   | 4 ++--
 hw/display/qxl.c              | 8 ++++----
 hw/dma/pxa2xx_dma.c           | 4 ++--
 hw/misc/mst_fpga.c            | 2 +-
 hw/usb/ccid-card-passthru.c   | 2 +-
 hw/usb/dev-smartcard-reader.c | 8 ++++----
 10 files changed, 19 insertions(+), 19 deletions(-)

-- 
2.26.2

On 3/13/21 11:11 AM, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (3):
>    hw/arm: Constify VMStateDescription
>    hw/display/qxl: Constify VMStateDescription
>    hw/usb: Constify VMStateDescription

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

Hi Philippe,

It seems that vmstate_ecc_state and vmstate_x86_cpu can also be constified.
Found by <git grep "VMStateDescription" | grep -v "const">.

Thanks,
Keqian

On 2021/3/14 1:11, Philippe Mathieu-Daudé wrote:
> VMStateDescription isn't supposed to be modified.
> 
> 
> 
> Philippe Mathieu-Daudé (3):
> 
>   hw/arm: Constify VMStateDescription
> 
>   hw/display/qxl: Constify VMStateDescription
> 
>   hw/usb: Constify VMStateDescription
> 
> 
> 
>  hw/arm/highbank.c             | 2 +-
> 
>  hw/arm/pxa2xx_pic.c           | 2 +-
> 
>  hw/arm/spitz.c                | 4 ++--
> 
>  hw/arm/strongarm.c            | 2 +-
> 
>  hw/arm/z2.c                   | 4 ++--
> 
>  hw/display/qxl.c              | 8 ++++----
> 
>  hw/dma/pxa2xx_dma.c           | 4 ++--
> 
>  hw/misc/mst_fpga.c            | 2 +-
> 
>  hw/usb/ccid-card-passthru.c   | 2 +-
> 
>  hw/usb/dev-smartcard-reader.c | 8 ++++----
> 
>  10 files changed, 19 insertions(+), 19 deletions(-)
> 
> 
> 

On 3/15/21 10:05 AM, Keqian Zhu wrote:
> Hi Philippe,
> 
> It seems that vmstate_ecc_state and vmstate_x86_cpu can also be constified.
> Found by <git grep "VMStateDescription" | grep -v "const">.

Correct (I only searched for the static ones).

Do you mind sending a patch?

> 
> Thanks,
> Keqian
> 
> On 2021/3/14 1:11, Philippe Mathieu-Daudé wrote:
>> VMStateDescription isn't supposed to be modified.

On 2021/4/8 21:22, Philippe Mathieu-Daudé wrote:
> On 3/15/21 10:05 AM, Keqian Zhu wrote:
>> Hi Philippe,
>>
>> It seems that vmstate_ecc_state and vmstate_x86_cpu can also be constified.
>> Found by <git grep "VMStateDescription" | grep -v "const">.
> 
> Correct (I only searched for the static ones).
> 
> Do you mind sending a patch?
OK, I'll do. :)

BRs,
Keqian

> 
>>
>> Thanks,
>> Keqian
>>
>> On 2021/3/14 1:11, Philippe Mathieu-Daudé wrote:
>>> VMStateDescription isn't supposed to be modified.
> .
>