Change in flashrom[master]: serprog.c: Use braces in both branches of conditional statement

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has submitted this change. ( https://review.coreboot.org/c/flashrom/+/54862 )

Change subject: serprog.c: Use braces in both branches of conditional statement
......................................................................

serprog.c: Use braces in both branches of conditional statement

As per the coding style, if one branch of a conditional statement
needs braces, all other branches need to have braces as well.

BUG=b:185191942
TEST=builds

Change-Id: I4fdccbd66f0351af98811bf7c2d8e15f0a99d852
Signed-off-by: Anastasia Klimchuk 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/54862
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
Reviewed-by: Edward O'Callaghan 
---
M serprog.c
1 file changed, 13 insertions(+), 9 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved
  Edward O'Callaghan: Looks good to me, approved

diff --git a/serprog.c b/serprog.c
index a70a289..e8c0d75 100644
--- a/serprog.c
+++ b/serprog.c
@@ -564,8 +564,9 @@
    	if (baud_str == NULL || *baud_str == '\0') {
    		baud = -1;
    		msg_pdbg("No baudrate specified, using the hardware's defaults.\n");
-		} else
+		} else {
    		baud = atoi(baud_str); // FIXME: replace atoi with strtoul
+		}
    	if (strlen(device) > 0) {
    		sp_fd = sp_openserport(device, baud);
    		if (sp_fd == SER_INV_FD) {
@@ -727,11 +728,11 @@
    			goto init_err_cleanup_exit;
    		}
    		if (strlen(f_spi_suffix) == 1) {
-				if (!strcasecmp(f_spi_suffix, "M"))
+				if (!strcasecmp(f_spi_suffix, "M")) {
    				f_spi_req *= 1000000;
-				else if (!strcasecmp(f_spi_suffix, "k"))
+				} else if (!strcasecmp(f_spi_suffix, "k")) {
    				f_spi_req *= 1000;
-				else {
+				} else {
    				msg_perr("Error: Garbage following 'spispeed' value.\n");
    				free(spispeed);
    				goto init_err_cleanup_exit;
@@ -747,17 +748,18 @@
    		buf[2] = (f_spi_req >> (2 * 8)) & 0xFF;
    		buf[3] = (f_spi_req >> (3 * 8)) & 0xFF;

-			if (sp_check_commandavail(S_CMD_S_SPI_FREQ) == 0)
+			if (sp_check_commandavail(S_CMD_S_SPI_FREQ) == 0) {
    			msg_pwarn(MSGHEADER "Warning: Setting the SPI clock rate is not supported!\n");
-			else if (sp_docommand(S_CMD_S_SPI_FREQ, 4, buf, 4, buf) == 0) {
+			} else if (sp_docommand(S_CMD_S_SPI_FREQ, 4, buf, 4, buf) == 0) {
    			f_spi = buf[0];
    			f_spi |= buf[1] << (1 * 8);
    			f_spi |= buf[2] << (2 * 8);
    			f_spi |= buf[3] << (3 * 8);
    			msg_pdbg(MSGHEADER "Requested to set SPI clock frequency to %u Hz. "
    				 "It was actually set to %u Hz\n", f_spi_req, f_spi);
-			} else
+			} else {
    			msg_pwarn(MSGHEADER "Setting SPI clock rate to %u Hz failed!\n", f_spi_req);
+			}
    	}
    	free(spispeed);
    	bt = serprog_buses_supported;
@@ -871,10 +873,12 @@
    	if (sp_docommand(S_CMD_S_PIN_STATE, 1, &en, 0, NULL) != 0) {
    		msg_perr("Error: could not enable output buffers\n");
    		goto init_err_cleanup_exit;
-		} else
+		} else {
    		msg_pdbg(MSGHEADER "Output drivers enabled\n");
-	} else
+		}
+	} else {
    	msg_pdbg(MSGHEADER "Warning: Programmer does not support toggling its output drivers\n");
+	}

sp_prev_was_write = 0;
    sp_streamed_transmit_ops = 0;