[Compile Regression in 2.4.25-pre8][PATCH 2/42]

Kronos
2004-02-02T18:20:43Z

ad1889.c:361: warning: unsigned int format, different type arg (arg 4)

It happens only with CONFIG_HIGHMEM64G=y as dma_addr_t becomes u64.
Fixed by casting dma_addr_t to dma64_addr_t in the printk.

diff -Nru -X dontdiff linux-2.4-vanilla/drivers/sound/ad1889.c linux-2.4/drivers/sound/ad1889.c
--- linux-2.4-vanilla/drivers/sound/ad1889.c Tue Nov 11 17:51:39 2003
+++ linux-2.4/drivers/sound/ad1889.c Sat Jan 31 16:12:10 2004
@@ -356,9 +356,9 @@
for (i = 0; i < AD_MAX_STATES; i++) {
out += sprintf(out, "DMA status for %s:\n",
(i == AD_WAV_STATE ? "WAV" : "ADC"));
- out += sprintf(out, "\t\t0x%p (IOVA: 0x%u)\n",
+ out += sprintf(out, "\t\t0x%p (IOVA: 0x%Lu)\n",
dev->state[i].dmabuf.rawbuf,
- dev->state[i].dmabuf.dma_handle);
+ (dma64_addr_t)dev->state[i].dmabuf.dma_handle);

out += sprintf(out, "\tread ptr: offset %u\n",
(unsigned int)dev->state[i].dmabuf.rd_ptr);

--
Reply-To: kronos@kronoz.cjb.net
Home: http://kronoz.cjb.net
No matter what you choose, you're still a luser.