diff options
author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-10 17:48:31 -0800 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-03-13 21:23:58 +0100 |
commit | 3e1cab447b648e5cfdaa55d098075ddd5ede0efc (patch) | |
tree | 44ce75b2b6a43330dc39c3eefd5ff424925ba966 /src/lib/imd.c | |
parent | 49fd42dc65d6686f11eff691e37371ea6130a902 (diff) | |
download | coreboot-3e1cab447b648e5cfdaa55d098075ddd5ede0efc.tar.xz |
src/lib: Remove unnecessary code
Fix the following warnings detected by checkpatch.pl:
WARNING: break is not useful after a goto or return
WARNING: Statements terminations use 1 semicolon
WARNING: else is not generally useful after a break or return
WARNING: void function return statements are not generally useful
TEST=Build and run on Galileo Gen2
Change-Id: I6f095c4e9cb1ee4ff2ebdf095ef612e1a8393231
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18762
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/lib/imd.c')
-rw-r--r-- | src/lib/imd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/imd.c b/src/lib/imd.c index e3a3927500..8f65fa8285 100644 --- a/src/lib/imd.c +++ b/src/lib/imd.c @@ -439,7 +439,7 @@ int imd_create_tiered_empty(struct imd *imd, size_t lg_root_size, size_t lg_entry_align, size_t sm_root_size, size_t sm_entry_align) { - size_t sm_region_size;; + size_t sm_region_size; const struct imd_entry *e; struct imdr *imdr; |