[coreboot-gerrit] Change in coreboot[master]: src: Remove unneeded whitespace before '\n'

Elyes HAOUAS (Code Review) gerrit at coreboot.org
Sat Jun 9 12:00:15 CEST 2018


Elyes HAOUAS has uploaded this change for review. ( https://review.coreboot.org/26991


Change subject: src: Remove unneeded whitespace before '\n'
......................................................................

src: Remove unneeded whitespace before '\n'

Change-Id: I630d49ab504d9f6e052806b516a600fa41b9a8da
Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
M src/drivers/intel/fsp2_0/hand_off_block.c
M src/mainboard/pcengines/apu2/romstage.c
M src/soc/amd/common/block/pi/def_callouts.c
M util/crossgcc/buildgcc
M util/romcc/romcc.c
M util/vgabios/device.c
M util/vgabios/testbios.c
7 files changed, 16 insertions(+), 16 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/91/26991/1

diff --git a/src/drivers/intel/fsp2_0/hand_off_block.c b/src/drivers/intel/fsp2_0/hand_off_block.c
index 1e0b5c8..05f7795 100644
--- a/src/drivers/intel/fsp2_0/hand_off_block.c
+++ b/src/drivers/intel/fsp2_0/hand_off_block.c
@@ -232,7 +232,7 @@
 			 (fvih->Count * sizeof (FIRMWARE_VERSION_INFO)));
 	size -= sizeof(SMBIOS_STRUCTURE);
 
