[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/6] hw/misc: Add a model of the Xilinx ZynqMP CRF
From: |
Peter Maydell |
Subject: |
Re: [PATCH v2 3/6] hw/misc: Add a model of the Xilinx ZynqMP CRF |
Date: |
Tue, 15 Feb 2022 17:20:10 +0000 |
On Thu, 3 Feb 2022 at 14:01, Edgar E. Iglesias <edgar.iglesias@gmail.com> wrote:
>
> From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
>
> Add a model of the Xilinx ZynqMP CRF. At the moment this
> is mostly a stub model.
> +static const VMStateDescription vmstate_crf = {
> + .name = TYPE_XLNX_ZYNQMP_CRF,
> + .version_id = 1,
> + .minimum_version_id = 1,
> + .minimum_version_id_old = 1,
This is still on my to-review list to look at properly, but
this setting of .minimum_version_id_old is incorrect. The
field is only used if the VMStateDescription sets a non-NULL
load_state_old function, which this one does not.
We cleaned this up in commit 17e313406126, and I'm about
to send a patch which drops the fields entirely.
thanks
-- PMM
- [PATCH v2 0/6] hw/arm: zynqmp: Add CRF and APU control to support PSCI, Edgar E. Iglesias, 2022/02/03
- [PATCH v2 1/6] hw/arm/xlnx-zynqmp: Add an unimplemented SERDES area, Edgar E. Iglesias, 2022/02/03
- [PATCH v2 4/6] hw/arm/xlnx-zynqmp: Connect the ZynqMP CRF, Edgar E. Iglesias, 2022/02/03
- [PATCH v2 2/6] target/arm: Make rvbar settable after realize, Edgar E. Iglesias, 2022/02/03
- [PATCH v2 3/6] hw/misc: Add a model of the Xilinx ZynqMP CRF, Edgar E. Iglesias, 2022/02/03
- [PATCH v2 5/6] hw/misc: Add a model of the Xilinx ZynqMP APU Control, Edgar E. Iglesias, 2022/02/03
- [PATCH v2 6/6] hw/arm/xlnx-zynqmp: Connect the ZynqMP APU Control, Edgar E. Iglesias, 2022/02/03