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>
diff --git a/src/lib/ramtest.c b/src/lib/ramtest.c
index 806e7d7..419d0eb 100644
--- a/src/lib/ramtest.c
+++ b/src/lib/ramtest.c
@@ -120,9 +120,8 @@
post_code(0xea);
printk(BIOS_DEBUG, "\nDRAM did _NOT_ verify!\n");
return 1;
- } else {
- printk(BIOS_DEBUG, "\nDRAM range verified.\n");
}
+ printk(BIOS_DEBUG, "\nDRAM range verified.\n");
return 0;
}