commit d7fc0688e526fcdf2107a05171b1846dde3b3edb
author Arthur Heymans <arthur@aheymans.xyz> Tue Nov 08 15:31:09 2022 +0100
committer Martin L Roth <gaumless@gmail.com> Mon Jan 09 06:19:11 2023 +0000
tree fa14864add16df90435a8a8f13b4f6f177720581
parent b5445ade38e5129ee19ddd97f4b3687fa28afd4b

sb/intel/common/spi: Fix building for 64bit

This avoids the warning of casting pointers to integers of different
size.

Change-Id: I7bcb6dbf286438115c854d618eaa2da21c81400d
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/69389
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin L Roth <gaumless@gmail.com>

src/southbridge/intel/common/spi.c

diff --git a/src/southbridge/intel/common/spi.c b/src/southbridge/intel/common/spi.c
index 1d274e8..75e9b21 100644
--- a/src/southbridge/intel/common/spi.c
+++ b/src/southbridge/intel/common/spi.c
@@ -157,8 +157,8 @@
 {
 	u8 v = read8(addr);
 
-	printk(BIOS_DEBUG, "read %2.2x from %4.4x\n",
-	       v, ((unsigned int)addr & 0xffff) - 0xf020);
+	printk(BIOS_DEBUG, "read %2.2x from %4.4lx\n",
+	       v, ((uintptr_t)addr & 0xffff) - 0xf020);
 	return v;
 }
 
@@ -166,8 +166,8 @@
 {
 	u16 v = read16(addr);
 
-	printk(BIOS_DEBUG, "read %4.4x from %4.4x\n",
-	       v, ((unsigned int)addr & 0xffff) - 0xf020);
+	printk(BIOS_DEBUG, "read %4.4x from %4.4lx\n",
+	       v, ((uintptr_t)addr & 0xffff) - 0xf020);
 	return v;
 }
 
@@ -175,30 +175,30 @@
 {
 	u32 v = read32(addr);
 
-	printk(BIOS_DEBUG, "read %8.8x from %4.4x\n",
-	       v, ((unsigned int)addr & 0xffff) - 0xf020);
+	printk(BIOS_DEBUG, "read %8.8x from %4.4lx\n",
+	       v, ((uintptr_t)addr & 0xffff) - 0xf020);
 	return v;
 }
 
 static void writeb_(u8 b, void *addr)
 {
 	write8(addr, b);
-	printk(BIOS_DEBUG, "wrote %2.2x to %4.4x\n",
-	       b, ((unsigned int)addr & 0xffff) - 0xf020);
+	printk(BIOS_DEBUG, "wrote %2.2x to %4.4lx\n",
+	       b, ((uintptr_t)addr & 0xffff) - 0xf020);
 }
 
 static void writew_(u16 b, void *addr)
 {
 	write16(addr, b);
-	printk(BIOS_DEBUG, "wrote %4.4x to %4.4x\n",
-	       b, ((unsigned int)addr & 0xffff) - 0xf020);
+	printk(BIOS_DEBUG, "wrote %4.4x to %4.4lx\n",
+	       b, ((uintptr_t)addr & 0xffff) - 0xf020);
 }
 
 static void writel_(u32 b, void *addr)
 {
 	write32(addr, b);
-	printk(BIOS_DEBUG, "wrote %8.8x to %4.4x\n",
-	       b, ((unsigned int)addr & 0xffff) - 0xf020);
+	printk(BIOS_DEBUG, "wrote %8.8x to %4.4lx\n",
+	       b, ((uintptr_t)addr & 0xffff) - 0xf020);
 }
 
 #else /* CONFIG_DEBUG_SPI_FLASH ^^^ enabled  vvv NOT enabled */