[PATCH 60/69] staging: unisys: virthba: Fix missing braces at end of if-else statements

Benjamin Romer benjamin.romer at unisys.com
Fri Dec 5 22:09:35 UTC 2014


From: Ken Depro <kenneth.depro at unisys.com>

This patch fixes checkpatch warnings that resulted from the else portion of
if-else statements missing braces, so that it conforms with the rest of the
statement.

Signed-off-by: Ken Depro <kenneth.depro at unisys.com>
Signed-off-by: Benjamin Romer <benjamin.romer at unisys.com>
---
 drivers/staging/unisys/virthba/virthba.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/unisys/virthba/virthba.c b/drivers/staging/unisys/virthba/virthba.c
index ae73d11..4e56a0d 100644
--- a/drivers/staging/unisys/virthba/virthba.c
+++ b/drivers/staging/unisys/virthba/virthba.c
@@ -1304,8 +1304,9 @@ drain_queue(struct virthba_info *virthbainfo, struct chaninfo *dc,
 				   (uintptr_t)cmdrsp->vdiskmgmt.scsicmd))
 				break;
 			complete_vdiskmgmt_command(cmdrsp);
-		} else
+		} else {
 			pr_err("Invalid cmdtype %d\n", cmdrsp->cmdtype);
+		}
 		/* cmdrsp is now available for reuse */
 	}
 }
@@ -1586,8 +1587,9 @@ virthba_serverdown(struct virtpci_dev *virtpcidev, u32 state)
 	} else if (virthbainfo->serverchangingstate) {
 		pr_err("Server already processing change state message\n");
 		return 0;
-	} else
+	} else {
 		pr_err("Server already down, but another server down message received.");
+	}
 
 	return 1;
 }
-- 
2.1.0



More information about the devel mailing list