Messages in this thread | | | Subject | Re: [PATCH] clocksource: mxs_timer: add missing semicolon when DEBUG is defined | From | Daniel Lezcano <> | Date | Mon, 18 Jan 2021 16:15:02 +0100 |
| |
On 18/01/2021 14:49, trix@redhat.com wrote: > From: Tom Rix <trix@redhat.com> > > When DEBUG is defined this error occurs > > drivers/clocksource/mxs_timer.c:138:1: error: > expected ‘;’ before ‘}’ token > > The preceding statement needs a semicolon. > > Fixes: eb8703e2ef7c ("clockevents/drivers/mxs: Migrate to new 'set-state' interface") > Signed-off-by: Tom Rix <trix@redhat.com> > --- > drivers/clocksource/mxs_timer.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clocksource/mxs_timer.c b/drivers/clocksource/mxs_timer.c > index bc96a4cbf26c..55aa6b72d075 100644 > --- a/drivers/clocksource/mxs_timer.c > +++ b/drivers/clocksource/mxs_timer.c > @@ -133,7 +133,7 @@ static void mxs_irq_clear(char *state) > timrot_irq_acknowledge(); > > #ifdef DEBUG > - pr_info("%s: changing mode to %s\n", __func__, state) > + pr_info("%s: changing mode to %s\n", __func__, state); > #endif /* DEBUG */
Mind to replace by pr_debug and remove the #ifdef ?
> } > >
-- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
| |