-	printk(BIOS_DEBUG, "Display FSP Version Info HOB \n");
+	printk(BIOS_DEBUG, "Display FSP Version Info HOB\n");
 	for (index = 0; index < fvih->Count; index++) {
 		cnt = strlen(str_ptr);
 
diff --git a/src/mainboard/pcengines/apu2/romstage.c b/src/mainboard/pcengines/apu2/romstage.c
index 093cad6..7ea89b8 100644
--- a/src/mainboard/pcengines/apu2/romstage.c
+++ b/src/mainboard/pcengines/apu2/romstage.c
@@ -74,8 +74,8 @@
 
 	/* Load MPB */
 	val = cpuid_eax(1);
-	printk(BIOS_DEBUG, "BSP Family_Model: %08x \n", val);
-	printk(BIOS_DEBUG, "cpu_init_detectedx = %08lx \n", cpu_init_detectedx);
+	printk(BIOS_DEBUG, "BSP Family_Model: %08x\n", val);
+	printk(BIOS_DEBUG, "cpu_init_detectedx = %08lx\n", cpu_init_detectedx);
 
 	post_code(0x37);
 	AGESAWRAPPER(amdinitreset);
diff --git a/src/soc/amd/common/block/pi/def_callouts.c b/src/soc/amd/common/block/pi/def_callouts.c
index 2c17a3f..c01cceb 100644
--- a/src/soc/amd/common/block/pi/def_callouts.c
+++ b/src/soc/amd/common/block/pi/def_callouts.c
@@ -196,7 +196,7 @@
 {
 	AGESA_STATUS Status = AGESA_UNSUPPORTED;
 
-	printk(BIOS_DEBUG, "%s Func: 0x%x,  Data: 0x%lx, Ptr: 0x%p \n",
+	printk(BIOS_DEBUG, "%s Func: 0x%x,  Data: 0x%lx, Ptr: 0x%p\n",
 		__func__, agesadata.Func, agesadata.Data, agesadata.ConfigPtr);
 
 	/* Check if this AP should run the function */
diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index bbe74eb..7a8aee0 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -1191,7 +1191,7 @@
 
 # Download, unpack, patch and build all packages
 
-printf "Downloading and verifing tarballs ... \n"
+printf "Downloading and verifing tarballs ...\n"
 mkdir -p tarballs
 for P in $PACKAGES; do
 	download "$P" || exit "$?"
@@ -1199,21 +1199,21 @@
 done
 printf "Downloaded tarballs ... ${green}ok${NC}\n"
 
-printf "Unpacking and patching ... \n"
+printf "Unpacking and patching ...\n"
 for P in $PACKAGES; do
 	unpack_and_patch $P || exit 1
 done
 printf "Unpacked and patched ... ${green}ok${NC}\n"
 
 if [ -n "$BOOTSTRAPONLY" ]; then
-	printf "Building bootstrap compiler only ... \n"
+	printf "Building bootstrap compiler only ...\n"
 	for pkg in GMP MPFR MPC GCC; do
 		build_for_host $pkg
 	done
 	exit 0
 fi
 
-printf "Building packages ... \n"
+printf "Building packages ...\n"
 for package in $PACKAGES; do
 	build $package
 done
diff --git a/util/romcc/romcc.c b/util/romcc/romcc.c
index 9a7d1a2..ebea26f 100644
--- a/util/romcc/romcc.c
+++ b/util/romcc/romcc.c
@@ -9199,7 +9199,7 @@
 
 		}
 #if DEBUG_DECOMPOSE_HIRES
-		fprintf(fp, "decompose next: %p \n", next);
+		fprintf(fp, "decompose next: %p\n", next);
 		fflush(fp);
 		fprintf(fp, "next->op: %d %s\n",
 			next->op, tops(next->op));
@@ -14284,7 +14284,7 @@
 #if DEBUG_EXPLICIT_CLOSURES
 	/* Print out the variables to be enclosed */
 	loc(state->dbgout, state, fcall);
-	fprintf(state->dbgout, "Alive: \n");
+	fprintf(state->dbgout, "Alive:\n");
 	for(set = *enclose; set; set = set->next) {
 		display_triple(state->dbgout, set->member);
 	}
@@ -25048,7 +25048,7 @@
 	fprintf(fp,
 		"\nUsage: romcc [options] <source>.c\n"
 		"Compile a C source file generating a binary that does not implicilty use RAM\n"
-		"Options: \n"
+		"Options:\n"
 		"-o <output file name>\n"
 		"-f<option>            Specify a generic compiler option\n"
 		"-m<option>            Specify a arch dependent option\n"
diff --git a/util/vgabios/device.c b/util/vgabios/device.c
index 3526e24..55141e5 100644
--- a/util/vgabios/device.c
+++ b/util/vgabios/device.c
@@ -80,7 +80,7 @@
 	taa_index++;
 	/* legacy ranges if its a VGA card... */
 	if ((bios_device.dev->class & 0xFF0000) == 0x030000) {
-		DEBUG_PRINTF("%s: VGA device found, adding legacy resources... \n", __func__);
+		DEBUG_PRINTF("%s: VGA device found, adding legacy resources...\n", __func__);
 		/* I/O 0x3B0-0x3BB */
 		translate_address_array[taa_index].info = IORESOURCE_FIXED | IORESOURCE_IO;
 		translate_address_array[taa_index].bus = bus;
@@ -116,7 +116,7 @@
 	taa_last_entry = taa_index - 1;
 #if CONFIG_X86EMU_DEBUG
 	//dump translate_address_array
-	printf("translate_address_array: \n");
+	printf("translate_address_array:\n");
 	translate_address_t ta;
 	int i;
 	for (i = 0; i <= taa_last_entry; i++) {
@@ -201,7 +201,7 @@
 	taa_last_entry = taa_index - 1;
 #if CONFIG_X86EMU_DEBUG
 	//dump translate_address_array
-	printf("translate_address_array: \n");
+	printf("translate_address_array:\n");
 	translate_address_t ta;
 	for (i = 0; i <= taa_last_entry; i++) {
 		ta = translate_address_array[i];
diff --git a/util/vgabios/testbios.c b/util/vgabios/testbios.c
index ba63ab3..0ad96aa 100644
--- a/util/vgabios/testbios.c
+++ b/util/vgabios/testbios.c
@@ -116,7 +116,7 @@
 static void usage(char *name)
 {
 	printf
-	    ("Usage: %s [-c codesegment] [-s size] [-b base] [-i ip] [-t] <filename> ... \n",
+	    ("Usage: %s [-c codesegment] [-s size] [-b base] [-i ip] [-t] <filename> ...\n",
 	     name);
 }
 
@@ -218,7 +218,7 @@
 			debugflag = strtol(optarg, 0, 0);
 			break;
 		default:
-			printf("Unknown option \n");
+			printf("Unknown option\n");
 			usage(argv[0]);
 			return 1;
 		}

-- 
To view, visit https://review.coreboot.org/26991
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I630d49ab504d9f6e052806b516a600fa41b9a8da
Gerrit-Change-Number: 26991
Gerrit-PatchSet: 1
Gerrit-Owner: Elyes HAOUAS <ehaouas at noos.fr>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180609/54a3458d/attachment.html>


More information about the coreboot-gerrit mailing list