diff options
| author | Andrew Morton <akpm@linux-foundation.org> | 2014-07-31 09:38:02 +1000 |
|---|---|---|
| committer | Stephen Rothwell <sfr@canb.auug.org.au> | 2014-08-06 16:44:49 +1000 |
| commit | 3aba15f3d027be44165cba753a16e4279a181dbb (patch) | |
| tree | 285ef941661eb09c58b16bc053c0320fdff0b380 | |
| parent | 9cf42345a9a1d5a4bc44c3ac1233b66a0a791178 (diff) | |
kexec-implementation-of-new-syscall-kexec_file_load-checkpatch-fixes
WARNING: else is not generally useful after a break or return
#673: FILE: kernel/kexec.c:2024:
+ return locate_mem_hole_top_down(start, end, kbuf);
+ else
total: 0 errors, 1 warnings, 677 lines checked
./patches/kexec-implementation-of-new-syscall-kexec_file_load.patch has style problems, please review.
If any of these errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
Please run checkpatch prior to sending patches
Cc: Vivek Goyal <vgoyal@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
| -rw-r--r-- | kernel/kexec.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/kexec.c b/kernel/kexec.c index ea65e0af39fb..b4a014c58cff 100644 --- a/kernel/kexec.c +++ b/kernel/kexec.c @@ -2025,8 +2025,7 @@ static int locate_mem_hole_callback(u64 start, u64 end, void *arg) */ if (kbuf->top_down) return locate_mem_hole_top_down(start, end, kbuf); - else - return locate_mem_hole_bottom_up(start, end, kbuf); + return locate_mem_hole_bottom_up(start, end, kbuf); } /* |