[coreboot-gerrit] Patch set updated for coreboot: soc/apollolake/lpc_lib: Make cros compile pass

Lijian Zhao (lijian.zhao@intel.com) gerrit at coreboot.org
Thu May 19 03:45:58 CEST 2016


Lijian Zhao (lijian.zhao at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14835

-gerrit

commit a3dfcf60fc3b254f40f71446898eb2df588bbfe5
Author: Zhao, Lijian <lijian.zhao at intel.com>
Date:   Mon May 16 11:41:41 2016 -0700

    soc/apollolake/lpc_lib: Make cros compile pass
    
    The print of size_t can pass upstream jenkins, but fails with CROS_SDK
    enviornment, "%z" fits for size_t anyway.
    
    Change-Id: Ic8dbab240463f2e484b73d55e21985fae2b0d9b7
    Signed-off-by: Lijian Zhao <lijian.zhao at intel.com>
---
 src/soc/intel/apollolake/lpc_lib.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/soc/intel/apollolake/lpc_lib.c b/src/soc/intel/apollolake/lpc_lib.c
index 654c83e..9428866 100644
--- a/src/soc/intel/apollolake/lpc_lib.c
+++ b/src/soc/intel/apollolake/lpc_lib.c
@@ -149,7 +149,7 @@ void lpc_open_mmio_window(uintptr_t base, size_t size)
 
 	if (lgmr & LGMR_EN) {
 		printk(BIOS_ERR,
-		       "LPC: Cannot open window to resource %lx size %lx\n",
+		       "LPC: Cannot open window to resource %lx size %zx\n",
 		       base, size);
 		printk(BIOS_ERR, "LPC: MMIO window already in use\n");
 		return;
@@ -157,7 +157,7 @@ void lpc_open_mmio_window(uintptr_t base, size_t size)
 
 	if (size > LGMR_WINDOW_SIZE) {
 		printk(BIOS_WARNING,
-		       "LPC:  Resource %lx size %lx larger than window(%x)\n",
+		       "LPC:  Resource %lx size %zx larger than window(%x)\n",
 		       base, size, LGMR_WINDOW_SIZE);
 	}
 
@@ -177,8 +177,8 @@ bool lpc_fits_fixed_mmio_window(uintptr_t base, size_t size)
 
 		if ((base >= range->base) && (res_end <= range_end)) {
 			printk(BIOS_DEBUG,
-			       "Resource %lx size %lx fits in fixed window"
-			       " %lx size %lx\n",
+			       "Resource %lx size %zx fits in fixed window"
+			       " %lx size %zx\n",
 			       base, size, range->base, range->size);
 			return true;
 		}



More information about the coreboot-gerrit mailing list