On 8/20/20 2:11 AM, Eduardo Habkost wrote: > The macro definition had an extra semicolon. This was never > noticed because the macro was only being used where it didn't > make a difference. > > Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> > Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> > Signed-off-by: Eduardo Habkost <ehabk...@redhat.com>
Reviewed-by: Cédric Le Goater <c...@kaod.org> Thanks, C. > --- > Changes v1 -> v2: none > > --- > Cc: "Cédric Le Goater" <c...@kaod.org> > Cc: Peter Maydell <peter.mayd...@linaro.org> > Cc: Andrew Jeffery <and...@aj.id.au> > Cc: Joel Stanley <j...@jms.id.au> > Cc: qemu-...@nongnu.org > Cc: qemu-devel@nongnu.org > --- > include/hw/timer/aspeed_timer.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/hw/timer/aspeed_timer.h b/include/hw/timer/aspeed_timer.h > index 948329893c..d7c7d8ad28 100644 > --- a/include/hw/timer/aspeed_timer.h > +++ b/include/hw/timer/aspeed_timer.h > @@ -26,7 +26,7 @@ > #include "hw/misc/aspeed_scu.h" > > #define ASPEED_TIMER(obj) \ > - OBJECT_CHECK(AspeedTimerCtrlState, (obj), TYPE_ASPEED_TIMER); > + OBJECT_CHECK(AspeedTimerCtrlState, (obj), TYPE_ASPEED_TIMER) > #define TYPE_ASPEED_TIMER "aspeed.timer" > #define TYPE_ASPEED_2400_TIMER TYPE_ASPEED_TIMER "-ast2400" > #define TYPE_ASPEED_2500_TIMER TYPE_ASPEED_TIMER "-ast2500" >