Re: [PATCH] mtd: mtdram: fix build error

From: kbuild test robot
Date: Wed Sep 30 2015 - 04:50:46 EST


Hi Sudip,

[auto build test results on next-20150930 -- if it's inappropriate base, please ignore]

config: cris-etrax-100lx_v2_defconfig (attached as .config)
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout f2fe6774b45537be1dca5fee8e89943524bc777b
# save the attached .config to linux build tree
make.cross ARCH=cris

All warnings (new ones prefixed by >>):

drivers/mtd/devices/mtdram.c: In function 'check_offs_len':
>> drivers/mtd/devices/mtdram.c:42:8: warning: comparison of distinct pointer types lacks a cast [enabled by default]
>> drivers/mtd/devices/mtdram.c:42:2: warning: right shift count >= width of type [enabled by default]
>> drivers/mtd/devices/mtdram.c:42:2: warning: passing argument 1 of '__div64_32' from incompatible pointer type [enabled by default]
include/asm-generic/div64.h:35:17: note: expected 'uint64_t *' but argument is of type 'long unsigned int *'
drivers/mtd/devices/mtdram.c:50:8: warning: comparison of distinct pointer types lacks a cast [enabled by default]
drivers/mtd/devices/mtdram.c:50:2: warning: right shift count >= width of type [enabled by default]
drivers/mtd/devices/mtdram.c:50:2: warning: passing argument 1 of '__div64_32' from incompatible pointer type [enabled by default]
include/asm-generic/div64.h:35:17: note: expected 'uint64_t *' but argument is of type 'long unsigned int *'

vim +42 drivers/mtd/devices/mtdram.c

26 module_param(total_size, ulong, 0);
27 MODULE_PARM_DESC(total_size, "Total device size in KiB");
28 module_param(erase_size, ulong, 0);
29 MODULE_PARM_DESC(erase_size, "Device erase block size in KiB");
30 #endif
31
32 // We could store these in the mtd structure, but we only support 1 device..
33 static struct mtd_info *mtd_info;
34
35 static int check_offs_len(struct mtd_info *mtd, loff_t ofs, uint64_t len)
36 {
37 int ret = 0;
38 unsigned long temp_len, rem;
39
40 /* Start address must align on block boundary */
41 temp_len = ofs;
> 42 rem = do_div(temp_len, mtd->erasesize);
43 if (rem) {
44 pr_debug("%s: unaligned address\n", __func__);
45 ret = -EINVAL;
46 }
47
48 /* Length must align on block boundary */
49 temp_len = len;
50 rem = do_div(temp_len, mtd->erasesize);

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: Binary data