| Message ID | 20200824215721.2960162-2-ehabkost@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | qom: Use typedefs instead of struct names on instance_size/class_size | expand |
On Mon, Aug 24, 2020 at 2:58 PM Eduardo Habkost <ehabkost@redhat.com> wrote: > > This makes the code consistent with the rest of QOM code in QEMU, > and will make automated conversion to type declaration macros > simpler. > > Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> Reviewed-by: Alistair Francis <alistair.francis@wdc.com> Alistair > --- > Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com> > Cc: Alistair Francis <alistair@alistair23.me> > Cc: Peter Maydell <peter.maydell@linaro.org> > Cc: qemu-arm@nongnu.org > Cc: qemu-devel@nongnu.org > --- > hw/dma/xilinx_axidma.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/dma/xilinx_axidma.c b/hw/dma/xilinx_axidma.c > index a4812e480a..5ad8bd3d2e 100644 > --- a/hw/dma/xilinx_axidma.c > +++ b/hw/dma/xilinx_axidma.c > @@ -634,7 +634,7 @@ static const TypeInfo axidma_info = { > static const TypeInfo xilinx_axidma_data_stream_info = { > .name = TYPE_XILINX_AXI_DMA_DATA_STREAM, > .parent = TYPE_OBJECT, > - .instance_size = sizeof(struct XilinxAXIDMAStreamSlave), > + .instance_size = sizeof(XilinxAXIDMAStreamSlave), > .class_init = xilinx_axidma_stream_class_init, > .class_data = &xilinx_axidma_data_stream_class, > .interfaces = (InterfaceInfo[]) { > @@ -646,7 +646,7 @@ static const TypeInfo xilinx_axidma_data_stream_info = { > static const TypeInfo xilinx_axidma_control_stream_info = { > .name = TYPE_XILINX_AXI_DMA_CONTROL_STREAM, > .parent = TYPE_OBJECT, > - .instance_size = sizeof(struct XilinxAXIDMAStreamSlave), > + .instance_size = sizeof(XilinxAXIDMAStreamSlave), > .class_init = xilinx_axidma_stream_class_init, > .class_data = &xilinx_axidma_control_stream_class, > .interfaces = (InterfaceInfo[]) { > -- > 2.26.2 > >
diff --git a/hw/dma/xilinx_axidma.c b/hw/dma/xilinx_axidma.c index a4812e480a..5ad8bd3d2e 100644 --- a/hw/dma/xilinx_axidma.c +++ b/hw/dma/xilinx_axidma.c @@ -634,7 +634,7 @@ static const TypeInfo axidma_info = { static const TypeInfo xilinx_axidma_data_stream_info = { .name = TYPE_XILINX_AXI_DMA_DATA_STREAM, .parent = TYPE_OBJECT, - .instance_size = sizeof(struct XilinxAXIDMAStreamSlave), + .instance_size = sizeof(XilinxAXIDMAStreamSlave), .class_init = xilinx_axidma_stream_class_init, .class_data = &xilinx_axidma_data_stream_class, .interfaces = (InterfaceInfo[]) { @@ -646,7 +646,7 @@ static const TypeInfo xilinx_axidma_data_stream_info = { static const TypeInfo xilinx_axidma_control_stream_info = { .name = TYPE_XILINX_AXI_DMA_CONTROL_STREAM, .parent = TYPE_OBJECT, - .instance_size = sizeof(struct XilinxAXIDMAStreamSlave), + .instance_size = sizeof(XilinxAXIDMAStreamSlave), .class_init = xilinx_axidma_stream_class_init, .class_data = &xilinx_axidma_control_stream_class, .interfaces = (InterfaceInfo[]) {
This makes the code consistent with the rest of QOM code in QEMU, and will make automated conversion to type declaration macros simpler. Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> --- Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com> Cc: Alistair Francis <alistair@alistair23.me> Cc: Peter Maydell <peter.maydell@linaro.org> Cc: qemu-arm@nongnu.org Cc: qemu-devel@nongnu.org --- hw/dma/xilinx_axidma.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)