On Thu, 3 Feb 2022 at 14:01, Edgar E. Iglesias <[email protected]> wrote: > > From: "Edgar E. Iglesias" <[email protected]> > > Add a model of the Xilinx ZynqMP CRF. At the moment this > is mostly a stub model. > > Reviewed-by: Philippe Mathieu-Daudé <[email protected]> > Signed-off-by: Edgar E. Iglesias <[email protected]> > ---
> +#define TYPE_XLNX_ZYNQMP_CRF "xlnx.zynqmp_crf" > + > +#define XILINX_CRF(obj) \ > + OBJECT_CHECK(XlnxZynqMPCRF, (obj), TYPE_XLNX_ZYNQMP_CRF) We prefer the OBJECT_DECLARE_SIMPLE_TYPE rather than directly defining a cast macro these days. (It also provides a typedef for you, among other things.) Apart from that, and dropping minimum_version_id_old, Reviewed-by: Peter Maydell <[email protected]> thanks -- PMM
