Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Linux: Kernel

linux-next: i2c tree build warning

 

 

Linux kernel RSS feed   Index | Next | Previous | View Threaded


sfr at canb

Nov 25, 2009, 7:06 PM

Post #1 of 4 (149 views)
Permalink
linux-next: i2c tree build warning

Hi Jean,

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").
--
Cheers,
Stephen Rothwell sfr [at] canb
http://www.canb.auug.org.au/~sfr/


khali at linux-fr

Nov 26, 2009, 12:18 AM

Post #2 of 4 (133 views)
Permalink
Re: linux-next: i2c tree build warning [In reply to]

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


sfr at canb

Nov 26, 2009, 12:50 AM

Post #3 of 4 (133 views)
Permalink
Re: linux-next: i2c tree build warning [In reply to]

Hi Jean,

On Thu, 26 Nov 2009 09:18:29 +0100 Jean Delvare <khali [at] linux-fr> wrote:
>
> 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?

Yes.

--
Cheers,
Stephen Rothwell sfr [at] canb
http://www.canb.auug.org.au/~sfr/


w.sang at pengutronix

Nov 26, 2009, 2:50 AM

Post #4 of 4 (136 views)
Permalink
Re: linux-next: i2c tree build warning [In reply to]

> 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.

Yes, I tested it on a few embedded boards which are all 32-bit. Will try to get
a 64-bit machine into my test-process. Sorry for this!

Thank you Stephen for reporting and Jean for helping out!

--
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Attachments: signature.asc (0.19 KB)

Linux kernel RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.