[coreboot-gerrit] Change in coreboot[master]: util/inteltool: cleanup checkpatch.pl warnings

Pratikkumar V Prajapati (Code Review) gerrit at coreboot.org
Tue Jul 25 20:22:38 CEST 2017


Pratikkumar V Prajapati has uploaded this change for review. ( https://review.coreboot.org/20771


Change subject: util/inteltool: cleanup checkpatch.pl warnings
......................................................................

util/inteltool: cleanup checkpatch.pl warnings

Cleanup following checkpatch.pl warning
"WARNING: braces {} are not necessary for single statement blocks"

Change-Id: Id5a7dc3a9672266b66d7f46db2ff087b98fd174d
Signed-off-by: Pratik Prajapati <pratikkumar.v.prajapati at intel.com>
---
M util/inteltool/inteltool.c
1 file changed, 5 insertions(+), 10 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/20771/1

diff --git a/util/inteltool/inteltool.c b/util/inteltool/inteltool.c
index b997ee1..2f3fc7d 100644
--- a/util/inteltool/inteltool.c
+++ b/util/inteltool/inteltool.c
@@ -515,10 +515,9 @@
 	printf("Southbridge: %04x:%04x (%s)\n",
 		sb->vendor_id, sb->device_id, sbname);
 
-	if (gfx) {
+	if (gfx)
 		printf("IGD: %04x:%04x (%s)\n",
 		       gfx->vendor_id, gfx->device_id, gfxname);
-	}
 
 	/* Now do the deed */
 
@@ -565,21 +564,17 @@
 		printf("\n\n");
 	}
 
-	if (dump_ambs) {
+	if (dump_ambs)
 		print_ambs(nb, pacc);
-	}
 
-	if (dump_spi) {
+	if (dump_spi)
 		print_spi(sb);
-	}
 
-	if (dump_gfx) {
+	if (dump_gfx)
 		print_gfx(gfx);
-	}
 
-	if (dump_ahci) {
+	if (dump_ahci)
 		print_ahci(ahci);
-	}
 
 	if (dump_sgx)
 		print_sgx();

-- 
To view, visit https://review.coreboot.org/20771
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id5a7dc3a9672266b66d7f46db2ff087b98fd174d
Gerrit-Change-Number: 20771
Gerrit-PatchSet: 1
Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170725/4969fa5f/attachment.html>


More information about the coreboot-gerrit mailing list