| Submitter | Dan Carpenter |
|---|---|
| Date | March 22, 2010, 12:02 p.m. |
| Message ID | <20100322115819.GA21571@bicker> |
| Download | mbox | patch |
| Permalink | /patch/48291/ |
| State | New |
| Headers | show |
Comments
On Mon, 2010-03-22 at 15:02 +0300, Dan Carpenter wrote: > This gets rid of a compile warning: > drivers/mtd/maps/dmv182.c:120: warning: format ‘%d’ expects type > ‘int’, but argument 2 has type ‘uint64_t’ > > Signed-off-by: Dan Carpenter <error27@gmail.com> > > diff --git a/drivers/mtd/maps/dmv182.c b/drivers/mtd/maps/dmv182.c > index d171674..680ba2d 100644 > --- a/drivers/mtd/maps/dmv182.c > +++ b/drivers/mtd/maps/dmv182.c > @@ -116,7 +116,7 @@ static int __init init_svme182(void) > return -ENXIO; > } > > - printk(KERN_NOTICE "SVME182 flash device: %dMiB at 0x%08x\n", > + printk(KERN_NOTICE "SVME182 flash device: %lldMiB at 0x%08x\n", > this_mtd->size >> 20, FLASH_BASE_ADDR); %lld is also wrong for uint64_t, I think on PPC it'll result in a warning. Instead, I'd use %llu and then a cast: (unsigned long long)(this_mtd->size >> 20)
Patch
diff --git a/drivers/mtd/maps/dmv182.c b/drivers/mtd/maps/dmv182.c index d171674..680ba2d 100644 --- a/drivers/mtd/maps/dmv182.c +++ b/drivers/mtd/maps/dmv182.c @@ -116,7 +116,7 @@ static int __init init_svme182(void) return -ENXIO; } - printk(KERN_NOTICE "SVME182 flash device: %dMiB at 0x%08x\n", + printk(KERN_NOTICE "SVME182 flash device: %lldMiB at 0x%08x\n", this_mtd->size >> 20, FLASH_BASE_ADDR); this_mtd->owner = THIS_MODULE;
This gets rid of a compile warning: drivers/mtd/maps/dmv182.c:120: warning: format ‘%d’ expects type ‘int’, but argument 2 has type ‘uint64_t’ Signed-off-by: Dan Carpenter <error27@gmail.com>