diff mbox series

[for-9.2,05/10] hw/dma/xilinx_axidma: Use semicolon at end of statement, not comma

Message ID 20240813165250.2717650-6-peter.maydell@linaro.org
State Superseded
Headers show
Series s390: Convert virtio-ccw, cpu to three-phase reset, and followup cleanup | expand

Commit Message

Peter Maydell Aug. 13, 2024, 4:52 p.m. UTC 
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.maydell@linaro.org>
---
 hw/dma/xilinx_axidma.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Richard Henderson Aug. 13, 2024, 11:08 p.m. UTC | #1 
On 8/14/24 02: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.maydell@linaro.org>
> ---
>   hw/dma/xilinx_axidma.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

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

r~
Thomas Huth Aug. 14, 2024, 10:55 a.m. UTC | #2 
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.maydell@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 <thuth@redhat.com>
diff mbox series

Patch

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);
 }