Subject: Re: linux-next: i2c tree build warning

From: Jean Delvare <[email-masked]>
Date: Thu, 26 Nov 2009 09:18:29 +0100

Hi Stephen,

On Thu, 26 Nov 2009 14:06:00 +1100, Stephen Rothwell wrote:
> Today's linux-next build (x86_64 allmodconfig) produced this warning:
> 
> drivers/misc/eeprom/at24.c: In function 'at24_eeprom_read':
> drivers/misc/eeprom/at24.c:230: warning: format '%zd' expects type 'signed size_t', but argument 6 has type 'int'
> 
> Introduced by commit 39bfab2519e6f148f2508d719f9fc5f4581aeefd ("at24: Use
> timeout also for read").

Thanks for reporting. The warning did not show on my 32-bit x86 system,
and I guess it did not show on Wolfram's system either, otherwise we'd
have fixed it.

Does the following patch fix the warning for you?

--- linux-2.6.32-rc8.orig/drivers/misc/eeprom/at24.c	2009-11-26 09:14:45.000000000 +0100
+++ linux-2.6.32-rc8/drivers/misc/eeprom/at24.c	2009-11-26 09:14:15.000000000 +0100
@@ -227,7 +227,7 @@ static ssize_t at24_eeprom_read(struct a
 			if (status == 2)
 				status = count;
 		}
-		dev_dbg(&client->dev, "read %zu@%d --> %zd (%ld)\n",
+		dev_dbg(&client->dev, "read %zu@%d --> %d (%ld)\n",
 				count, offset, status, jiffies);
 
 		if (status == count)

If it does then I'll merge this into Wolfram's patch.

Thanks,
-- 
Jean Delvare