diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-11-25 14:25:54 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-12-12 05:15:49 +0300 |
| commit | 601e6e3cc5bf6adb7d076fe24d10f6191a25ba9b (patch) | |
| tree | 54a81af42d88f143cf316388e3ed80522440b886 /arch/sparc/kernel/leon_pci_grpci2.c | |
| parent | b5c3206190f1fddd100b3060eb15f0d775ffeab8 (diff) | |
| download | linux-601e6e3cc5bf6adb7d076fe24d10f6191a25ba9b.tar.xz | |
sparc: leon: Fix a retry loop in leon_init_timers()
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>
Diffstat (limited to 'arch/sparc/kernel/leon_pci_grpci2.c')
0 files changed, 0 insertions, 0 deletions