V4L/DVB (12924): SAA7164: Fix some 32/64bit compile time warnings

SAA7164: Fix some 32/64bit compile time warnings

Signed-off-by: Steven Toth <stoth@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

diff --git a/drivers/media/video/saa7164/saa7164-api.c b/drivers/media/video/saa7164/saa7164-api.c
index 8cef1df..105b68e 100644 (file)
--- a/drivers/media/video/saa7164/saa7164-api.c
+++ b/drivers/media/video/saa7164/saa7164-api.c
@@ -129,8 +129,8 @@ int saa7164_api_dump_subdevs(struct saa7164_dev *dev, u8 *buf, int len)
        u32 currpath = 0;
 
        dprintk(DBGLVL_API,
-               "%s(?,?,%d) sizeof(tmComResDescrHeader_t) = %lu bytes\n",
-               __func__, len, sizeof(tmComResDescrHeader_t));
+               "%s(?,?,%d) sizeof(tmComResDescrHeader_t) = %d bytes\n",
+               __func__, len, (u32)sizeof(tmComResDescrHeader_t));
 
        for (idx = 0; idx < (len - sizeof(tmComResDescrHeader_t)); ) {
 

diff --git a/drivers/media/video/saa7164/saa7164-buffer.c b/drivers/media/video/saa7164/saa7164-buffer.c
index 4176544..240c6dc 100644 (file)
--- a/drivers/media/video/saa7164/saa7164-buffer.c
+++ b/drivers/media/video/saa7164/saa7164-buffer.c
@@ -107,19 +107,16 @@ struct saa7164_buffer *saa7164_buffer_alloc(struct saa7164_tsport *port,
        memset(buf->pt_cpu, 0xff, buf->pt_size);
 
        dprintk(DBGLVL_BUF, "%s()   allocated buffer @ 0x%p\n", __func__, buf);
-       dprintk(DBGLVL_BUF, "  pci_cpu @ 0x%llx    dma @ 0x%llx len = 0x%x\n",
-               (u64)buf->cpu, (u64)buf->dma, buf->pci_size);
-       dprintk(DBGLVL_BUF, "   pt_cpu @ 0x%llx pt_dma @ 0x%llx len = 0x%x\n",
-               (u64)buf->pt_cpu, (u64)buf->pt_dma, buf->pt_size);
+       dprintk(DBGLVL_BUF, "  pci_cpu @ 0x%p    dma @ 0x%p len = 0x%x\n",
+               buf->cpu, (void *)buf->dma, buf->pci_size);
+       dprintk(DBGLVL_BUF, "   pt_cpu @ 0x%p pt_dma @ 0x%p len = 0x%x\n",
+               buf->pt_cpu, (void *)buf->pt_dma, buf->pt_size);
 
        /* Format the Page Table Entries to point into the data buffer */
        for (i = 0 ; i < SAA7164_PT_ENTRIES; i++) {
 
                *(buf->pt_cpu + i) = buf->dma + (i * 0x1000); /* TODO */
 
-               dprintk(DBGLVL_BUF, "    pt[%02d] = 0x%llx -> 0x%llx\n",
-                       i, (u64)buf->pt_cpu, (u64)*(buf->pt_cpu));
-
        }
 
        goto ret;

diff --git a/drivers/media/video/saa7164/saa7164-bus.c b/drivers/media/video/saa7164/saa7164-bus.c
index 28f630d..8d813f5 100644 (file)
--- a/drivers/media/video/saa7164/saa7164-bus.c
+++ b/drivers/media/video/saa7164/saa7164-bus.c
@@ -210,8 +210,8 @@ int saa7164_bus_set(struct saa7164_dev *dev, tmComResInfo_t* msg, void *buf)
                dprintk(DBGLVL_BUS, "%s() space_rem = %x\n", __func__,
                        space_rem);
 
-               dprintk(DBGLVL_BUS, "%s() sizeof(*msg) = %lu\n", __func__,
-                       sizeof(*msg));
+               dprintk(DBGLVL_BUS, "%s() sizeof(*msg) = %d\n", __func__,
+                       (u32)sizeof(*msg));
 
                if (space_rem < sizeof(*msg)) {
                        dprintk(DBGLVL_BUS, "%s() tr4\n", __func__);

diff --git a/drivers/media/video/saa7164/saa7164-core.c b/drivers/media/video/saa7164/saa7164-core.c
index 0495709..4b155a3 100644 (file)
--- a/drivers/media/video/saa7164/saa7164-core.c
+++ b/drivers/media/video/saa7164/saa7164-core.c
@@ -281,8 +281,8 @@ void saa7164_dumpregs(struct saa7164_dev *dev, u32 addr)
 
 static void saa7164_dump_hwdesc(struct saa7164_dev *dev)
 {
-       dprintk(1, "@0x%p hwdesc sizeof(tmComResHWDescr_t) = %lu bytes\n",
-               &dev->hwdesc, sizeof(tmComResHWDescr_t));
+       dprintk(1, "@0x%p hwdesc sizeof(tmComResHWDescr_t) = %d bytes\n",
+               &dev->hwdesc, (u32)sizeof(tmComResHWDescr_t));
 
        dprintk(1, " .bLength = 0x%x\n", dev->hwdesc.bLength);
        dprintk(1, " .bDescriptorType = 0x%x\n", dev->hwdesc.bDescriptorType);
@@ -312,8 +312,8 @@ static void saa7164_dump_hwdesc(struct saa7164_dev *dev)
 static void saa7164_dump_intfdesc(struct saa7164_dev *dev)
 {
        dprintk(1, "@0x%p intfdesc "
-               "sizeof(tmComResInterfaceDescr_t) = %lu bytes\n",
-               &dev->intfdesc, sizeof(tmComResInterfaceDescr_t));
+               "sizeof(tmComResInterfaceDescr_t) = %d bytes\n",
+               &dev->intfdesc, (u32)sizeof(tmComResInterfaceDescr_t));
 
        dprintk(1, " .bLength = 0x%x\n", dev->intfdesc.bLength);
        dprintk(1, " .bDescriptorType = 0x%x\n", dev->intfdesc.bDescriptorType);
@@ -333,8 +333,8 @@ static void saa7164_dump_intfdesc(struct saa7164_dev *dev)
 
 static void saa7164_dump_busdesc(struct saa7164_dev *dev)
 {
-       dprintk(1, "@0x%p busdesc sizeof(tmComResBusDescr_t) = %lu bytes\n",
-               &dev->busdesc, sizeof(tmComResBusDescr_t));
+       dprintk(1, "@0x%p busdesc sizeof(tmComResBusDescr_t) = %d bytes\n",
+               &dev->busdesc, (u32)sizeof(tmComResBusDescr_t));
 
        dprintk(1, " .CommandRing   = 0x%016Lx\n", dev->busdesc.CommandRing);
        dprintk(1, " .ResponseRing  = 0x%016Lx\n", dev->busdesc.ResponseRing);
@@ -359,15 +359,15 @@ static void saa7164_get_descriptors(struct saa7164_dev *dev)
 
        if (dev->hwdesc.bLength != sizeof(tmComResHWDescr_t)) {
                printk(KERN_ERR "Structure tmComResHWDescr_t is mangled\n");
-               printk(KERN_ERR "Need %x got %lu\n", dev->hwdesc.bLength,
-                       sizeof(tmComResHWDescr_t));
+               printk(KERN_ERR "Need %x got %d\n", dev->hwdesc.bLength,
+                       (u32)sizeof(tmComResHWDescr_t));
        } else
                saa7164_dump_hwdesc(dev);
 
        if (dev->intfdesc.bLength != sizeof(tmComResInterfaceDescr_t)) {
                printk(KERN_ERR "struct tmComResInterfaceDescr_t is mangled\n");
-               printk(KERN_ERR "Need %x got %lu\n", dev->intfdesc.bLength,
-                       sizeof(tmComResInterfaceDescr_t));
+               printk(KERN_ERR "Need %x got %d\n", dev->intfdesc.bLength,
+                       (u32)sizeof(tmComResInterfaceDescr_t));
        } else
                saa7164_dump_intfdesc(dev);
 

diff --git a/drivers/media/video/saa7164/saa7164-dvb.c b/drivers/media/video/saa7164/saa7164-dvb.c
index f21520f..271962d 100644 (file)
--- a/drivers/media/video/saa7164/saa7164-dvb.c
+++ b/drivers/media/video/saa7164/saa7164-dvb.c
@@ -151,7 +151,7 @@ static int saa7164_dvb_cfg_tsport(struct saa7164_tsport *port)
        saa7164_writel(port->bufsize, params->pitch * params->numberoflines);
 
        dprintk(DBGLVL_DVB, " configured:\n");
-       dprintk(DBGLVL_DVB, "   lmmio       0x%llx\n", (u64)dev->lmmio);
+       dprintk(DBGLVL_DVB, "   lmmio       0x%p\n", dev->lmmio);
        dprintk(DBGLVL_DVB, "   bufcounter  0x%x = 0x%x\n", port->bufcounter,
                saa7164_readl(port->bufcounter));
 
@@ -178,13 +178,13 @@ static int saa7164_dvb_cfg_tsport(struct saa7164_tsport *port)
                saa7164_writel(port->bufptr32l + ((sizeof(u32) * 2) * i), 0);
 
                dprintk(DBGLVL_DVB,
-                       "   buf[%d] offset 0x%lx (0x%x) "
-                       "buf 0x%lx/%lx (0x%x/%x)\n",
+                       "   buf[%d] offset 0x%llx (0x%x) "
+                       "buf 0x%llx/%llx (0x%x/%x)\n",
                        i,
-                       port->bufoffset + (i * sizeof(u32)),
+                       (u64)port->bufoffset + (i * sizeof(u32)),
                        saa7164_readl(port->bufoffset + (sizeof(u32) * i)),
-                       port->bufptr32h + ((sizeof(u32) * 2) * i),
-                       port->bufptr32l + ((sizeof(u32) * 2) * i),
+                       (u64)port->bufptr32h + ((sizeof(u32) * 2) * i),
+                       (u64)port->bufptr32l + ((sizeof(u32) * 2) * i),
                        saa7164_readl(port->bufptr32h + ((sizeof(u32) * i)
                                * 2)),
                        saa7164_readl(port->bufptr32l + ((sizeof(u32) * i)

diff --git a/drivers/media/video/saa7164/saa7164.h b/drivers/media/video/saa7164/saa7164.h
index ed38118..338804f 100644 (file)
--- a/drivers/media/video/saa7164/saa7164.h
+++ b/drivers/media/video/saa7164/saa7164.h
@@ -390,11 +390,8 @@ extern unsigned int debug;
        } while (0)
 
 #define saa7164_readl(reg) readl(dev->lmmio + ((reg) >> 2))
-#define saa7164_writel(reg, value) \
-do { \
-       printk(KERN_ERR "writel(%x, %llx)\n", value, (u64)(dev->lmmio + ((reg) >> 2))); \
-       writel((value), dev->lmmio + ((reg) >> 2)); \
-} while (0)
+#define saa7164_writel(reg, value) writel((value), dev->lmmio + ((reg) >> 2))
+
 
 #define saa7164_readb(reg)             readl(dev->bmmio + (reg))
 #define saa7164_writeb(reg, value)     writel((value), dev->bmmio + (reg))