sparc: leon: Fix a retry loop in leon_init_timers()
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 25 Nov 2016 11:25:54 +0000 (14:25 +0300)
committerBen Hutchings <ben@decadent.org.uk>
Thu, 16 Mar 2017 02:18:31 +0000 (02:18 +0000)
commit5980435b45c75ce52e67565120cc6087b6c30793
tree68cafd8822a94c63299c161dc7a4b989ff326668
parent1af5e763960296c76705a20b52777de65abe2e9e
sparc: leon: Fix a retry loop in leon_init_timers()

commit 601e6e3cc5bf6adb7d076fe24d10f6191a25ba9b upstream.

The original code causes a static checker warning because it has a
continue inside a do { } while (0); loop.  In that context, a continue
and a break are equivalent.  The intent was to go back to the start of
the loop so the continue was a bug.

I've added a retry label at the start and changed the continue to a goto
retry.  Then I removed the do { } while (0) loop and pulled the code in
one indent level.

Fixes: 2791c1a43900 ("SPARC/LEON: added support for selecting Timer Core and Timer within core")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
arch/sparc/kernel/leon_kernel.c