On 13/08/2024 18.52, Peter Maydell wrote:
In axidma_class_init() we accidentally used a comma at the end of
a statement rather than a semicolon. This has no ill effects, but
it's obviously not intended and it means that Coccinelle scripts
for instance will fail to match on the two statements. Use a
semicolon instead.

Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
---
  hw/dma/xilinx_axidma.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/dma/xilinx_axidma.c b/hw/dma/xilinx_axidma.c
index c9cfc3169b8..7707634253a 100644
--- a/hw/dma/xilinx_axidma.c
+++ b/hw/dma/xilinx_axidma.c
@@ -626,7 +626,7 @@ static void axidma_class_init(ObjectClass *klass, void 
*data)
  {
      DeviceClass *dc = DEVICE_CLASS(klass);
- dc->realize = xilinx_axidma_realize,
+    dc->realize = xilinx_axidma_realize;
      dc->reset = xilinx_axidma_reset;
      device_class_set_props(dc, axidma_properties);
  }

Reviewed-by: Thomas Huth <th...@redhat.com>


Reply via email to