commit 9c7c6f7213decfc0d0fee4bbc911a291ee93bcdb
author Lee Leahy <leroy.p.leahy@intel.com> Thu Mar 16 11:24:09 2017 -0700
committer Lee Leahy <leroy.p.leahy@intel.com> Fri Mar 17 03:12:39 2017 +0100
tree d9e23ea1f89f75c6e178734bbe375860515bf3e2
parent d94cff6ab26d482554309041a9317cc3bf5e4b02

arch/x86: Fix issues with braces detected by checkpatch

Fix the following errors and warnings detected by checkpatch.pl:

ERROR: open brace '{' following function declarations go on the next line
ERROR: that open brace { should be on the previous line
ERROR: else should follow close brace '}'
WARNING: braces {} are not necessary for any arm of this statement
WARNING: braces {} are not necessary for single statement blocks

TEST=Build and run on Galileo Gen2

Change-Id: I13d1967757e106c8300a15baed25d920c52a1a95
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18861
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>

src/arch/x86/acpigen.c

diff --git a/src/arch/x86/acpigen.c b/src/arch/x86/acpigen.c
index d3ec05f..7e4775c 100644
--- a/src/arch/x86/acpigen.c
+++ b/src/arch/x86/acpigen.c
@@ -204,9 +204,8 @@
 void acpigen_emit_stream(const char *data, int size)
 {
 	int i;
-	for (i = 0; i < size; i++) {
+	for (i = 0; i < size; i++)
 		acpigen_emit_byte(data[i]);
-	}
 }
 
 void acpigen_emit_string(const char *string)
@@ -238,7 +237,8 @@
  * Check sections 5.3, 18.2.2 and 18.4 of ACPI spec 3.0 for details.
  */
 
-static void acpigen_emit_simple_namestring(const char *name) {
+static void acpigen_emit_simple_namestring(const char *name)
+{
 	int i;
 	char ud[] = "____";
 	for (i = 0; i < 4; i++) {
@@ -251,13 +251,15 @@
 	}
 }
 
-static void acpigen_emit_double_namestring(const char *name, int dotpos) {
+static void acpigen_emit_double_namestring(const char *name, int dotpos)
+{
 	acpigen_emit_byte(DUAL_NAME_PREFIX);
 	acpigen_emit_simple_namestring(name);
 	acpigen_emit_simple_namestring(&name[dotpos + 1]);
 }
 
-static void acpigen_emit_multi_namestring(const char *name) {
+static void acpigen_emit_multi_namestring(const char *name)
+{
 	int count = 0;
 	unsigned char *pathlen;
 	acpigen_emit_byte(MULTI_NAME_PREFIX);
@@ -279,7 +281,8 @@
 }
 
 
-void acpigen_emit_namestring(const char *namepath) {
+void acpigen_emit_namestring(const char *namepath)
+{
 	int dotcount = 0, i;
 	int dotpos = 0;
 
@@ -311,13 +314,12 @@
 		i++;
 	}
 
-	if (dotcount == 0) {
+	if (dotcount == 0)
 		acpigen_emit_simple_namestring(namepath);
-	} else if (dotcount == 1) {
+	else if (dotcount == 1)
 		acpigen_emit_double_namestring(namepath, dotpos);
-	} else {
+	else
 		acpigen_emit_multi_namestring(namepath);
-	}
 }
 
 void acpigen_write_name(const char *name)