[coreboot-gerrit] New patch to review for coreboot: 2c52d6a inteltool: use inttypes for prints in memory.c

Stefan Tauner (stefan.tauner@gmx.at) gerrit at coreboot.org
Fri Apr 5 01:15:52 CEST 2013


Stefan Tauner (stefan.tauner at gmx.at) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3024

-gerrit

commit 2c52d6afacf60278f3f3952178adbe49fa536191
Author: Stefan Tauner <stefan.tauner at gmx.at>
Date:   Fri Apr 5 01:15:04 2013 +0200

    inteltool: use inttypes for prints in memory.c
    
    This fixes at least one warning on my machine where "llx" is replaced by PRIx64.
    
    Change-Id: Iee3e5027d327d4d5f8e6d8b2d53d051f74bfc354
    Signed-off-by: Stefan Tauner <stefan.tauner at gmx.at>
---
 util/inteltool/memory.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/util/inteltool/memory.c b/util/inteltool/memory.c
index 6847e5c..506620d 100644
--- a/util/inteltool/memory.c
+++ b/util/inteltool/memory.c
@@ -197,25 +197,25 @@ int print_mchbar(struct pci_dev *nb, struct pci_access *pacc)
 		for (i = 0; i < size; i++) {
 			switch (mch_registers[i].size) {
 				case 8:
-					printf("mchbase+0x%04x: 0x%016llx (%s)\n",
+					printf("mchbase+0x%04x: 0x%016"PRIx64" (%s)\n",
 						mch_registers[i].addr,
 						*(uint64_t *)(mchbar+mch_registers[i].addr),
 						mch_registers[i].name);
 					break;
 				case 4:
-					printf("mchbase+0x%04x: 0x%08x         (%s)\n",
+					printf("mchbase+0x%04x: 0x%08"PRIx32"         (%s)\n",
 						mch_registers[i].addr,
 						*(uint32_t *)(mchbar+mch_registers[i].addr),
 						mch_registers[i].name);
 					break;
 				case 2:
-					printf("mchbase+0x%04x: 0x%04x             (%s)\n",
+					printf("mchbase+0x%04x: 0x%04"PRIx16"             (%s)\n",
 						mch_registers[i].addr,
 						*(uint16_t *)(mchbar+mch_registers[i].addr),
 						mch_registers[i].name);
 					break;
 				case 1:
-					printf("mchbase+0x%04x: 0x%02x               (%s)\n",
+					printf("mchbase+0x%04x: 0x%02"PRIx8"               (%s)\n",
 						mch_registers[i].addr,
 						*(uint8_t *)(mchbar+mch_registers[i].addr),
 						mch_registers[i].name);
@@ -225,7 +225,7 @@ int print_mchbar(struct pci_dev *nb, struct pci_access *pacc)
 	} else {
 		for (i = 0; i < size; i += 4) {
 			if (*(uint32_t *)(mchbar + i))
-				printf("0x%04x: 0x%08x\n", i, *(uint32_t *)(mchbar+i));
+				printf("0x%04x: 0x%08"PRIx32"\n", i, *(uint32_t *)(mchbar+i));
 		}
 	}
 



More information about the coreboot-gerrit mailing list