UPSTREAM: 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: If08294e1369f06ca18d321d83b2ec4400f9e5891
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Original-Commit-Id: 3e1cab447b
Original-Change-Id: I6f095c4e9cb1ee4ff2ebdf095ef612e1a8393231
Original-Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Original-Reviewed-on: https://review.coreboot.org/18762
Original-Tested-by: build bot (Jenkins)
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/454571

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;
 

src/lib/jpeg.c

@@ -417,7 +417,6 @@ int jpeg_decode(unsigned char *buf, unsigned char *pic,
 				break;
 			default:
 				return ERR_DEPTH_MISMATCH;
-				break;
 			}
 		}
 	}

src/lib/prog_loaders.c

@@ -166,7 +166,6 @@ static struct prog global_payload =
 
 void __attribute__((weak)) mirror_payload(struct prog *payload)
 {
-	return;
 }
 
 void payload_load(void)

src/lib/ramtest.c

@@ -120,9 +120,8 @@ static int ram_bitset_nodie(unsigned long start)
 		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;
 }